Re: [LTP] [PATCH] New core test of cgroups and extended attributes

[chrubis@suse.cz]

Hi!
> > I do not want to add custom printf() format strings on purpose, as that
> > would:
> >
> > 1) create confusion
> >
> > 2) possibly break the checks on the format done by compiler
> >
> > Anybody else has better idea?
> Since we're going to print buf in hex, may be we can use
> tst_resm_hex(flags, buf, buf_len, fmt, ...) to make it more descriptive?

Naming things right is hard. I agree that 'hex' should be part of the
name but ideally something like 'dump' or 'buf' should be there as well
at least in form of the first letter (to keep the name reasonably short).

tst_resm_bhex() or tst_resm_dhex() ?

-- 
Cyril Hrubis
chrubis@suse.cz

------------------------------------------------------------------------------
AlienVault Unified Security Management (USM) platform delivers complete
security visibility with the essential security capabilities. Easily and
efficiently configure, manage, and operate all of your security controls
from a single console and one unified framework. Download a free trial.
http://p.sf.net/sfu/alienvault_d2d
_______________________________________________
Ltp-list mailing list
Ltp-list@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/ltp-list