-
-
Notifications
You must be signed in to change notification settings - Fork 347
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
add method CtStatement#comment()
#2013
Comments
Honestly, I'm against this. it had API footprint to spoon (that is already huge) and it can be done by the client side in one or two lines. We can maybe create an utility class that contains "useful" method like this. |
Hey, Will this feature be added? If yes, can I work on this? |
Welcome to Spoon. Yes, feel free to make a pull-request, that's a good exercise :-) |
Sir, I tried reading about the problem and also looked into the similar thread about adding blank lines and about delete. I see that CtStatement extends CtElement interface, which already has the addComment method along with others related to comments. Could you please clarify if what we want from CtStatement#comment() is
becomes
becomes
I understand that 2 is already implemented. Is it something else we are trying to add? |
yes, the intention is the first semantics you describe. |
Add Boilerplate test class statementComment Fixes INRIA#2013
body and as child element of another block. Add test methods. Fixes INRIA#2013
…d of CtStatementImpl Fixes INRIA#2013
Idea: Should there be a way to and is there a usecase for "uncomment" of a legal/valid statement which has been commented by using CtStatement#comment? |
thanks for the proposal, that's not required for now IMHO. |
Remove #comment from CtClassImpl, CtUnaryOperatorImpl, CtInvocationImpl and CtConstructorCallImpl Based on comments and suggestions at INRIA#3494 Fixes INRIA#2013
…edOperationException Fixes INRIA#2013
to comment code in the pretty-printed code, this is a traceable version of (
#delete
)The text was updated successfully, but these errors were encountered: