[svn.haxx.se] · SVN Dev · SVN Users · SVN Org · TSVN Dev · TSVN Users · Subclipse Dev · Subclipse Users · this month's index

Re: [PATCH] ctypes-python-bindings branch Move client helper classes to new file

From: Daniel Rall <dlr_at_collab.net>
Date: 2007-05-31 01:28:08 CEST

> * csvn/client.py:
> (SvnDate, LogEntry, _LogMessageReciever, User, RepositoryURI, ):
                                                              ^^
> Removed from this file, moved to csvn/client_helpers.py. _fs* still
> reside here, due to visibility.
>
> * csvn/client_helpers.py:
> (SvnDate, LogEntry, _LogMessageReciever, User, RepositoryURI, ): New
                                                              ^^
> home for all these functions.

You can drop the trailing comma and space from the symbol list. Also,
I recommend indenting the following line a little bit to make it more
obvious that it's part of the comment corresponding to the above
symbol list.

Nice work!

  • application/pgp-signature attachment: stored
Received on Thu May 31 01:29:46 2007

This is an archived mail posted to the Subversion Dev mailing list.