[mythtv-commits] [MythTV/mythtv] bff41d: clazy: Use qstring multi-arg for consecutive strin...

[David Hampton]

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

  Changed paths:
    M mythtv/programs/mythbackend/scheduler.cpp

  Log Message:
  -----------
  clazy: Use qstring multi-arg for consecutive string arguments.

This is more efficient as it only performs a single memory allocation
instead of four memory allocations.

https://github.com/KDE/clazy/blob/master/docs/checks/README-qstring-arg.md


  Commit: 6b097d066d9474b0a2eb976d0a473389571213ed
      https://github.com/MythTV/mythtv/commit/6b097d066d9474b0a2eb976d0a473389571213ed
  Author: David Hampton <mythtv at love2code.net>
  Date:   2024-11-21 (Thu, 21 Nov 2024)

  Changed paths:
    M mythtv/programs/mythbackend/servicesv2/v2powerPriority.h

  Log Message:
  -----------
  tidy: Don't explicitly re-declare default initializers.

If an field is initialized to the default value for a variable type,
you should use the default initializer instead of explicitly declaring
it.  Removing all default initializers makes it obvious where there
are non-default values.  This also reduces duplicate code when there
are multiple constructors for an object, as the variable's default
value is specified only once.

https://clang.llvm.org/extra/clang-tidy/checks/modernize/use-default-member-init.html


  Commit: 693c591631b94ba93dc5b9ede8ff2b3107450fd6
      https://github.com/MythTV/mythtv/commit/693c591631b94ba93dc5b9ede8ff2b3107450fd6
  Author: David Hampton <mythtv at love2code.net>
  Date:   2024-11-21 (Thu, 21 Nov 2024)

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

  Log Message:
  -----------
  tidy: Add braces around else clause.

For consistency and ease of readability, once an if/then/else clause
has started using braces around the bodies of an "if" clause, it
should use braces around all the rest of the bodies.

https://clang.llvm.org/extra/clang-tidy/checks/readability/braces-around-statements.html


  Commit: 42ab7a84edd9f6e9e189ace07a95f5c560200c6f
      https://github.com/MythTV/mythtv/commit/42ab7a84edd9f6e9e189ace07a95f5c560200c6f
  Author: David Hampton <mythtv at love2code.net>
  Date:   2024-11-21 (Thu, 21 Nov 2024)

  Changed paths:
    M mythtv/libs/libmythtv/channelscan/hdhrchannelfetcher.cpp

  Log Message:
  -----------
  tidy: Fix unitialized variable warning.

https://clang.llvm.org/extra/clang-tidy/checks/cppcoreguidelines/init-variables.html


  Commit: 447ea618b2950f748bc6b353fbf2cfdc7a08a51c
      https://github.com/MythTV/mythtv/commit/447ea618b2950f748bc6b353fbf2cfdc7a08a51c
  Author: David Hampton <mythtv at love2code.net>
  Date:   2024-11-21 (Thu, 21 Nov 2024)

  Changed paths:
    M mythtv/libs/libmyth/audio/audiooutputjack.cpp

  Log Message:
  -----------
  tidy: A couple of minor edits to jack audio.

1) Add braces around a couple of else clauses.

2) Use std::max when calculating max latency.

https://clang.llvm.org/extra/clang-tidy/checks/readability/braces-around-statements.html
https://clang.llvm.org/extra/clang-tidy/checks/readability/use-std-min-max.html


Compare: https://github.com/MythTV/mythtv/compare/1b7ecb130436...447ea618b295

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