[mythtv] #define MAX
Daniel Kristjansson
danielk at cuymedia.net
Mon Dec 12 08:19:07 EST 2005
On Mon, 2005-12-12 at 20:03 +1000, Paul Andreassen wrote:
> Hi,
>
> While writing a patch, I needed a max function and so wrote a simple marco as
> any c programmer does. Thinking about it further I realised that not only
> does MythTv use c++ but also QT, so surely there must already be a max
> function. Sure enough QT has one called QMAX defined in qglobal.h. Better
> yet c++ has one called max as a template in <limits>
>
> By greping MythTv source there are multiple definations. The worst is the
> #define max(a,b) ((a < b) ? b : a) which has to be fixed.
The ones in libmyth[soundtouch|samplerate] and libav[codec|util] need
to be left alone, we sync these with external libraries so we don't
want to have any changes that aren't needed in there.
> What is the correct max function to use?
I use std::max and std::min because they are type correct template
functions rather than macros.
-- Daniel
More information about the mythtv-dev
mailing list