Re: svn commit: rev 6716 - in branches/fs-schema-changes/subversion: include libsvn_fs libsvn_repos svnadmin tests/libsvn_fs tests/libsvn_repos

kfogel@collab.net wrote:

>cmpilato@tigris.org writes:
>
>
>>+/* Our coolio opaque history object. */
>>+struct svn_fs_history_t
>>+{
>>+ svn_fs_t *fs;
>>+ const char *path;
>>+ svn_revnum_t revision;
>>+ const char *path_hint;
>>+ svn_revnum_t rev_hint;
>>+ int is_interesting;
>>+};
>>
>>
>
>So coolio that none of its fields need to be documented :-) ?
>(Seriously -- I try not to be a stickler about internal structs when
>the fields are obvious, but in this case some docs might clarify,
>particularly the hint fields and is_interesting).
>
And sometimes I wonder why we have svn_boolean_t, then people keep using
int...

-- 
Brane Čibej   <brane_at_xbc.nu>   http://www.xbc.nu/brane/
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org