Re[2]: [cedet-semantic] cedet-1.0beta3b issues

>>> Markus Gritsch <gritsch@...> seems to think that:
>Eric M. Ludlam wrote:
>
> > >* Unfortunately I cannot use the new CVS Emacs from
> > >http://www.crasseux.com/emacs/ because (lmcompile-do-highlight) needs a
> > >replacement for `compile-reinitialize-errors. At leased it is a known
> > >issue due to the comment in lmcompile.el.
> > >
> > >* As suggested in a previously posted patch lmcompile.el line 164
>should
> > >use 'linemark-go-face instead of 'linemark-funny-face to highlight the
> > >matches, because the funny face is already used by bookmarks.
> >
> > Thanks for the reminder. I put it in this time. I also found why
> > lmcompile doesn't work with my CVS Emacs, which I think is too old for
> > your previously mentioned problem.
>
>So do you have a version of lmcompile.el which works with the current
>CVS Emacs?
My emacs is from Jan 22, 2004, so it is not the latest and greatest,
and as such, I just don't know.
> > The semantic idle handling code tries to be smart about when
> > debug-on-error is on. I've just added this patch to my local copy
> > which should then also guarantee that C-g is enabled. Do you want to
> > try it?
>
>No thanks, I think my 'hanging'-problem is related to something else
>(see below), because 1) I have set debug-on-error to nil and 2) I can
>successfully interrupt the hanging by your suggested C-g.
>
> > I haven't had the semantic parser lock me out in a very long time,
> > though it could be the language you are using. Is it related to a
> > particular file?
>
>Maybe it's indeed related to a particular language. I had the hanging
>when trying out if the reversed methods order patch in Python classes
>was applied in the beta3, and now I can reliably reproduce the hanging:
>
>I tested it using GNU Emacs from http://www.crasseux.com/emacs/ dating
>from 2003-12-28 on Windows XP.
>
>*) start Emacs
>*) C-x C-f test.py <RET>
>*) type in the following code:
>
>class C:
> def
[ ... ]
The first problem is in the lexical analyzer. It gets confused here,
and I have the following patch.
Getting past the lexer leads to a hang in wisent. Perhaps David has
a notion of how to handle that part.
Eric
--------------
*** wisent-python.el.~1.44.~ 2004-06-29 22:57:01.000000000 -0400
--- wisent-python.el 2004-08-03 22:32:00.000000000 -0400
***************
*** 1,6 ****
;;; wisent-python.el --- LALR grammar for Python
;;
! ;; Copyright (C) 2002 Richard Kim
;;
;; Author: Richard Kim <ryk@...>
;; Maintainer: Richard Kim <ryk@...>
--- 1,6 ----
;;; wisent-python.el --- LALR grammar for Python
;;
! ;; Copyright (C) 2002, 2004 Richard Kim
;;
;; Author: Richard Kim <ryk@...>
;; Maintainer: Richard Kim <ryk@...>
***************
*** 263,271 ****
(error "You have found a bug in python-next-line")))
;; the point now should be at the end of a line
(forward-line 1)
! (while (and (looking-at "\\s-*\\(\\s<\\|$\\)")
! (not (eobp))) ;; skip blank and comment lines
! (forward-line 1))))
(defun python-scan-lists ( &optional target-column )
"Without actually changing the position, return the buffer position of
--- 263,272 ----
(error "You have found a bug in python-next-line")))
;; the point now should be at the end of a line
(forward-line 1)
! (while (looking-at "\\s-*\\(\\s<\\|$\\)") ;; skip blank and comment lines
! (forward-line 1)
! (if (eobp) (error "Unterminated List."))
! )))
(defun python-scan-lists ( &optional target-column )
"Without actually changing the position, return the buffer position of
--
Eric Ludlam: zappo@..., eric@...
Home: http://www.ludlam.net Siege: http://www.siege-engine.com
Emacs: http://cedet.sourceforge.net GNU: http://www.gnu.org

Thread view

Hi!
This weekend I tried the recently released cedet-1.0beta3b and found the
following issues:
* Unfortunately I cannot use the new CVS Emacs from
http://www.crasseux.com/emacs/ because (lmcompile-do-highlight) needs a
replacement for `compile-reinitialize-errors. At leased it is a known
issue due to the comment in lmcompile.el.
* As suggested in a previously posted patch lmcompile.el line 164 should
use 'linemark-go-face instead of 'linemark-funny-face to highlight the
matches, because the funny face is already used by bookmarks.
* Sometimes I get 100% CPU usage when editing a file, and the Emacs
process must be killed. I don't know what causes this and have to
investigate further.
Kind regards,
Markus

>>> Markus Gritsch <gritsch@...> seems to think that:
>Hi!
>
>This weekend I tried the recently released cedet-1.0beta3b and found the
>following issues:
>
>* Unfortunately I cannot use the new CVS Emacs from
>http://www.crasseux.com/emacs/ because (lmcompile-do-highlight) needs a
>replacement for `compile-reinitialize-errors. At leased it is a known
>issue due to the comment in lmcompile.el.
>
>* As suggested in a previously posted patch lmcompile.el line 164 should
>use 'linemark-go-face instead of 'linemark-funny-face to highlight the
>matches, because the funny face is already used by bookmarks.
Thanks for the reminder. I put it in this time. I also found why
lmcompile doesn't work with my CVS Emacs, which I think is too old for
your previously mentioned problem.
>* Sometimes I get 100% CPU usage when editing a file, and the Emacs
>process must be killed. I don't know what causes this and have to
>investigate further.
[ ... ]
I've had Emacs go into a tizzy where C-g just took a long time to
recover, particularly if I had debug on error enabled, and the stack
was very long.
I haven't had the semantic parser lock me out in a very long time,
though it could be the language you are using. Is it related to a
particular file?
The semantic idle handling code tries to be smart about when
debug-on-error is on. I've just added this patch to my local copy
which should then also guarantee that C-g is enabled. Do you want to
try it?
Eric
*** semantic-fw.el.~1.44.~ 2004-07-28 22:32:22.000000000 -0400
--- semantic-fw.el 2004-08-02 20:54:57.000000000 -0400
***************
*** 246,254 ****
Avoid using a large BODY since it is duplicated."
;;(declare (debug t) (indent 1))
`(if debug-on-error
! (progn ,@body)
(condition-case err
! (progn ,@body)
(error
(message ,format (error-message-string err))
nil))))
--- 246,254 ----
Avoid using a large BODY since it is duplicated."
;;(declare (debug t) (indent 1))
`(if debug-on-error
! (let ((inhibit-quit nil)) ,@body)
(condition-case err
! (let ((inhibit-quit nil) ,@body))
(error
(message ,format (error-message-string err))
nil))))
--
Eric Ludlam: zappo@..., eric@...
Home: http://www.ludlam.net Siege: http://www.siege-engine.com
Emacs: http://cedet.sourceforge.net GNU: http://www.gnu.org

Eric M. Ludlam wrote:
> >* Unfortunately I cannot use the new CVS Emacs from
> >http://www.crasseux.com/emacs/ because (lmcompile-do-highlight) needs a
> >replacement for `compile-reinitialize-errors. At leased it is a known
> >issue due to the comment in lmcompile.el.
> >
> >* As suggested in a previously posted patch lmcompile.el line 164
should
> >use 'linemark-go-face instead of 'linemark-funny-face to highlight the
> >matches, because the funny face is already used by bookmarks.
>
> Thanks for the reminder. I put it in this time. I also found why
> lmcompile doesn't work with my CVS Emacs, which I think is too old for
> your previously mentioned problem.
So do you have a version of lmcompile.el which works with the current
CVS Emacs?
> The semantic idle handling code tries to be smart about when
> debug-on-error is on. I've just added this patch to my local copy
> which should then also guarantee that C-g is enabled. Do you want to
> try it?
No thanks, I think my 'hanging'-problem is related to something else
(see below), because 1) I have set debug-on-error to nil and 2) I can
successfully interrupt the hanging by your suggested C-g.
> I haven't had the semantic parser lock me out in a very long time,
> though it could be the language you are using. Is it related to a
> particular file?
Maybe it's indeed related to a particular language. I had the hanging
when trying out if the reversed methods order patch in Python classes
was applied in the beta3, and now I can reliably reproduce the hanging:
I tested it using GNU Emacs from http://www.crasseux.com/emacs/ dating
from 2003-12-28 on Windows XP.
*) start Emacs
*) C-x C-f test.py <RET>
*) type in the following code:
class C:
def
and wait some seconds -> 100% CPU probably due to the incomplete Python
code.
Kind regards,
Markus

>>> Markus Gritsch <gritsch@...> seems to think that:
>Eric M. Ludlam wrote:
>
> > >* Unfortunately I cannot use the new CVS Emacs from
> > >http://www.crasseux.com/emacs/ because (lmcompile-do-highlight) needs a
> > >replacement for `compile-reinitialize-errors. At leased it is a known
> > >issue due to the comment in lmcompile.el.
> > >
> > >* As suggested in a previously posted patch lmcompile.el line 164
>should
> > >use 'linemark-go-face instead of 'linemark-funny-face to highlight the
> > >matches, because the funny face is already used by bookmarks.
> >
> > Thanks for the reminder. I put it in this time. I also found why
> > lmcompile doesn't work with my CVS Emacs, which I think is too old for
> > your previously mentioned problem.
>
>So do you have a version of lmcompile.el which works with the current
>CVS Emacs?
My emacs is from Jan 22, 2004, so it is not the latest and greatest,
and as such, I just don't know.
> > The semantic idle handling code tries to be smart about when
> > debug-on-error is on. I've just added this patch to my local copy
> > which should then also guarantee that C-g is enabled. Do you want to
> > try it?
>
>No thanks, I think my 'hanging'-problem is related to something else
>(see below), because 1) I have set debug-on-error to nil and 2) I can
>successfully interrupt the hanging by your suggested C-g.
>
> > I haven't had the semantic parser lock me out in a very long time,
> > though it could be the language you are using. Is it related to a
> > particular file?
>
>Maybe it's indeed related to a particular language. I had the hanging
>when trying out if the reversed methods order patch in Python classes
>was applied in the beta3, and now I can reliably reproduce the hanging:
>
>I tested it using GNU Emacs from http://www.crasseux.com/emacs/ dating
>from 2003-12-28 on Windows XP.
>
>*) start Emacs
>*) C-x C-f test.py <RET>
>*) type in the following code:
>
>class C:
> def
[ ... ]
The first problem is in the lexical analyzer. It gets confused here,
and I have the following patch.
Getting past the lexer leads to a hang in wisent. Perhaps David has
a notion of how to handle that part.
Eric
--------------
*** wisent-python.el.~1.44.~ 2004-06-29 22:57:01.000000000 -0400
--- wisent-python.el 2004-08-03 22:32:00.000000000 -0400
***************
*** 1,6 ****
;;; wisent-python.el --- LALR grammar for Python
;;
! ;; Copyright (C) 2002 Richard Kim
;;
;; Author: Richard Kim <ryk@...>
;; Maintainer: Richard Kim <ryk@...>
--- 1,6 ----
;;; wisent-python.el --- LALR grammar for Python
;;
! ;; Copyright (C) 2002, 2004 Richard Kim
;;
;; Author: Richard Kim <ryk@...>
;; Maintainer: Richard Kim <ryk@...>
***************
*** 263,271 ****
(error "You have found a bug in python-next-line")))
;; the point now should be at the end of a line
(forward-line 1)
! (while (and (looking-at "\\s-*\\(\\s<\\|$\\)")
! (not (eobp))) ;; skip blank and comment lines
! (forward-line 1))))
(defun python-scan-lists ( &optional target-column )
"Without actually changing the position, return the buffer position of
--- 263,272 ----
(error "You have found a bug in python-next-line")))
;; the point now should be at the end of a line
(forward-line 1)
! (while (looking-at "\\s-*\\(\\s<\\|$\\)") ;; skip blank and comment lines
! (forward-line 1)
! (if (eobp) (error "Unterminated List."))
! )))
(defun python-scan-lists ( &optional target-column )
"Without actually changing the position, return the buffer position of
--
Eric Ludlam: zappo@..., eric@...
Home: http://www.ludlam.net Siege: http://www.siege-engine.com
Emacs: http://cedet.sourceforge.net GNU: http://www.gnu.org