Manuscript (Case [Issue]22472) Compiler Opal - RBMethodNode selectorAndArgumentNames does not skip comments before selector

Classic

List

Threaded

♦

♦

Locked 1 message

Pharo Issue Tracker

Reply | Threaded

Open this post in threaded view

♦

♦

|

Manuscript (Case [Issue]22472) Compiler Opal - RBMethodNode selectorAndArgumentNames does not skip comments before selector

Denis Kudryashov edited Case 22472: RBMethodNode selectorAndArgumentNames does not skip comments before selector: Bug in  Compiler Opal: 1. Pharo Image  •  You are subscribed to this case Maybe you right about this method. The name looks how you describe it. But I think we need another method which will give us real "method header" from sources. Tonel is one possible user of it. It splits method by declaration and body with []. (it does not use parser for now because of portability issue but it is different story). So I would add two methods (but with better names): - aMethodNode printHeader - aMethodNode printBody Priority: 5 – Fix If Time Status: Work Needed Assigned to: Everyone Milestone: Pharo7.0 Go to Case No longer need updates? Unsubscribe from this case. Don't want Manuscript notifications anymore? Update your preferences. A Manuscript case was edited by Denis Kudryashov. Case ID: 22472 Title: RBMethodNode selectorAndArgumentNames does not skip comments before selector Status: Work Needed Category: Bug Project: Compiler Opal Area: 1. Pharo Image Priority: 5 - Fix If Time Milestone: Pharo7.0: 02.09.2018 (Past) Assigned To: Everyone URL: https://pharo.manuscript.com/f/cases/22472 Maybe you right about this method. The name looks how you describe it. But I think we need another method which will give us real "method header" from sources. Tonel is one possible user of it. It splits method by declaration and body with []. (it does not use parser for now because of portability issue but it is different story). So I would add two methods (but with better names): - aMethodNode printHeader - aMethodNode printBody You are subscribed to this case. If you do not want to receive automatic notifications in the future, unsubscribe (https://pharo.manuscript.com/default.asp?pre=preUnsubscribe&pg=pgEditBug&command=view&ixBug=22472) from this case. _______________________________________________ Pharo-bugtracker mailing list [hidden email] https://lists.gforge.inria.fr/mailman/listinfo/pharo-bugtracker