[systemd-devel] [PATCH 2/2] accelerometer: display short options too

[Paul Menzel]

Am Dienstag, den 30.12.2014, 18:58 +0100 schrieb Robert Milasan:
> ---
>  src/udev/accelerometer/accelerometer.c | 4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)

Awesome! Thank you for splitting these up and submitting a v2!

> diff --git a/src/udev/accelerometer/accelerometer.c
> b/src/udev/accelerometer/accelerometer.c index 23d6f78..9c13d97 100644
> --- a/src/udev/accelerometer/accelerometer.c
> +++ b/src/udev/accelerometer/accelerometer.c
> @@ -198,8 +198,8 @@ static void test_orientation(struct udev *udev,
>  static void help(void)
>  {
>          printf("Usage: accelerometer [options] <device path>\n"
> -               "  --debug         debug to stderr\n"
> -               "  --help          print this help text\n\n");
> +               "  -d,--debug         debug to stderr\n"
> +               "  -h,--help          print this help text\n\n");

1. I still wonder if the description is aligned after this patch is
applied.
2. A space after the comma, `-d, --debug` would be helpful in my
opinion.

>  }
>  
>  int main (int argc, char** argv)


Thanks,

Paul
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 181 bytes
Desc: This is a digitally signed message part
URL: <http://lists.freedesktop.org/archives/systemd-devel/attachments/20141230/f816e497/attachment.sig>