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

Re: svn commit: r36363 - trunk/subversion/libsvn_ra_serf

From: Senthil Kumaran S <senthil_at_collab.net>
Date: Fri, 06 Mar 2009 19:13:28 +0530

Greg Stein wrote:
> Geez... making a duplicate to satisfy a const warning? That is *not*
> the right approach.
>
> Instead, you can easily see that the handler is not attempting to
> modify the header at all. It should have been declared as a const
> parameter. *That* is the proper change.

Greg, thanks for your comments, I shall see to it that, such things does not
happen in future. Let me get closer in reviewing the changes, than what I ve
did now.

Thank You.

-- 
Senthil Kumaran S
http://www.stylesen.org/
------------------------------------------------------
http://subversion.tigris.org/ds/viewMessage.do?dsForumId=462&dsMessageId=1277372
Received on 2009-03-06 14:43:58 CET

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

This site is subject to the Apache Privacy Policy and the Apache Public Forum Archive Policy.