On 23/10/16 00:32, Stefan Beller wrote: > From: Junio C Hamano <gitster@xxxxxxxxx> > > Export attr_name_valid() function, and a helper function that > returns the message to be given when a given <name, len> pair > is not a good name for an attribute. > > We could later update the message to exactly spell out what the > rules for a good attribute name are, etc. > > Signed-off-by: Junio C Hamano <gitster@xxxxxxxxx> > Signed-off-by: Stefan Beller <sbeller@xxxxxxxxxx> > --- [snip] > +extern int attr_name_valid(const char *name, size_t namelen); > +extern void invalid_attr_name_message(struct strbuf *, const char *, int); > + The symbol 'attr_name_valid()' is not used outside of attr.c, even by the end of this series. Do you expect this function to be used in any future series? (The export is deliberate and it certainly seems like it should be part of the public interface, but ...) In contrast, the 'invalid_attr_name_message()' function is called from code in pathspec.c, which relies on 'git_attr_counted()' to call 'attr_name_valid()' internally to check for validity. :-D ATB, Ramsay Jones