Le 10/06/2013 10:37, Matthieu Moy a écrit :
> Célestin Matte <celestin.ma...@ensimag.fr> writes:
>> @@ -1285,8 +1285,7 @@ sub get_mw_namespace_id {
>>              # Look at configuration file, if the record for that namespace 
>> is
>>              # already cached. Namespaces are stored in form:
>>              # "Name_of_namespace:Id_namespace", ex.: "File:6".
>> -            my @temp = split(/\n/, run_git("config --get-all remote."
>> -                                            . $remotename 
>> .".namespaceCache"));
>> +            my @temp = split(/\n/, run_git("config --get-all 
>> remote.${remotename}.namespaceCache"));
> I tend to prefer the former, as it avoids long lines (> 80 columns)

The 80-columns rule is already broken in *many* places in this file.
I know this is not a valid reason to do it more often, but in my opinion
the second version is easier to read (it's easy to miss the dots in the
first version), so we would gain from the change.

Célestin Matte
To unsubscribe from this list: send the line "unsubscribe git" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to