Commit 58a98f9a authored by Boris Sazonov's avatar Boris Sazonov Committed by Commit Bot

[Android] Enable ModifierOrder checkstyle check

Error example (lines wrapped):
ui/android/java/src/org/chromium/ui/UiUtils.java:85:12: "private"
  modifier out of order with the JLS suggestions. The correct order:
  public protected private abstract default static final transient
  volatile synchronized native strictfp. See https://crbug.com/1003711
  for details.

Bug: 1003711
Change-Id: Ib71d5e948dfc8a59581c7080d9241ac5523b2556
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1803140
Auto-Submit: Boris Sazonov <bsazonov@chromium.org>
Reviewed-by: default avatarTed Choc <tedchoc@chromium.org>
Commit-Queue: Boris Sazonov <bsazonov@chromium.org>
Cr-Commit-Position: refs/heads/master@{#696775}
parent b3d552e5
...@@ -17,6 +17,10 @@ ...@@ -17,6 +17,10 @@
<module name="IllegalCatch"> <module name="IllegalCatch">
<property name="severity" value="info"/> <property name="severity" value="info"/>
</module> </module>
<module name="ModifierOrder">
<message key="mod.order" value="&quot;{0}&quot; modifier out of order with the JLS suggestions. The correct order: public protected private abstract default static final transient volatile synchronized native strictfp. See https://crbug.com/1003711 for details."/>
<property name="severity" value="error"/>
</module>
<module name="RedundantImport"> <module name="RedundantImport">
<message key="import.redundant" value="Redundant import: {0}. Use :JavaImportOrganize (ECLIM) or Ctrl+Shift+O (Eclipse) to sort imports"/> <message key="import.redundant" value="Redundant import: {0}. Use :JavaImportOrganize (ECLIM) or Ctrl+Shift+O (Eclipse) to sort imports"/>
<property name="severity" value="error"/> <property name="severity" value="error"/>
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment