On Thu, 2005-03-10 at 19:30, Julian Foad wrote:
> Madan U Sreenivasan wrote:
> > On Wed, 2005-03-09 at 22:54, Julian Foad wrote:
> >>Madan U Sreenivasan wrote:
> > But svn_subst_keywords_t is not related to svn_canonicalize_keywords()
> > .... :-? bigger question is which part of the .h, should I add a NOTE
> > that is only a pointer to another file and not related to any part of
> > the .h itself....
>
> If svn_subst_keywords_t is not related to svn_canonicalize_keywords(), then why
> did you put the comment adjacent to the definition of that type? Perhaps the
> comment should instead go where SVN_KEYWORD_... are defined.
oh, rest of the code that use the #define reside in subst.c. So, I
wanted a comment in svn_subst.h to point to props.c mentioning that the
#defines are used there too ( this is in case someone decides to add a
new keyword in future ). This was John's suggestion too.
Now, regarding the location of the comment in svn_subst.h, I was not
too sure... thats why I put it near 'related' code...
On the other hand, svn_types.h is a better place to put this.... but I
was not sure about touching a file as widely used as svn_types.h for
commenting.... anyways, since you suggested it too, I think I will go
ahead and put this comment in svn_types.h....
Regards,
Madan.
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org
Received on Fri Mar 11 08:51:48 2005