[mythtv-commits] [MythTV/mythtv] 39f0e3: libmythfreesurround: include cleanup

[David Hampton]

  Branch: refs/heads/master
  Home:   https://github.com/MythTV/mythtv
  Commit: 39f0e39831cf568e031ea11ec22f21c5d32fb5fc
      https://github.com/MythTV/mythtv/commit/39f0e39831cf568e031ea11ec22f21c5d32fb5fc
  Author: Scott Theisen <scott.the.elm at gmail.com>
  Date:   2024-10-07 (Mon, 07 Oct 2024)

  Changed paths:
    M mythtv/libs/libmythfreesurround/freesurround.cpp
    M mythtv/libs/libmythfreesurround/freesurround.h

  Log Message:
  -----------
  libmythfreesurround: include cleanup

QString is only for the logging, which is the only
use of other MythTV code.


  Commit: 7ea0c01f137dbb49fc5476d3436d1a7312d7bd65
      https://github.com/MythTV/mythtv/commit/7ea0c01f137dbb49fc5476d3436d1a7312d7bd65
  Author: Scott Theisen <scott.the.elm at gmail.com>
  Date:   2024-10-07 (Mon, 07 Oct 2024)

  Changed paths:
    M mythtv/.cppcheck-suppress
    M mythtv/CMakeLists.txt
    M mythtv/docs/doxygen-architecture-document.cpp
    M mythtv/libs/CMakeLists.txt
    M mythtv/libs/libmyth/CMakeLists.txt
    M mythtv/libs/libmyth/audio/audiooutputbase.cpp
    A mythtv/libs/libmyth/audio/freesurround.cpp
    A mythtv/libs/libmyth/audio/freesurround.h
    A mythtv/libs/libmyth/audio/freesurround_decoder.cpp
    A mythtv/libs/libmyth/audio/freesurround_decoder.h
    M mythtv/libs/libmyth/libmyth.pro
    R mythtv/libs/libmythfreesurround/CMakeLists.txt
    R mythtv/libs/libmythfreesurround/el_processor.cpp
    R mythtv/libs/libmythfreesurround/el_processor.h
    R mythtv/libs/libmythfreesurround/freesurround.cpp
    R mythtv/libs/libmythfreesurround/freesurround.h
    R mythtv/libs/libmythfreesurround/libmythfreesurround.pro
    M mythtv/libs/libmythmetadata/CMakeLists.txt
    M mythtv/libs/libmythmetadata/libmythmetadata.pro
    M mythtv/libs/libs.pro

  Log Message:
  -----------
  merge libmythfreesurround into libmyth/audio

which was its only user.

If it didn't use mythlogging.h, and thus QString, it would be
entirely standard C++, not depending on Qt or any of our libraries,
only depending on FFmpeg.


  Commit: bfe31f353e38e286b27fb57ca5c4272578c4ad1b
      https://github.com/MythTV/mythtv/commit/bfe31f353e38e286b27fb57ca5c4272578c4ad1b
  Author: Scott Theisen <scott.the.elm at gmail.com>
  Date:   2024-10-07 (Mon, 07 Oct 2024)

  Changed paths:
    M mythtv/docs/doxygen-architecture-document.cpp

  Log Message:
  -----------
  doxygen-architecture-document.cpp: remove obsolete reference to libmythnvctrl

which was moved to external/libXNVCtrl in cdc57f4c504cba9e5063af24b3d38c311577b41f in 2017
and removed entirely in bd6c1ef45976809a9e5fb84fc8b00573b7fcc039 in 2019.


  Commit: 1bf48b5f3fe12052efd391b1ef6975784367ac19
      https://github.com/MythTV/mythtv/commit/1bf48b5f3fe12052efd391b1ef6975784367ac19
  Author: Scott Theisen <scott.the.elm at gmail.com>
  Date:   2024-10-07 (Mon, 07 Oct 2024)

  Changed paths:
    M mythtv/libs/libmyth/audio/freesurround.cpp
    M mythtv/libs/libmyth/audio/freesurround.h
    M mythtv/libs/libmyth/audio/freesurround_decoder.cpp

  Log Message:
  -----------
  libmyth/audio/freesurround*: silence -Wshadow

freesurround_decoder.cpp: This was identical to a code block before the if.

freesurround.cpp:

The fsurround_params constructor is never called explicitly, so just set
the default values in the struct definition.  Those values are immediately
overwritten in the FreeSurround constructor anyways.

In FreeSurround::putFrames(), copy the shadowed variables into the places
they are used.


  Commit: 5c887163422cf782169a8f59918870dd7e7e21e6
      https://github.com/MythTV/mythtv/commit/5c887163422cf782169a8f59918870dd7e7e21e6
  Author: David Hampton <mythtv at love2code.net>
  Date:   2024-12-30 (Mon, 30 Dec 2024)

  Changed paths:
    M mythtv/.cppcheck-suppress
    M mythtv/CMakeLists.txt
    M mythtv/docs/doxygen-architecture-document.cpp
    M mythtv/libs/CMakeLists.txt
    M mythtv/libs/libmyth/CMakeLists.txt
    M mythtv/libs/libmyth/audio/audiooutputbase.cpp
    A mythtv/libs/libmyth/audio/freesurround.cpp
    A mythtv/libs/libmyth/audio/freesurround.h
    A mythtv/libs/libmyth/audio/freesurround_decoder.cpp
    A mythtv/libs/libmyth/audio/freesurround_decoder.h
    M mythtv/libs/libmyth/libmyth.pro
    R mythtv/libs/libmythfreesurround/CMakeLists.txt
    R mythtv/libs/libmythfreesurround/el_processor.cpp
    R mythtv/libs/libmythfreesurround/el_processor.h
    R mythtv/libs/libmythfreesurround/freesurround.cpp
    R mythtv/libs/libmythfreesurround/freesurround.h
    R mythtv/libs/libmythfreesurround/libmythfreesurround.pro
    M mythtv/libs/libmythmetadata/CMakeLists.txt
    M mythtv/libs/libmythmetadata/libmythmetadata.pro
    M mythtv/libs/libs.pro

  Log Message:
  -----------
  Merge remote-tracking branch 'ulmus-scott/freesurround'


Compare: https://github.com/MythTV/mythtv/compare/db3c62340875...5c887163422c

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