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

Enable "error prone" library that does static code analysis during code compilation #421

Merged
merged 28 commits into from
Oct 24, 2024

Conversation

asolntsev
Copy link
Contributor

No description provided.

@asolntsev asolntsev self-assigned this Oct 16, 2024
@asolntsev asolntsev marked this pull request as draft October 16, 2024 08:24
@asolntsev asolntsev added refactoring java Pull requests that update Java code labels Oct 16, 2024
@asolntsev asolntsev added this to the 9.10.0 milestone Oct 16, 2024
@asolntsev asolntsev changed the title [error-prone] add missing @Override annotations Enable "error prone" library that does static code analysis during code compilation Oct 16, 2024
@asolntsev asolntsev force-pushed the refactor/error-prone branch from e746151 to 4d14841 Compare October 16, 2024 09:14
in these cases, system default locale/charset is used, which can cause unpredictable behavour and random errors.
instead of arrays, let's operate with old good java objects: RGBColor and HSBColor.
* remove unused field `dim`
* replace `setDocument` methods by constructor with document parameter.
previously, it didn't remove anything because `imageComponents` key type is `CacheKey`, not `Element`.

P.S. But it seems that this `remove` method is not used at all.
.. to allow running `BrowserStartup` Swing application.
@asolntsev asolntsev force-pushed the refactor/error-prone branch from 0255194 to 7785f59 Compare October 23, 2024 18:31
@asolntsev asolntsev force-pushed the refactor/error-prone branch from 5da0ffe to b34084f Compare October 23, 2024 19:54
@asolntsev asolntsev force-pushed the refactor/error-prone branch from 7fe64d9 to 181e348 Compare October 23, 2024 21:46
@asolntsev asolntsev force-pushed the refactor/error-prone branch from cec21d3 to fa617c1 Compare October 23, 2024 22:03
@asolntsev asolntsev requested a review from pbrant October 24, 2024 08:34
@asolntsev asolntsev marked this pull request as ready for review October 24, 2024 08:34
@asolntsev asolntsev removed the request for review from pbrant October 24, 2024 08:34
@asolntsev asolntsev merged commit 90c3a4c into main Oct 24, 2024
4 checks passed
@asolntsev asolntsev deleted the refactor/error-prone branch October 24, 2024 15:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
java Pull requests that update Java code refactoring
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant