Hi, On Sun, Nov 27, 2011 at 2:54 PM, Luca Barbato <[email protected]> wrote: > On 27/11/11 23:30, Ronald S. Bultje wrote: >> >> Hi, >> >> On Sun, Nov 27, 2011 at 2:24 PM, Luca Barbato<[email protected]> wrote: >>> >>> On 27/11/11 22:53, Ronald S. Bultje wrote: >>>> >>>> On Sun, Nov 27, 2011 at 1:42 PM, Luca Barbato<[email protected]> >>>> wrote: >>>>> >>>>> Simplify a little error resilience calls >>>>> --- >>>>> libavcodec/error_resilience.c | 26 +++++++++++++------------- >>>>> libavcodec/h263dec.c | 14 +++++++------- >>>>> libavcodec/h264.c | 16 ++++++++-------- >>>>> libavcodec/intrax8.c | 2 +- >>>>> libavcodec/mpegvideo.h | 4 ++++ >>>>> libavcodec/rv10.c | 2 +- >>>>> libavcodec/rv34.c | 4 ++-- >>>>> libavcodec/vc1dec.c | 18 +++++++++--------- >>>>> 8 files changed, 45 insertions(+), 41 deletions(-) >>>> >>>> I don't quite like the names "ER_END", how about ER_BLK_END or >>>> ER_BLOCK_END or ER_MB/MACROBLOCK_END or so? Same for error. >>> >>> MB_END and MB_ERROR Could be good? >> >> I like ER_ prefixes. Non-prefixed stuff for things shared all over >> libavcodec/ is a recipe for disaster if you ask me. ER_MB_END/ERROR >> sounds good. > > Shall I namespace the other macros as well while I'm at it?
If you want, yes please. Ronald _______________________________________________ libav-devel mailing list [email protected] https://lists.libav.org/mailman/listinfo/libav-devel
