Skip to content

Add logging to extractor #1403

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.

Sign up for GitHub

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

Merged
TobiGr merged 8 commits into from
Apr 3, 2026

More ExtractorLoggerTest tests

30416f4
Select commit
Loading
Failed to load commit list.
Merged

Add logging to extractor #1403

More ExtractorLoggerTest tests
30416f4
Select commit
Loading
Failed to load commit list.
SonarQubeCloud / SonarCloud Code Analysis succeeded Apr 3, 2026 in 47s

Quality Gate passed

Issues
22 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud

Annotations

Check failure on line 208 in extractor/src/main/java/org/schabi/newpipe/extractor/utils/ExtractorLogger.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Add a nested comment explaining why this method is empty, throw an UnsupportedOperationException or complete the implementation. 

See more on https://sonarcloud.io/project/issues?id=TeamNewPipe_NewPipeExtractor&issues=AZ1RU1fZDjkrgjeP1ugW&open=AZ1RU1fZDjkrgjeP1ugW&pullRequest=1403

Check warning on line 19 in extractor/src/main/java/org/schabi/newpipe/extractor/Extractor.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Rename this field "TAG" to match the regular expression '^[a-z][a-zA-Z0-9]*$'. 

See more on https://sonarcloud.io/project/issues?id=TeamNewPipe_NewPipeExtractor&issues=AZ1RU1frDjkrgjeP1ugg&open=AZ1RU1frDjkrgjeP1ugg&pullRequest=1403

Check warning on line 157 in extractor/src/main/java/org/schabi/newpipe/extractor/utils/ExtractorLogger.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Reduce the total number of break and continue statements in this loop to use at most one. 

See more on https://sonarcloud.io/project/issues?id=TeamNewPipe_NewPipeExtractor&issues=AZ1RU1fZDjkrgjeP1ugN&open=AZ1RU1fZDjkrgjeP1ugN&pullRequest=1403

Check failure on line 149 in extractor/src/main/java/org/schabi/newpipe/extractor/utils/ExtractorLogger.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Refactor this method to reduce its Cognitive Complexity from 18 to the 15 allowed. 

See more on https://sonarcloud.io/project/issues?id=TeamNewPipe_NewPipeExtractor&issues=AZ1RU1fZDjkrgjeP1ugO&open=AZ1RU1fZDjkrgjeP1ugO&pullRequest=1403

Check warning on line 106 in extractor/src/test/java/org/schabi/newpipe/extractor/utils/ExtractorLoggerTest.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Update this method so that its implementation is not identical to "unmatchedBraceLeavesRemainder" on line 60. 

See more on https://sonarcloud.io/project/issues?id=TeamNewPipe_NewPipeExtractor&issues=AZ1RU1dXDjkrgjeP1ugM&open=AZ1RU1dXDjkrgjeP1ugM&pullRequest=1403

Check warning on line 232 in extractor/src/main/java/org/schabi/newpipe/extractor/utils/ExtractorLogger.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Replace this use of System.err by a logger. 

See more on https://sonarcloud.io/project/issues?id=TeamNewPipe_NewPipeExtractor&issues=AZ1RU1fZDjkrgjeP1ugd&open=AZ1RU1fZDjkrgjeP1ugd&pullRequest=1403

Check warning on line 186 in extractor/src/main/java/org/schabi/newpipe/extractor/utils/ExtractorLogger.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

This block of commented-out lines of code should be removed. 

See more on https://sonarcloud.io/project/issues?id=TeamNewPipe_NewPipeExtractor&issues=AZ1RU1fZDjkrgjeP1ugS&open=AZ1RU1fZDjkrgjeP1ugS&pullRequest=1403

Check failure on line 207 in extractor/src/main/java/org/schabi/newpipe/extractor/utils/ExtractorLogger.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Add a nested comment explaining why this method is empty, throw an UnsupportedOperationException or complete the implementation. 

See more on https://sonarcloud.io/project/issues?id=TeamNewPipe_NewPipeExtractor&issues=AZ1RU1fZDjkrgjeP1ugV&open=AZ1RU1fZDjkrgjeP1ugV&pullRequest=1403

Check warning on line 228 in extractor/src/main/java/org/schabi/newpipe/extractor/utils/ExtractorLogger.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Replace this use of System.err by a logger. 

See more on https://sonarcloud.io/project/issues?id=TeamNewPipe_NewPipeExtractor&issues=AZ1RU1fZDjkrgjeP1ugc&open=AZ1RU1fZDjkrgjeP1ugc&pullRequest=1403

Check warning on line 237 in extractor/src/main/java/org/schabi/newpipe/extractor/utils/ExtractorLogger.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Replace this use of System.err by a logger. 

See more on https://sonarcloud.io/project/issues?id=TeamNewPipe_NewPipeExtractor&issues=AZ1RU1fZDjkrgjeP1ugf&open=AZ1RU1fZDjkrgjeP1ugf&pullRequest=1403

Check warning on line 215 in extractor/src/main/java/org/schabi/newpipe/extractor/utils/ExtractorLogger.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Replace this use of System.out by a logger. 

See more on https://sonarcloud.io/project/issues?id=TeamNewPipe_NewPipeExtractor&issues=AZ1RU1fZDjkrgjeP1ugZ&open=AZ1RU1fZDjkrgjeP1ugZ&pullRequest=1403

Check warning on line 96 in extractor/src/test/java/org/schabi/newpipe/extractor/utils/ExtractorLoggerTest.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Update this method so that its implementation is not identical to "unmatchedBraceLeavesRemainder" on line 60. 

See more on https://sonarcloud.io/project/issues?id=TeamNewPipe_NewPipeExtractor&issues=AZ1RU1dXDjkrgjeP1ugL&open=AZ1RU1dXDjkrgjeP1ugL&pullRequest=1403

Check failure on line 210 in extractor/src/main/java/org/schabi/newpipe/extractor/utils/ExtractorLogger.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Add a nested comment explaining why this method is empty, throw an UnsupportedOperationException or complete the implementation. 

See more on https://sonarcloud.io/project/issues?id=TeamNewPipe_NewPipeExtractor&issues=AZ1RU1fZDjkrgjeP1ugY&open=AZ1RU1fZDjkrgjeP1ugY&pullRequest=1403

Check warning on line 160 in extractor/src/main/java/org/schabi/newpipe/extractor/utils/ExtractorLogger.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

This block of commented-out lines of code should be removed. 

See more on https://sonarcloud.io/project/issues?id=TeamNewPipe_NewPipeExtractor&issues=AZ1RU1fZDjkrgjeP1ugP&open=AZ1RU1fZDjkrgjeP1ugP&pullRequest=1403

Check warning on line 177 in extractor/src/main/java/org/schabi/newpipe/extractor/utils/ExtractorLogger.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

This block of commented-out lines of code should be removed. 

See more on https://sonarcloud.io/project/issues?id=TeamNewPipe_NewPipeExtractor&issues=AZ1RU1fZDjkrgjeP1ugR&open=AZ1RU1fZDjkrgjeP1ugR&pullRequest=1403

Check failure on line 206 in extractor/src/main/java/org/schabi/newpipe/extractor/utils/ExtractorLogger.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Add a nested comment explaining why this method is empty, throw an UnsupportedOperationException or complete the implementation. 

See more on https://sonarcloud.io/project/issues?id=TeamNewPipe_NewPipeExtractor&issues=AZ1RU1fZDjkrgjeP1ugU&open=AZ1RU1fZDjkrgjeP1ugU&pullRequest=1403

Check failure on line 209 in extractor/src/main/java/org/schabi/newpipe/extractor/utils/ExtractorLogger.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Add a nested comment explaining why this method is empty, throw an UnsupportedOperationException or complete the implementation. 

See more on https://sonarcloud.io/project/issues?id=TeamNewPipe_NewPipeExtractor&issues=AZ1RU1fZDjkrgjeP1ugX&open=AZ1RU1fZDjkrgjeP1ugX&pullRequest=1403

Check failure on line 205 in extractor/src/main/java/org/schabi/newpipe/extractor/utils/ExtractorLogger.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Add a nested comment explaining why this method is empty, throw an UnsupportedOperationException or complete the implementation. 

See more on https://sonarcloud.io/project/issues?id=TeamNewPipe_NewPipeExtractor&issues=AZ1RU1fZDjkrgjeP1ugT&open=AZ1RU1fZDjkrgjeP1ugT&pullRequest=1403

Check warning on line 223 in extractor/src/main/java/org/schabi/newpipe/extractor/utils/ExtractorLogger.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Replace this use of System.out by a logger. 

See more on https://sonarcloud.io/project/issues?id=TeamNewPipe_NewPipeExtractor&issues=AZ1RU1fZDjkrgjeP1ugb&open=AZ1RU1fZDjkrgjeP1ugb&pullRequest=1403

Check warning on line 220 in extractor/src/main/java/org/schabi/newpipe/extractor/utils/ExtractorLogger.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Replace this use of System.err by a logger. 

See more on https://sonarcloud.io/project/issues?id=TeamNewPipe_NewPipeExtractor&issues=AZ1RU1fZDjkrgjeP1uga&open=AZ1RU1fZDjkrgjeP1uga&pullRequest=1403

Check warning on line 236 in extractor/src/main/java/org/schabi/newpipe/extractor/utils/ExtractorLogger.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Replace this use of System.err by a logger. 

See more on https://sonarcloud.io/project/issues?id=TeamNewPipe_NewPipeExtractor&issues=AZ1RU1fZDjkrgjeP1uge&open=AZ1RU1fZDjkrgjeP1uge&pullRequest=1403

Check warning on line 168 in extractor/src/main/java/org/schabi/newpipe/extractor/utils/ExtractorLogger.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

This block of commented-out lines of code should be removed. 

See more on https://sonarcloud.io/project/issues?id=TeamNewPipe_NewPipeExtractor&issues=AZ1RU1fZDjkrgjeP1ugQ&open=AZ1RU1fZDjkrgjeP1ugQ&pullRequest=1403
View more details on SonarQubeCloud