[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