Skip to content

fix: remove malformed javadoc to fix -Werror build failure (#7393)#7394

Merged
DenizAltunkapan merged 4 commits intoTheAlgorithms:masterfrom
Senrian:fix/javadoc-warnings-7393
Apr 22, 2026
Merged

fix: remove malformed javadoc to fix -Werror build failure (#7393)#7394
DenizAltunkapan merged 4 commits intoTheAlgorithms:masterfrom
Senrian:fix/javadoc-warnings-7393

Conversation

@Senrian
Copy link
Copy Markdown
Contributor

@Senrian Senrian commented Apr 22, 2026

Removes malformed javadoc that causes build failures with -Werror.

Changes

  • AnyBaseToAnyBase.java: Fix * * @author@author in class javadoc
  • ReverseString.java: Fix * * @param@param in method javadoc

Fixes #7393

penggaolai and others added 3 commits April 4, 2026 04:15
When searching for a non-null key in an array that contains null elements,
the sentinel linear search would throw a NullPointerException because it
called array[i].compareTo(key) without checking if array[i] is null.

Added null check for array[i] in the while loop condition to prevent NPE
and return the correct index when array elements themselves are null.

Issue: TheAlgorithms#7318 (related)
@codecov-commenter
Copy link
Copy Markdown

codecov-commenter commented Apr 22, 2026

Codecov Report

❌ Patch coverage is 0% with 1 line in your changes missing coverage. Please review.
✅ Project coverage is 79.54%. Comparing base (b3e31b5) to head (508f840).
⚠️ Report is 1 commits behind head on master.

Files with missing lines Patch % Lines
...m/thealgorithms/searches/SentinelLinearSearch.java 0.00% 0 Missing and 1 partial ⚠️
Additional details and impacted files
@@            Coverage Diff            @@
##             master    #7394   +/-   ##
=========================================
  Coverage     79.53%   79.54%           
- Complexity     7176     7178    +2     
=========================================
  Files           798      798           
  Lines         23467    23467           
  Branches       4617     4617           
=========================================
+ Hits          18665    18666    +1     
  Misses         4055     4055           
+ Partials        747      746    -1     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@DenizAltunkapan DenizAltunkapan enabled auto-merge (squash) April 22, 2026 09:23
@DenizAltunkapan DenizAltunkapan merged commit 0ad5d90 into TheAlgorithms:master Apr 22, 2026
7 checks passed
@Senrian Senrian deleted the fix/javadoc-warnings-7393 branch April 22, 2026 10:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Build fails because existing documentation warnings are treated as errors by -Werror

4 participants