Skip to content

Fix: minor analyzer warnings#590

Merged
dnzbk merged 2 commits intonzbgetcom:developfrom
Coeur:coeur/analyzer
Jun 14, 2025
Merged

Fix: minor analyzer warnings#590
dnzbk merged 2 commits intonzbgetcom:developfrom
Coeur:coeur/analyzer

Conversation

@Coeur
Copy link
Copy Markdown
Contributor

@Coeur Coeur commented Jun 13, 2025

Description

Fix minor analyzer warnings:

  • Extra ';' outside of a function is incompatible with C++98
  • 'break' will never be executed
  • foundUints is never used
  • Not a Doxygen trailing comment
  • Zero as null pointer constant
  • minutes is garbage value
  • dereference of null nzbInfo pointer
  • addbuf is garbage value
  • difSizeCount is never used
  • dereference of null m_postInfo pointer
  • origName is garbage value
  • fileList can be empty even after empty() check
  • sort can be null

@dnzbk dnzbk self-requested a review June 14, 2025 07:33
Copy link
Copy Markdown
Collaborator

@dnzbk dnzbk left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you

@dnzbk dnzbk merged commit 93cb448 into nzbgetcom:develop Jun 14, 2025
9 checks passed
@Coeur Coeur deleted the coeur/analyzer branch June 14, 2025 18:27
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.

2 participants