bug#30726: [26.0.91 9] optimistic forward-sexp

bug#30726: [26.0.91 9] optimistic forward-sexp

[Andreas Röhler]

In GNU Emacs 26.0.91 (build 1, i686-pc-linux-gnu, GTK+ Version 3.14.5)
  of 2018-01-27

 From 
https://emacs.stackexchange.com/questions/39253/sh-mode-scan-error-containing-expression-ends-prematurely

Error occurs when forward-sexp is called from closing paren at last line 
of example code below

(echo '#!/bin/bash'
echo 'myfunc() {'
echo "cat <<'z'"
seq 135
echo zzzzzzzzzzzzzzzzzzzzzzzzzzzzzzzzzzzzzzzzzzzz
echo z
echo '}') >out

Bug-source is in but-last line of the def in lisp.el, where

     (goto-char (or (scan-sexps (point) arg) (buffer-end arg)))

scan-sexps sends the error.

Wrapping in into (ignore-errors...


(or (ignore-errors (scan-sexps (point) arg))

makes it gone.

Cheers,

Andreas

bug#30726: [26.0.91 9] optimistic forward-sexp

[Glenn Morris]

Andreas Röhler wrote:

> https://emacs.stackexchange.com/questions/39253/sh-mode-scan-error-containing-expression-ends-prematurely

Already reported as bug#30721.
(It would be great if people forwarding other people's stackexchange
issues would do basic checks first.)

> Error occurs when forward-sexp is called from closing paren at last
> line of example code below
>
> (echo '#!/bin/bash'
> echo 'myfunc() {'
> echo "cat <<'z'"
> seq 135
> echo zzzzzzzzzzzzzzzzzzzzzzzzzzzzzzzzzzzzzzzzzzzz
> echo z
> echo '}') >out

Note that the OP meant you to run that as shell code to generate a file,
"out", and then visit it.

You appear to have discovered that using forward-sexp inside eg

(

)

will print "containing expression ends prematurely" ?

bug#30726: [26.0.91 9] optimistic forward-sexp

[Andreas Röhler]

On 06.03.2018 08:47, Glenn Morris wrote:
> Andreas Röhler wrote:
> 
>> https://emacs.stackexchange.com/questions/39253/sh-mode-scan-error-containing-expression-ends-prematurely
> 
> Already reported as bug#30721.
> (It would be great if people forwarding other people's stackexchange
> issues would do basic checks first.)
> 
>> Error occurs when forward-sexp is called from closing paren at last
>> line of example code below
>>
>> (echo '#!/bin/bash'
>> echo 'myfunc() {'
>> echo "cat <<'z'"
>> seq 135
>> echo zzzzzzzzzzzzzzzzzzzzzzzzzzzzzzzzzzzzzzzzzzzz
>> echo z
>> echo '}') >out
> 
> Note that the OP meant you to run that as shell code to generate a file,
> "out", and then visit it.
> 
> You appear to have discovered that using forward-sexp inside eg
> 
> (
> 
> )
> 
> will print "containing expression ends prematurely" ?
> 


Right. Which it shouldn't IMO, but navigate sexp's until EOB is reached.

bug#30726: [26.0.91 9] optimistic forward-sexp

[Andreas Röhler]

On 06.03.2018 08:47, Glenn Morris wrote:
> Andreas Röhler wrote:
> 
>> https://emacs.stackexchange.com/questions/39253/sh-mode-scan-error-containing-expression-ends-prematurely
> 
> Already reported as bug#30721.
> (It would be great if people forwarding other people's stackexchange
> issues would do basic checks first.)
> 
>> Error occurs when forward-sexp is called from closing paren at last
>> line of example code below
>>
>> (echo '#!/bin/bash'
>> echo 'myfunc() {'
>> echo "cat <<'z'"
>> seq 135
>> echo zzzzzzzzzzzzzzzzzzzzzzzzzzzzzzzzzzzzzzzzzzzz
>> echo z
>> echo '}') >out
> 
> Note that the OP meant you to run that as shell code to generate a file,
> "out", and then visit it.
> 
> You appear to have discovered that using forward-sexp inside eg
> 
> (
> 
> )
> 
> will print "containing expression ends prematurely" ?
> 

When calling the way the OP intended

get the error as below from inside the singlequoted, which also is an 
old issue:

To avoid these wrote ar-forward-sexp

which see here: 
https://github.com/andreas-roehler/thingatpt-utils-core/blob/master/ar-subr.el



;;;;;


Debugger entered--Lisp error: (scan-error "Containing expression ends 
prematurely" 515 515)
   signal(scan-error ("Containing expression ends prematurely" 515 515))
   smie-forward-sexp-command(1)
   forward-sexp(1)
   funcall-interactively(forward-sexp 1)
   call-interactively(forward-sexp nil nil)
   command-execute(forward-sexp)

bug#30726: [26.0.91 9] optimistic forward-sexp

[Lars Ingebrigtsen]

Andreas Röhler <andreas.roehler@easy-emacs.de> writes:

>> Note that the OP meant you to run that as shell code to generate a
>> file,
>> "out", and then visit it.
>> You appear to have discovered that using forward-sexp inside eg
>> (
>> )
>> will print "containing expression ends prematurely" ?
>> 
>
> Right. Which it shouldn't IMO, but navigate sexp's until EOB is reached.

It has to -- we rely on it signalling an error for various commands to
work.  In these cases, the user may want to go up in the list and then
do a forward-sexp, though.

But I don't think there's any bug here, so I'm closing this bug report.

-- 
(domestic pets only, the antidote for overdose, milk.)
   bloggy blog: http://lars.ingebrigtsen.no