"Ronald S. Bultje" <[email protected]> writes: >>> and av_clip_uint8_fast() which on arm is av_clip_uint8() and on x86 >>> is cm[]? >> >> _fast suffixes are stupid. In this case, what you really mean is >> restricted input range. > > Well function name isn't that important to me. Anyone up for > implementing this?
Function names should indicate what the functions do. If they only differ in a _fast suffix, there is no apparent reason to ever use the non-fast one. In this case we'd be dealing with one function operating on the full integer range and another operating on a reduced range (and thus might have a speed advantage). This should, if possible, be reflected in the names and clearly documented. -- Måns Rullgård [email protected] _______________________________________________ libav-devel mailing list [email protected] https://lists.libav.org/mailman/listinfo/libav-devel
