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

Fix eclipse warnings and add java documentation (#1670) #1671

Merged

Conversation

speckyspooky
Copy link
Contributor

Fix eclipse warnings and add java documentation (#1670)

@speckyspooky speckyspooky added this to the 4.16 milestone May 7, 2024
@speckyspooky speckyspooky self-assigned this May 7, 2024
@speckyspooky
Copy link
Contributor Author

Hi Wim, I added some java docs to solve eclipse warnings and changed some code warnings.
I changed also the "PropertyParser". This seems to me very special/old stuff. Please double check it to avoid mistakes from my side.
Thanks in advance!

Copy link
Contributor

@wimjongman wimjongman left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Very nice, Thomas. Thank you.

@speckyspooky speckyspooky merged commit 9944092 into eclipse-birt:master May 8, 2024
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants