Skip to content
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

Defect (Issue #445) : Call Introspector.decapitalize instead of home made function #449

Closed
wants to merge 1 commit into from

Conversation

herveDarritchon
Copy link
Contributor

To correct a bug during creation of getter and setter, I change the call of initialCaps for getter and setter with Introspector.decapitalize from package java.beans.

…etter with Introspector.decapitalize from pakage java.beans.
@fehguy
Copy link
Contributor

fehguy commented Feb 23, 2015

Hi thanks for submitting. Two things that would help a lot

  1. Any PR should go against the development branch, not master, since master is "clean and released". The development branch is develop_2.0
  2. If you can generally avoid reformatting code so the changes can be more easily detected, it can be merged much more easily.

I'm closing this out so you can send a PR against the dev branch.

@fehguy fehguy closed this Feb 23, 2015
@herveDarritchon
Copy link
Contributor Author

sorry for the mess :)
I do my change against the develop_2.0 branch.

Is it possible to think of reformatting all the code one day, because it's easy to read ;)

thanks

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants