[mythtv-commits] [MythTV/mythtv] 93d99c: freesurround/el_processor.h: convert tabs to space...

[Scott Theisen]

  Branch: refs/heads/master
  Home:   https://github.com/MythTV/mythtv
  Commit: 93d99c021639abe1bbc53b26b2020a0e195f782a
      https://github.com/MythTV/mythtv/commit/93d99c021639abe1bbc53b26b2020a0e195f782a
  Author: ulmus-scott <scott.the.elm at gmail.com>
  Date:   2022-02-20 (Sun, 20 Feb 2022)

  Changed paths:
    M mythtv/libs/libmythfreesurround/el_processor.h

  Log Message:
  -----------
  freesurround/el_processor.h: convert tabs to spaces, fix ° mangling from 3d1e2e5eef6fddde3701cd1c9af27acf08752ff6


  Commit: 364a1e32a86cf9032635e16488ce6daf53f74f80
      https://github.com/MythTV/mythtv/commit/364a1e32a86cf9032635e16488ce6daf53f74f80
  Author: ulmus-scott <scott.the.elm at gmail.com>
  Date:   2022-02-20 (Sun, 20 Feb 2022)

  Changed paths:
    M mythtv/libs/libmythfreesurround/el_processor.cpp

  Log Message:
  -----------
  freesurround/el_processor.cpp: enhance readability of windowing

use array access instead of iterators and combine loops


  Commit: ff373513b901cefad56ae842ee42fb5507c539a6
      https://github.com/MythTV/mythtv/commit/ff373513b901cefad56ae842ee42fb5507c539a6
  Author: ulmus-scott <scott.the.elm at gmail.com>
  Date:   2022-02-20 (Sun, 20 Feb 2022)

  Changed paths:
    M mythtv/libs/libmythfreesurround/el_processor.cpp

  Log Message:
  -----------
  freesurround/el_processor.cpp: remove internal FFmpeg header

m_dft[RL] seem to be twice as big as necessary, but I kept the size for no functional change.

Strict aliasing fixes.

av_fft_init with 13 is only correct since the block size is always 8192.

FFmpeg free()s the context in av_fft_end


  Commit: 1b00c58f3bb63f03a222d9a0abf2c1fb0a22074e
      https://github.com/MythTV/mythtv/commit/1b00c58f3bb63f03a222d9a0abf2c1fb0a22074e
  Author: ulmus-scott <scott.the.elm at gmail.com>
  Date:   2022-02-20 (Sun, 20 Feb 2022)

  Changed paths:
    M mythtv/libs/libmythfreesurround/el_processor.cpp

  Log Message:
  -----------
  freesurround/el_processor.cpp: strict aliasing fixes

also improve readability


  Commit: b156dc7b490821db60ae92d970d8c5a09c1e4aa7
      https://github.com/MythTV/mythtv/commit/b156dc7b490821db60ae92d970d8c5a09c1e4aa7
  Author: ulmus-scott <scott.the.elm at gmail.com>
  Date:   2022-02-20 (Sun, 20 Feb 2022)

  Changed paths:
    M mythtv/libs/libmythfreesurround/el_processor.cpp

  Log Message:
  -----------
  freesurround/el_processor.cpp: use std::min, max, clamp


  Commit: e779cc2555988a6619a4e15870003e716879160e
      https://github.com/MythTV/mythtv/commit/e779cc2555988a6619a4e15870003e716879160e
  Author: ulmus-scott <scott.the.elm at gmail.com>
  Date:   2022-02-20 (Sun, 20 Feb 2022)

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

  Log Message:
  -----------
  freesurround/el_processor: PIMPL with nested private class


  Commit: 44888375826cd67fc7e4d66a065d19be1a82c4e2
      https://github.com/MythTV/mythtv/commit/44888375826cd67fc7e4d66a065d19be1a82c4e2
  Author: ulmus-scott <scott.the.elm at gmail.com>
  Date:   2022-02-20 (Sun, 20 Feb 2022)

  Changed paths:
    M mythtv/libs/libmythfreesurround/el_processor.cpp

  Log Message:
  -----------
  freesurround/el_processor.cpp: use std::hypot for better accuracy


  Commit: a3d5d513138927a4e051756a893f2944278c5331
      https://github.com/MythTV/mythtv/commit/a3d5d513138927a4e051756a893f2944278c5331
  Author: ulmus-scott <scott.the.elm at gmail.com>
  Date:   2022-02-20 (Sun, 20 Feb 2022)

  Changed paths:
    M mythtv/libs/libmythfreesurround/el_processor.cpp

  Log Message:
  -----------
  freesurround/el_processor.cpp: add clarifying comments


  Commit: f4812fa80385a33cb6e8abfd2d1a484e3b0195b5
      https://github.com/MythTV/mythtv/commit/f4812fa80385a33cb6e8abfd2d1a484e3b0195b5
  Author: ulmus-scott <scott.the.elm at gmail.com>
  Date:   2022-02-20 (Sun, 20 Feb 2022)

  Changed paths:
    M mythtv/libs/libmythfreesurround/el_processor.cpp

  Log Message:
  -----------
  freesurround/el_processor.cpp: rename Impl::clamp to Impl::clamp_unit_mag

This more clearly expresses what the function does and avoids confusion with std::clamp.


Compare: https://github.com/MythTV/mythtv/compare/cfdad2d3b402...f4812fa80385