Re: revision/date handling in the client library

Philip Martin <philip@codematters.co.uk> writes:
> typedef struct svn_client_revision_s {
>
> enum svn_client_revision_type type;
>
> svn_revnum_t number;
> apr_time_t date;
>
> } svn_client_revision_t;

I think we could `union' those last two fields, no?

(Btw, is "_s" your struct suffix? We usually just use _t for both...)

> Look at the ugly revision handling in svn_client_diff to see how bad
> it gets at present.
>
> Comments?

I like! Would you like me to do it, or were you planning to do it?
I'm fine with either way.

-K

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org
Received on Sat Oct 21 14:37:00 2006