Opened 9 years ago
Closed 9 years ago
Last modified 8 years ago
#12359 closed Patch - Feature (fixed)
C++11 feature patch - external/libmythsoundtouch/cpu_detect_x86_gcc.cpp
Reported by: | Owned by: | JYA | |
---|---|---|---|
Priority: | trivial | Milestone: | 0.28 |
Component: | MythTV - Audio Output | Version: | Master Head |
Severity: | low | Keywords: | |
Cc: | Ticket locked: | no |
Description
The C++11 standard requires a space between literal and identifier in order to support user defined literal types via suffixes.
This patch provides the needed spaces around the use of the defines.
As C++11 is not currently required by MythTV, this should be considered a really low priority in the code cleanliness queue.
Note: A better fix would be to update to the latest soundtouch library, but that requires much more analysis and testing time to insure that the improvements to soundtouch in current versions (esp. multi-channel) are compatible with the existing MythTV patches for multi-channel. This is a minimal update for C++11 compliance.
github ref: https://github.com/garybuhrmaster/mythtv/commit/8c92ae1ba905696f4ab4bf82296e8969fbcd0756
github git-am ref: https://github.com/garybuhrmaster/mythtv/commit/8c92ae1ba905696f4ab4bf82296e8969fbcd0756.patch
In eb2f96e9d5e8a9707476809346f3312348645d19/mythtv: