[Devel] Declare const

Kir Kolyshkin kir at openvz.org
Sun Apr 28 22:32:32 PDT 2013


On 04/28/2013 10:19 PM, Igor M Podlesny wrote:
> On 29 April 2013 13:10, Kir Kolyshkin <kir at openvz.org> wrote:
> [...]
>> Applied. Please make subjects more informative,
>> for example in this case it could be something like:
>>
>> unescapestr(): declare src argument as const
>>
>> (I actually fixed it that way)
>     Oookay, actually sometimes I did it alike -- verbose, but then
> started to think that it's kinda redundant since anyway there's a
> commit body describing what's been changed.
>
> --
In the context of say http://git.openvz.org/?p=vzctl;a=summary
lines such as "Declare const" doesn't make any sense.

What I usually try to do is prepend a vzctl command so it makes more 
sense to a user
(such as for example "vzctl set --hostname: stricter argument 
validation"). If there is
something that doesn't make sense for end user, such as refactoring, I 
give a function
name or a file name.




More information about the Devel mailing list