Blair Zajac wrote:
> Daniel Näslund wrote:
>> On Thu, Aug 27, 2009 at 07:29:03PM +0100, Stefan Sperling wrote:
>>> On Thu, Aug 27, 2009 at 06:51:28PM +0200, Daniel Näslund wrote:
>> I used your suggestions for the log message.
>
> Hi Daniel,
>
> Thanks for all your work on svn.
>
> Minor nit. When you have a long line like this:
>
> void
> svn_stream_set_line_transformer_callback(svn_stream_t *stream,
> svn_io_line_transformer_cb_t
> line_transformer_cb)
> {
> stream->line_transformer_cb = line_transformer_cb;
> }
>
> It took me a second to see that line_transformer_cb is not a type but the
> variable name from the previous line. For long lines, we use something like
> this style:
>
> void
> svn_stream_set_line_transformer_callback
> (svn_stream_t *stream,
> svn_io_line_transformer_cb_t line_transformer_cb)
Oops, make that a two space indent:
void
svn_stream_set_line_transformer_callback
(svn_stream_t *stream,
svn_io_line_transformer_cb_t line_transformer_cb)
Blair
------------------------------------------------------
http://subversion.tigris.org/ds/viewMessage.do?dsForumId=462&dsMessageId=2388001
Received on 2009-08-27 22:01:06 CEST