[lvm-devel] [PATCH 3/3] Change default prefix.

[Zdenek Kabelac]

Dne 6.10.2011 11:43, Zdenek Kabelac napsal(a):
> Use ""  instead of "  ".
> So commands like lvs print from the begining of the line.
> (no need to trim space around).
>
> Signed-off-by: Zdenek Kabelac<zkabelac at redhat.com>
> ---
>   doc/example.conf.in   |    6 +++---
>   lib/config/defaults.h |    6 +++++-
>   2 files changed, 8 insertions(+), 4 deletions(-)
>
> diff --git a/doc/example.conf.in b/doc/example.conf.in
> index d38f90e..58748cc 100644
> --- a/doc/example.conf.in
> +++ b/doc/example.conf.in
> @@ -248,9 +248,9 @@ log {
>       command_names = 0
>
>       # A prefix to use before the message text (but after the command name,
> -    # if selected).  Default is two spaces, so you can see/grep the severity
> -    # of each message.
> -    prefix = "  "
> +    # if selected). Set to "  " to revert to the default behaviour
> +    # prior to version 2.02.89.
> +    prefix = ""
>
>       # To make the messages look similar to the original LVM tools use:
>       #   indent = 0
> diff --git a/lib/config/defaults.h b/lib/config/defaults.h
> index be016f6..df1dcba 100644
> --- a/lib/config/defaults.h
> +++ b/lib/config/defaults.h
> @@ -99,7 +99,11 @@
>   #define DEFAULT_MAXIMISE_CLING 1
>   #define DEFAULT_CLUSTERED 0
>
> -#define DEFAULT_MSG_PREFIX "  "
> +/*
> + * NOTE: _find_config_str() does not allow empty string "",
> + * but when the default msg prefix is also "" it works as expected.
> + */
> +#define DEFAULT_MSG_PREFIX ""

The comment here is not true anymore with patch 2/3 as I've reorderd patch set 
somewhat. So committed version will not have it.

Zdenek