[bug] python-mode, jit-lock, syntax-pss during file load

[bug] python-mode, jit-lock, syntax-pss during file load

[Karl Chen]

(syntax-ppss) doesn't work correctly at file load when
font-lock-support-mode is 'jit-lock-mode.

Symptom: when (re)loading a python file whose initial contents
look like:

'''   ...  '
blah1:
  blah2

'''

If jit-lock-mode is on, then python-mode guesses python-indent of
2 because it thinks "blah1:" is not in a string.

If you evaluate (python-guess-indent) after the file has loaded,
it returns 4.  It only returns 2 while the file is loading.  (It
was annoying that python-guess-indent returns different results
when debugging is enabled!)

Other font lock support modes (or no font lock) might also have
problems.  I think the fix is to force syntaxification/
fontification before python-guess-indent is called.





-- 
Karl 2004-06-11 17:07

Re: [bug] python-mode, jit-lock, syntax-pss during file load

[Dave Love]

Karl Chen <quarl@hkn.eecs.berkeley.edu> writes:

> Other font lock support modes (or no font lock) might also have
> problems.

Proper handling of quotes definitely depends on font-lock
(unfortunately).

> I think the fix is to force syntaxification/
> fontification before python-guess-indent is called.

At that stage, I guess it's only necessary to
font-lock-fontify-syntactically-region up to the limit that it checks
for the indent.

another jit-lock bug

[Karl Chen]

emacs -q

(global-set-key "\r" (lambda () (interactive) (newline) (indent-according-to-mode)))
(global-font-lock-mode)
(setq font-lock-support-mode 'jit-lock-mode)

C-x C-f /tmp/a.sh

if true ; then
    # when you press return here, auto indent succeeds
fi

case z in
    asdf)
# auto indent fails here only when font-lock-support-mode is
# jit-lock-mode, using (indent-according-to-mode) as part of an
# interactive key (if you manually evaluate
# (indent-according-to-mode) it works)




-- 
Karl 2004-06-22 01:39

Re: another jit-lock bug

[Richard Stallman]

Is anyone looking at this bug report?
I don't know the code in jit-lock at all.

Gerd, can you look at it?