clang-format should warn about unused imports |
||
Issue descriptionAnother thing we'd lose if we currently switched to clang-format only and removed checkstyle: ** Presubmit ERRORS ** build/android/incremental_install/java/org/chromium/incrementalinstall/BootstrapApplication.java:9:8: Unused import: android.content.ComponentName. Use :JavaImportOrganize (ECLIM) or Ctrl+Shift+O (Eclipse) to sort imports Of course, clang-format should just remove the unused import rather than warning :P.
,
Nov 12
This issue has been Available for over a year. If it's no longer important or seems unlikely to be fixed, please consider closing it out. If it is important, please re-triage the issue. Sorry for the inconvenience if the bug really should have been left as Available. For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot |
||
►
Sign in to add a comment |
||
Comment 1 by dtapu...@chromium.org
, Nov 10 2017