[mythtv-commits] [MythTV/mythtv] 618505: tidy: Clean up new tidy warnings in v2 services.

[David Hampton]

  Branch: refs/heads/master
  Home:   https://github.com/MythTV/mythtv
  Commit: 618505155e994aacee52cf9a72273ef5a40dc0ed
      https://github.com/MythTV/mythtv/commit/618505155e994aacee52cf9a72273ef5a40dc0ed
  Author: David Hampton <mythtv at love2code.net>
  Date:   2024-11-02 (Sat, 02 Nov 2024)

  Changed paths:
    M mythtv/programs/mythbackend/servicesv2/v2dvr.cpp
    M mythtv/programs/mythbackend/servicesv2/v2playGroup.h

  Log Message:
  -----------
  tidy: Clean up new tidy warnings in v2 services.

1) Add a set of braces
2) Replace "if (a) return true; return false;" with "return a;".
3) Remove redundant variable initialization.

https://clang.llvm.org/extra/clang-tidy/checks/modernize/use-default-member-init.html
https://clang.llvm.org/extra/clang-tidy/checks/readability/braces-around-statements.html
https://clang.llvm.org/extra/clang-tidy/checks/readability/simplify-boolean-expr.html


  Commit: 9128e318eb110328e26a30443e1d50e5188bfc16
      https://github.com/MythTV/mythtv/commit/9128e318eb110328e26a30443e1d50e5188bfc16
  Author: David Hampton <mythtv at love2code.net>
  Date:   2024-11-02 (Sat, 02 Nov 2024)

  Changed paths:
    M mythtv/programs/mythbackend/CMakeLists.txt

  Log Message:
  -----------
  Fix cmake compilation.

Need to add a new header files to the list of files to be processed
for QObjects.


Compare: https://github.com/MythTV/mythtv/compare/598454424d60...9128e318eb11

To unsubscribe from these emails, change your notification settings at https://github.com/MythTV/mythtv/settings/notifications