This is an automated email from the ASF dual-hosted git repository.
dsoumis pushed a commit to branch 11.0.x
in repository https://gitbox.apache.org/repos/asf/tomcat.git
The following commit(s) were added to refs/heads/11.0.x by this push:
new 444cd862d7 Improve codestyle and inspection in Intellij IDE
444cd862d7 is described below
commit 444cd862d7d17b62f97f47d37220a452a491989c
Author: Dimitris Soumis <[email protected]>
AuthorDate: Tue Sep 16 21:58:04 2025 +0300
Improve codestyle and inspection in Intellij IDE
---
res/ide-support/idea/codeStyles/Project.xml | 97 +++++++++++++++++++++-
.../idea/inspectionProfiles/Project_Default.xml | 35 ++++++--
2 files changed, 124 insertions(+), 8 deletions(-)
diff --git a/res/ide-support/idea/codeStyles/Project.xml
b/res/ide-support/idea/codeStyles/Project.xml
index adf710b10e..2550a5cc80 100644
--- a/res/ide-support/idea/codeStyles/Project.xml
+++ b/res/ide-support/idea/codeStyles/Project.xml
@@ -17,9 +17,102 @@
-->
<component name="ProjectCodeStyleConfiguration">
<code_scheme name="Project" version="173">
+ <option name="USE_TAB_CHARACTER" value="false"/>
+ <option name="RIGHT_MARGIN" value="120"/>
+ <option name="FORMATTER_TAGS_ENABLED" value="true"/>
+ <JavaCodeStyleSettings>
+ <option name="ENABLE_JAVADOC_FORMATTING" value="true"/>
+ <option name="JD_ALIGN_PARAM_COMMENTS" value="false"/>
+ <option name="JD_ALIGN_EXCEPTION_COMMENTS" value="false"/>
+ <option name="JD_P_AT_EMPTY_LINES" value="false"/>
+ <option name="JD_DO_NOT_WRAP_ONE_LINE_COMMENTS" value="true"/>
+ <option name="JD_KEEP_EMPTY_PARAMETER" value="false"/>
+ <option name="JD_KEEP_EMPTY_EXCEPTION" value="false"/>
+ <option name="JD_KEEP_EMPTY_RETURN" value="false"/>
+ <option name="JD_PRESERVE_LINE_FEEDS" value="true"/>
+ <option name="CLASS_COUNT_TO_USE_IMPORT_ON_DEMAND" value="9999"/>
+ <option name="NAMES_COUNT_TO_USE_IMPORT_ON_DEMAND" value="9999"/>
+ <option name="IMPORT_LAYOUT_TABLE">
+ <value>
+ <package name="" withSubpackages="true" static="false"/>
+ <emptyLine/>
+ <package name="" withSubpackages="true" static="true"/>
+ </value>
+ </option>
+ </JavaCodeStyleSettings>
<codeStyleSettings language="JAVA">
- <option name="RIGHT_MARGIN" value="100" />
- <option name="WRAP_ON_TYPING" value="0" />
+ <option name="WRAP_LONG_LINES" value="true"/>
+ <option name="KEEP_LINE_BREAKS" value="true"/>
+ <option name="CALL_PARAMETERS_WRAP" value="1"/>
+ <option name="METHOD_PARAMETERS_WRAP" value="1"/>
+ <option name="EXTENDS_LIST_WRAP" value="1"/>
+ <option name="THROWS_LIST_WRAP" value="1"/>
+ <option name="METHOD_CALL_CHAIN_WRAP" value="1"/>
+ <option name="BINARY_OPERATION_WRAP" value="1"/>
+ <option name="BINARY_OPERATION_SIGN_ON_NEXT_LINE" value="true"/>
+ <option name="TERNARY_OPERATION_WRAP" value="1"/>
+ <option name="TERNARY_OPERATION_SIGNS_ON_NEXT_LINE" value="true"/>
+ <option name="FOR_STATEMENT_WRAP" value="1"/>
+ <option name="ARRAY_INITIALIZER_WRAP" value="1"/>
+ <option name="KEEP_SIMPLE_BLOCKS_IN_ONE_LINE" value="false"/>
+ <option name="KEEP_SIMPLE_METHODS_IN_ONE_LINE" value="false"/>
+ <option name="KEEP_SIMPLE_CLASSES_IN_ONE_LINE" value="false"/>
+ <option name="KEEP_SIMPLE_LAMBDAS_IN_ONE_LINE" value="false"/>
+ <option name="KEEP_CONTROL_STATEMENT_IN_ONE_LINE" value="false"/>
+ <option name="IF_BRACE_FORCE" value="1"/>
+ <option name="WHILE_BRACE_FORCE" value="1"/>
+ <option name="FOR_BRACE_FORCE" value="1"/>
+ <option name="DOWHILE_BRACE_FORCE" value="1"/>
+ <option name="SPACE_BEFORE_CLASS_LBRACE" value="true"/>
+ <option name="SPACE_BEFORE_METHOD_LBRACE" value="true"/>
+ <option name="SPACE_BEFORE_IF_LBRACE" value="true"/>
+ <option name="SPACE_BEFORE_ELSE_LBRACE" value="true"/>
+ <option name="SPACE_BEFORE_FOR_LBRACE" value="true"/>
+ <option name="SPACE_BEFORE_WHILE_LBRACE" value="true"/>
+ <option name="SPACE_BEFORE_SWITCH_LBRACE" value="true"/>
+ <option name="SPACE_BEFORE_TRY_LBRACE" value="true"/>
+ <option name="SPACE_BEFORE_CATCH_LBRACE" value="true"/>
+ <option name="SPACE_BEFORE_FINALLY_LBRACE" value="true"/>
+ <option name="SPACE_BEFORE_METHOD_PARENTHESES" value="false"/>
+ <option name="SPACE_BEFORE_METHOD_CALL_PARENTHESES" value="false"/>
+ <option name="BLANK_LINES_AFTER_PACKAGE" value="1"/>
+ <option name="BLANK_LINES_BEFORE_IMPORTS" value="1"/>
+ <option name="BLANK_LINES_AFTER_IMPORTS" value="1"/>
+ <option name="BLANK_LINES_AROUND_CLASS" value="1"/>
+ <option name="BLANK_LINES_AROUND_FIELD" value="0"/>
+ <option name="BLANK_LINES_AROUND_METHOD" value="1"/>
+ <option name="KEEP_BLANK_LINES_IN_DECLARATIONS" value="1"/>
+ <option name="KEEP_BLANK_LINES_IN_CODE" value="1"/>
+ <option name="KEEP_BLANK_LINES_BEFORE_RBRACE" value="1"/>
+ <option name="WRAP_COMMENTS" value="true"/>
+ <option name="LINE_COMMENT_AT_FIRST_COLUMN" value="false"/>
+ <option name="BLOCK_COMMENT_AT_FIRST_COLUMN" value="false"/>
+ <option name="INDENT_SIZE" value="4"/>
+ <option name="TAB_SIZE" value="4"/>
+ <option name="CONTINUATION_INDENT_SIZE" value="8"/>
+ <option name="BRACE_STYLE" value="1"/>
+ <option name="CLASS_BRACE_STYLE" value="1"/>
+ <option name="METHOD_BRACE_STYLE" value="1"/>
+ <option name="ELSE_ON_NEW_LINE" value="false"/>
+ <option name="CATCH_ON_NEW_LINE" value="false"/>
+ <option name="FINALLY_ON_NEW_LINE" value="false"/>
+ <option name="SPACE_BEFORE_IF_PARENTHESES" value="true"/>
+ <option name="SPACE_BEFORE_FOR_PARENTHESES" value="true"/>
+ <option name="SPACE_BEFORE_WHILE_PARENTHESES" value="true"/>
+ <option name="SPACE_BEFORE_SWITCH_PARENTHESES" value="true"/>
+ <option name="SPACE_BEFORE_TRY_PARENTHESES" value="true"/>
+ <option name="SPACE_AROUND_ASSIGNMENT_OPERATORS" value="true"/>
+ <option name="SPACE_AROUND_LOGICAL_OPERATORS" value="true"/>
+ <option name="SPACE_AROUND_RELATIONAL_OPERATORS" value="true"/>
+ <option name="SPACE_AROUND_ADDITIVE_OPERATORS" value="true"/>
+ <option name="SPACE_AROUND_MULTIPLICATIVE_OPERATORS" value="true"/>
+ <option name="SPACE_AROUND_BITWISE_OPERATORS" value="true"/>
+ <option name="SPACE_AROUND_EQUALITY_OPERATORS" value="true"/>
+ <option name="SPACE_WITHIN_METHOD_PARENTHESES" value="false"/>
+ <option name="SPACE_WITHIN_IF_PARENTHESES" value="false"/>
+ <option name="SPACE_WITHIN_FOR_PARENTHESES" value="false"/>
+ <option name="SPACE_WITHIN_WHILE_PARENTHESES" value="false"/>
+ <option name="SPACE_WITHIN_SWITCH_PARENTHESES" value="false"/>
</codeStyleSettings>
<codeStyleSettings language="XML">
<indentOptions>
diff --git a/res/ide-support/idea/inspectionProfiles/Project_Default.xml
b/res/ide-support/idea/inspectionProfiles/Project_Default.xml
index dd01fa3eae..4228d88854 100644
--- a/res/ide-support/idea/inspectionProfiles/Project_Default.xml
+++ b/res/ide-support/idea/inspectionProfiles/Project_Default.xml
@@ -18,11 +18,34 @@
<component name="InspectionProjectProfileManager">
<profile version="1.0">
<option name="myName" value="Project Default" />
- <inspection_tool class="AutoBoxing" enabled="true" level="WARNING"
enabled_by_default="true">
- <option name="ignoreAddedToCollection" value="false" />
- </inspection_tool>
- <inspection_tool class="AutoUnboxing" enabled="true" level="WARNING"
enabled_by_default="true" />
- <inspection_tool class="UnnecessaryBoxing" enabled="false" level="WARNING"
enabled_by_default="false" />
- <inspection_tool class="UnnecessaryUnboxing" enabled="false"
level="WARNING" enabled_by_default="false" />
+ <inspection_tool class="ControlFlowStatementWithoutBraces" enabled="true"
level="WARNING" enabled_by_default="true"/>
+ <inspection_tool class="ForCanBeForeach" enabled="true"
level="WEAK_WARNING" enabled_by_default="true"/>
+ <inspection_tool class="FieldMayBeFinal" enabled="true"
level="WEAK_WARNING" enabled_by_default="true"/>
+ <inspection_tool class="RedundantModifier" enabled="true" level="WARNING"
enabled_by_default="true"/>
+ <inspection_tool class="UnnecessarySemicolon" enabled="true"
level="WEAK_WARNING" enabled_by_default="true"/>
+ <inspection_tool class="UnnecessaryParentheses" enabled="true"
level="WEAK_WARNING" enabled_by_default="true"/>
+ <inspection_tool class="StaticAccessedFromInstance" enabled="true"
level="WEAK_WARNING" enabled_by_default="true"/>
+ <inspection_tool class="SerializableHasSerialVersionUIDField"
enabled="true" level="WARNING" enabled_by_default="true"/>
+ <inspection_tool class="override" enabled="true" level="WARNING"
enabled_by_default="true"/>
+ <inspection_tool class="NonSynchronizedMethodOverridesSynchronizedMethod"
enabled="true" level="WARNING" enabled_by_default="true"/>
+ <inspection_tool class="resource" enabled="true" level="WARNING"
enabled_by_default="true"/>
+ <inspection_tool class="IOResourceOpenedButNotSafelyClosed" enabled="true"
level="WARNING" enabled_by_default="true"/>
+ <inspection_tool class="SocketOpenedButNotSafelyClosed" enabled="true"
level="WARNING" enabled_by_default="true"/>
+ <inspection_tool class="ChannelOpenedButNotSafelyClosed" enabled="true"
level="WARNING" enabled_by_default="true"/>
+ <inspection_tool class="UnusedImport" enabled="true" level="WARNING"
enabled_by_default="true"/>
+ <inspection_tool class="UnusedDeclaration" enabled="true" level="WARNING"
enabled_by_default="true"/>
+ <inspection_tool class="UnusedAssignment" enabled="true" level="WARNING"
enabled_by_default="true"/>
+ <inspection_tool class="RedundantThrows" enabled="true" level="WARNING"
enabled_by_default="true"/>
+ <inspection_tool class="DuplicateThrows" enabled="true" level="WARNING"
enabled_by_default="true"/>
+ <inspection_tool class="RedundantCast" enabled="true" level="WARNING"
enabled_by_default="true"/>
+ <inspection_tool class="StatementWithEmptyBody" enabled="true"
level="WARNING" enabled_by_default="true"/>
+ <inspection_tool class="EmptyMethod" enabled="true" level="WARNING"
enabled_by_default="true"/>
+ <inspection_tool class="UnnecessaryBoxing" enabled="true" level="WARNING"
enabled_by_default="true"/>
+ <inspection_tool class="UnnecessaryUnboxing" enabled="true"
level="WARNING" enabled_by_default="true"/>
+ <inspection_tool class="BoxingBoxedValue" enabled="true" level="WARNING"
enabled_by_default="true"/>
+ <inspection_tool class="AutoUnboxing" enabled="true" level="WARNING"
enabled_by_default="true"/>
+ <inspection_tool class="FinalizeNotProtected" enabled="true"
level="WARNING" enabled_by_default="true"/>
+ <inspection_tool class="FinalizeDeclaration" enabled="true"
level="WARNING" enabled_by_default="true"/>
+ <inspection_tool class="FinalizeCalledExplicitly" enabled="true"
level="WARNING" enabled_by_default="true"/>
</profile>
</component>
\ No newline at end of file
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
