CEDET

Hi!

Klaus Berndl adviced me so send the following bug report:

Semantic returns the method tag-list in Python Classes reversed.  The 
following example demonstrates the bug:

----------------
class Test:
     def b(self):
         pass

     def c(self):
         pass

     def a(self):
         pass
----------------

Typing
M-: (semantic-tag-children-compatibility (car (semantic-fetch-tags))) RET
shows that the methods are returned in the wrong order (a c b).  Klaus 
guessed, that there might be an nreverse (or reverse) on the tag-list 
missing.

Kind regards,
Markus

>>> Markus Gritsch <gritsch@...> seems to think that:
>Hi!
>
>Klaus Berndl adviced me so send the following bug report:
>
>Semantic returns the method tag-list in Python Classes reversed.  The 
>following example demonstrates the bug:
>
>----------------
>class Test:
>     def b(self):
>         pass
>
>     def c(self):
>         pass
>
>     def a(self):
>         pass
>----------------
>
>Typing
>M-: (semantic-tag-children-compatibility (car (semantic-fetch-tags))) RET
>shows that the methods are returned in the wrong order (a c b).  Klaus 
>guessed, that there might be an nreverse (or reverse) on the tag-list 
>missing.
  [ ... ]

Hi,

  Sorry for a late reply.  I've put this on my todo list of things to
solve.

  If you would like to try it yourself, there is indeed an nreverse on
line 629 of wisent-python.wy which, as far as I can tell eventually,
parses the indented "def" statements in python.  You could fiddle with
that, and use "C-c C-c" to rebuild, and then test your change.

Thanks
Eric

-- 
          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:
 >
>   Sorry for a late reply.  I've put this on my todo list of things to
> solve.
> 
>   If you would like to try it yourself, there is indeed an nreverse on
> line 629 of wisent-python.wy which, as far as I can tell eventually,
> parses the indented "def" statements in python.  You could fiddle with
> that, and use "C-c C-c" to rebuild, and then test your change.

Better late than never ;)  Thanks for the hint; I will try it at the 
weekend.

Be well,
Markus

Eric M. Ludlam wrote:
> If you would like to try it yourself, there is indeed an nreverse on
> line 629 of wisent-python.wy which, as far as I can tell eventually,
> parses the indented "def" statements in python.  You could fiddle with
> that, and use "C-c C-c" to rebuild, and then test your change.

Yes, this worked perfectly, thank you very much,
Markus

>>> Markus Gritsch <gritsch@...> seems to think that:
>Eric M. Ludlam wrote:
>> If you would like to try it yourself, there is indeed an nreverse on
>> line 629 of wisent-python.wy which, as far as I can tell eventually,
>> parses the indented "def" statements in python.  You could fiddle with
>> that, and use "C-c C-c" to rebuild, and then test your change.
>
>Yes, this worked perfectly, thank you very much,
>Markus

What was the end result in the code?  Just removing the nreverse?

Thanks
Eric

-- 
          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:
> 
> What was the end result in the code?  Just removing the nreverse?

First I tried to replace "nreverse" by "list", but this didn't work.
(nreverse (nreverse $2)) is for sure not the best solution, but since I 
am no Lisp expert, I didn't come up with a better solution, and it works 
fine for me :)

Kind regards, Markus

Hi Markus,

> First I tried to replace "nreverse" by "list", but this didn't work.
> (nreverse (nreverse $2)) is for sure not the best solution, but since I 
> am no Lisp expert, I didn't come up with a better solution, and it works 
> fine for me :)

I think (progn $2) should work ;-)

David