-
-
Notifications
You must be signed in to change notification settings - Fork 2.4k
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
when variable starts with single letter like gName, the getter & setter is wrong #1803
Comments
Dup of #757 ? |
This has been discussed a few times. Summary: There is no clear "right" or "wrong" in this case, and changing lombok here will be a breaking change that will lead to compile errors in several projects when they upgrade the lombok version. So I think the answer is still "won't do"... |
@janrieke I know it, but eclipse and IDEA all auto generate like |
There are multiple conflicting conventions within the java world,
unfortunately. "Convention over configuration" is all nice and dandy but
when there are multiple conflicting conventions you're still stuck.
This is the choice the lombok devs made, they seem to stand by it, so odds
of it changing are pretty much zero.
…On Thu, Aug 16, 2018 at 10:01 AM Darren Luo ***@***.***> wrote:
@janrieke <https://github.com/janrieke> I know it, but eclipse and IDEA
all auto generate like getgName(), when I use most of the IDE of JAVA,
they do the same thing, so I think it's a convention in JAVA's world, I
find the problem when I use mapstruct, the plugin also use getgName(), so
the two plugins are conflict.
Maybe My title is not suitable, because my English is very pool.
"convention over configuration", so I think lombok should handle this
problem, when people upgrade the version of lombok lead compile errors,
lombok can give a configuration in the new version to transfer the first
letter to uppercase, and ask the people who meet the error to use the
configuration.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
<#1803 (comment)>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/AAKCRfXrc8WbizI1pSBAX74YEnVAVLCWks5uRSa9gaJpZM4VtaEy>
.
--
"Don't only practice your art, but force your way into it's secrets, for it
and knowledge can raise men to the divine."
-- Ludwig von Beethoven
|
Duplicate of #757 - we will not be fixing this. |
Duplicate of #2693 |
If I use ide auto generate, the result should be:
The text was updated successfully, but these errors were encountered: