On Thu, Mar 31, 2016 at 12:34:16AM -0400, Jeff Mahoney wrote: > -extern const char *xlookup(const struct xlat *, const unsigned int); > -extern const char *xlat_search(const struct xlat *, const size_t, const > unsigned int); > +extern const char *xlookup(const struct xlat *, const uint64_t); > +extern const char *xlat_search(const struct xlat *, const size_t, const > uint64_t);
xlookup has several users that are not prepared for this change yet so it also needs an xlookup64 variant. xlat_search has only one user that passes unsigned int, so it's OK. > -extern void printxvals(const unsigned int, const char *, const struct xlat > *, ...); > +extern void printxvals(const uint64_t, const char *, const struct xlat *, > ...); printxvals is OK, all its users are ready for the change... > #define printxval(xlat, val, dflt) printxvals(val, dflt, xlat, NULL) ... except the main user, printxval, which is not. Looks like we need printxval and printxval64 wrappers. > -extern void addflags(const struct xlat *, int); > -extern int printflags(const struct xlat *, int, const char *); > -extern const char *sprintflags(const char *, const struct xlat *, int); > +extern void addflags(const struct xlat *, unsigned int); > +extern void addflags64(const struct xlat *, uint64_t); > +extern int printflags(const struct xlat *, unsigned int, const char *); > +extern int printflags64(const struct xlat *, uint64_t, const char *); > +extern const char *sprintflags(const char *, const struct xlat *, unsigned > int); > +extern const char *sprintflags64(const char *, const struct xlat *, > uint64_t); All users of addflags are ready for the change so addflags64 is not needed. All these new wrappers could be implemented as static inline functions right in defs.h so that compiler would do the right thing without extra costs. -- ldv
pgplqnVMGkKj7.pgp
Description: PGP signature
------------------------------------------------------------------------------ Transform Data into Opportunity. Accelerate data analysis in your applications with Intel Data Analytics Acceleration Library. Click to learn more. http://pubads.g.doubleclick.net/gampad/clk?id=278785471&iu=/4140
_______________________________________________ Strace-devel mailing list Strace-devel@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/strace-devel