Move more checkstyle checks from 'new code' style to main style

This commit is contained in:
ByteHamster 2022-01-15 12:57:23 +01:00
parent ba17dd53f8
commit 9c01712276
9 changed files with 21 additions and 8 deletions

View File

@ -73,6 +73,7 @@ public class SearchFragment extends Fragment {
private SearchView searchView; private SearchView searchView;
private Handler automaticSearchDebouncer; private Handler automaticSearchDebouncer;
private long lastQueryChange = 0; private long lastQueryChange = 0;
/** /**
* Create a new SearchFragment that searches all feeds. * Create a new SearchFragment that searches all feeds.
*/ */

View File

@ -120,7 +120,7 @@ public class FeedItemMenuHandler {
* @return true if selectedItem is not null. * @return true if selectedItem is not null.
*/ */
public static boolean onPrepareMenu(Menu menu, FeedItem selectedItem, int... excludeIds) { public static boolean onPrepareMenu(Menu menu, FeedItem selectedItem, int... excludeIds) {
if (menu == null || selectedItem == null ) { if (menu == null || selectedItem == null) {
return false; return false;
} }
boolean rc = onPrepareMenu(menu, selectedItem); boolean rc = onPrepareMenu(menu, selectedItem);

View File

@ -45,6 +45,9 @@
<module name="FallThrough"/> <module name="FallThrough"/>
<module name="UpperEll"/> <module name="UpperEll"/>
<module name="ModifierOrder"/> <module name="ModifierOrder"/>
<module name="EmptyLineSeparator">
<property name="allowNoEmptyLineBetweenFields" value="true"/>
</module>
<module name="SeparatorWrap"> <module name="SeparatorWrap">
<property name="id" value="SeparatorWrapDot"/> <property name="id" value="SeparatorWrapDot"/>
<property name="tokens" value="DOT"/> <property name="tokens" value="DOT"/>
@ -123,6 +126,7 @@
value="COMMA, SEMI, POST_INC, POST_DEC, DOT, ELLIPSIS, METHOD_REF"/> value="COMMA, SEMI, POST_INC, POST_DEC, DOT, ELLIPSIS, METHOD_REF"/>
<property name="allowLineBreaks" value="true"/> <property name="allowLineBreaks" value="true"/>
</module> </module>
<module name="ParenPad"/>
<module name="AnnotationLocation"> <module name="AnnotationLocation">
<property name="id" value="AnnotationLocationMostCases"/> <property name="id" value="AnnotationLocationMostCases"/>
<property name="tokens" <property name="tokens"
@ -146,6 +150,9 @@
<module name="SingleLineJavadoc"> <module name="SingleLineJavadoc">
<property name="ignoreInlineTags" value="false"/> <property name="ignoreInlineTags" value="false"/>
</module> </module>
<module name="EmptyCatchBlock">
<property name="exceptionVariableName" value="expected"/>
</module>
<module name="NestedIfDepth"> <module name="NestedIfDepth">
<property name="max" value="4"/> <property name="max" value="4"/>
</module> </module>

View File

@ -4,4 +4,9 @@
"http://www.puppycrawl.com/dtds/suppressions_1_1.dtd"> "http://www.puppycrawl.com/dtds/suppressions_1_1.dtd">
<suppressions> <suppressions>
<suppress checks="MethodName" files="core/src/test/java/android/util/Log.java" /> <suppress checks="MethodName" files="core/src/test/java/android/util/Log.java" />
<suppress checks="LineLength" files="strings.xml"/>
<suppress checks="." files="[\\/]generated-sources[\\/]"/>
<suppress checks="." files="[\\/]build[\\/]"/>
<suppress checks="." files="[\\/].idea[\\/]"/>
</suppressions> </suppressions>

View File

@ -115,12 +115,7 @@ public class OpmlBackupAgent extends BackupAgentHelper {
} catch (IOException e) { } catch (IOException e) {
Log.e(TAG, "Error during backup", e); Log.e(TAG, "Error during backup", e);
} finally { } finally {
if (writer != null) { IOUtils.closeQuietly(writer);
try {
writer.close();
} catch (IOException e) {
}
}
} }
} }

View File

@ -11,6 +11,7 @@
* See the License for the specific language governing permissions and * See the License for the specific language governing permissions and
* limitations under the License. * limitations under the License.
*/ */
package de.danoeh.antennapod.core.export; package de.danoeh.antennapod.core.export;
public class CommonSymbols { public class CommonSymbols {

View File

@ -160,6 +160,7 @@ public class HttpDownloader extends Downloader {
try { try {
contentLength = Integer.parseInt(contentLen); contentLength = Integer.parseInt(contentLen);
} catch (NumberFormatException e) { } catch (NumberFormatException e) {
e.printStackTrace();
} }
} }
Log.d(TAG, "content length: " + contentLength); Log.d(TAG, "content length: " + contentLength);

View File

@ -725,7 +725,9 @@ public class LocalPSMP extends PlaybackServiceMediaPlayer {
if (mediaPlayer.isPlaying()) { if (mediaPlayer.isPlaying()) {
mediaPlayer.stop(); mediaPlayer.stop();
} }
} catch (Exception ignore) { } } catch (Exception e) {
e.printStackTrace();
}
mediaPlayer.release(); mediaPlayer.release();
mediaPlayer = null; mediaPlayer = null;
} }

View File

@ -215,6 +215,7 @@ public final class Log {
* Added for this custom Log implementation, not in android sources. * Added for this custom Log implementation, not in android sources.
*/ */
private static final int WTF = 8; private static final int WTF = 8;
static int wtf(int logId, String tag, String msg, Throwable tr, boolean localStack, static int wtf(int logId, String tag, String msg, Throwable tr, boolean localStack,
boolean system) { boolean system) {
return printlns(LOG_ID_MAIN, WTF, tag, msg, tr); return printlns(LOG_ID_MAIN, WTF, tag, msg, tr);