[mythtv-commits] [MythTV/mythtv] 9a0e0d: tidy: Modernize a couple of "display" loops.

David Hampton noreply at github.com
Thu Nov 4 12:02:04 UTC 2021


  Branch: refs/heads/master
  Home:   https://github.com/MythTV/mythtv
  Commit: 9a0e0dd75836b728d8daebe192a59b78df9e244f
      https://github.com/MythTV/mythtv/commit/9a0e0dd75836b728d8daebe192a59b78df9e244f
  Author: David Hampton <mythtv at love2code.net>
  Date:   2021-11-03 (Wed, 03 Nov 2021)

  Changed paths:
    M mythtv/libs/libmythui/platforms/mythdisplaydrm.cpp
    M mythtv/libs/libmythui/platforms/mythdisplaymutter.cpp
    M mythtv/libs/libmythui/platforms/mythdisplayx11.cpp

  Log Message:
  -----------
  tidy: Modernize a couple of "display" loops.

Convert these loops to the "new" c++11 range-based loops.

https://clang.llvm.org/extra/clang-tidy/checks/modernize-loop-convert.html


  Commit: c5755a3f860fdb8778d70ab5f0ce26ae803ef2a1
      https://github.com/MythTV/mythtv/commit/c5755a3f860fdb8778d70ab5f0ce26ae803ef2a1
  Author: David Hampton <mythtv at love2code.net>
  Date:   2021-11-03 (Wed, 03 Nov 2021)

  Changed paths:
    M mythtv/libs/libmythui/mythuitype.cpp

  Log Message:
  -----------
  tidy: Modernize a MythUI loops.

Convert this loop to the "new" c++11 range-based loop.

https://clang.llvm.org/extra/clang-tidy/checks/modernize-loop-convert.html


  Commit: 17826ec4ebb070065e60ed0f690a27e11a04b92c
      https://github.com/MythTV/mythtv/commit/17826ec4ebb070065e60ed0f690a27e11a04b92c
  Author: David Hampton <mythtv at love2code.net>
  Date:   2021-11-03 (Wed, 03 Nov 2021)

  Changed paths:
    M mythtv/libs/libmythtv/io/mythfifowriter.cpp
    M mythtv/libs/libmythupnp/httprequest.cpp

  Log Message:
  -----------
  tidy: Fix two "braces around statements" warnings.

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


  Commit: 2a97d4c10354c5534d8ba11a92797344d74a0e71
      https://github.com/MythTV/mythtv/commit/2a97d4c10354c5534d8ba11a92797344d74a0e71
  Author: David Hampton <mythtv at love2code.net>
  Date:   2021-11-03 (Wed, 03 Nov 2021)

  Changed paths:
    M mythtv/programs/mythfrontend/networkcontrol.cpp

  Log Message:
  -----------
  tidy: Fix "unhandled exception at new" warnings in networkcontrol.cpp.

According to the C++ standard, calls to new can throw an exception
upon failure or return a null pointer.  Specifically indicate that
this call may not throw but must return a null pointer, and handle the
failure case. The only result of an allocation failure here is that a
courtesy "shutting down" message isn't sent to clients.

https://clang.llvm.org/extra/clang-tidy/checks/bugprone-unhandled-exception-at-new.html


  Commit: 5f45cf8c82b25a3a62c4c39b04ee294ea03957ee
      https://github.com/MythTV/mythtv/commit/5f45cf8c82b25a3a62c4c39b04ee294ea03957ee
  Author: David Hampton <mythtv at love2code.net>
  Date:   2021-11-03 (Wed, 03 Nov 2021)

  Changed paths:
    M mythtv/programs/mythfrontend/videolist.cpp

  Log Message:
  -----------
  tidy: Fix "readability-redundant-smartptr-get" warning in videolist.cpp.

Remove the redundant call.

https://clang.llvm.org/extra/clang-tidy/checks/readability-redundant-smartptr-get.html


  Commit: a67d5b4644ccde1932decb4439e1d369c00d557d
      https://github.com/MythTV/mythtv/commit/a67d5b4644ccde1932decb4439e1d369c00d557d
  Author: David Hampton <mythtv at love2code.net>
  Date:   2021-11-03 (Wed, 03 Nov 2021)

  Changed paths:
    M mythtv/libs/libmythbase/test/test_template/test_template.cpp

  Log Message:
  -----------
  Quiet a "variable set but not used" warning.


  Commit: 1c38021549653c179d743ed2782dd2f9eceeb1f8
      https://github.com/MythTV/mythtv/commit/1c38021549653c179d743ed2782dd2f9eceeb1f8
  Author: David Hampton <mythtv at love2code.net>
  Date:   2021-11-03 (Wed, 03 Nov 2021)

  Changed paths:
    M .clang-tidy

  Log Message:
  -----------
  Fix missing comma in clang-tidy config file.


  Commit: 4ce98b40aa73f942cffc1bce01ee1b63aeb95858
      https://github.com/MythTV/mythtv/commit/4ce98b40aa73f942cffc1bce01ee1b63aeb95858
  Author: David Hampton <mythtv at love2code.net>
  Date:   2021-11-03 (Wed, 03 Nov 2021)

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

  Log Message:
  -----------
  tidy: Fix a few "braces around statements" warnings in the scheduler.

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


Compare: https://github.com/MythTV/mythtv/compare/c5819a0682f3...4ce98b40aa73


More information about the mythtv-commits mailing list