Re: [patch] change increment/decrement operators from postfix to prefix

From: Kinkie <gkinkie_at_gmail.com>
Date: Mon, 2 Jul 2012 16:38:39 +0200

On Mon, Jul 2, 2012 at 4:21 PM, Alex Rousskov
<rousskov_at_measurement-factory.com> wrote:
> On 07/02/2012 04:08 AM, Kinkie wrote:
>> Hi all,
>> this patch (for trunk) is a first chunk of a janitorial task
>> changing all operator++ and operator-- from postfix to prefix form
>> where it doesn't impact the code flow. There are other cases where a
>> more in-depth refactoring may improve readability, but I'll save them
>> for more focused activities.
>
> Ideally, this should not go in without some empirical evidence that it
> helps performance, but I cannot object to this change. As you probably
> know, I advocated using prefix increments in new code for a long time
> (mostly for reasons associated with complex objects such as some iterators).
>
> Please update Squid style guide on wiki to add this rule so that the new
> code has fewer postfix increments.

Done.

>> I'd like to be authorized to commit directly to trunk for this (and
>> subsequent) chunks.
>
> Please do a single merge or a single patch commit if possible. No reason
> to pollute default bzr log with tens of identical comments, IMO. If you
> merge from you branch, that branch may have hundreds of commits, of
> course (they will all be hidden from the default bzr log).

Ok, will do.
I'll keep as a local branch and commit all at once when done or at
least mostly done.

-- 
    /kinkie
Received on Mon Jul 02 2012 - 14:38:46 MDT

This archive was generated by hypermail 2.2.0 : Mon Jul 02 2012 - 12:00:05 MDT