On 04/28/2013 10:19 PM, Igor M Podlesny wrote:
On 29 April 2013 13:10, Kir Kolyshkin <k...@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.
_______________________________________________
Devel mailing list
Devel@openvz.org
https://lists.openvz.org/mailman/listinfo/devel
