Skip to content
  • Hanna Reitz's avatar
    63898712
    option: Make option help nicer to read · 63898712
    Hanna Reitz authored
    
    
    This adds some whitespace into the option help (including indentation)
    and puts angle brackets around the type names.  Furthermore, the list
    name is no longer printed as part of every line, but only once in
    advance, and only if the caller did not print a caption already.
    
    This patch also restores the description alignment we had before commit
    9cbef9d6, just at 24 instead of 16 characters like we used to.
    This increase is because now we have the type and two spaces of
    indentation before the description, and with a usual type name length of
    three chracters, this sums up to eight additional characters -- which
    means that we now need 24 characters to get the same amount of padding
    for most options.  Also, 24 is a third of 80, which makes it kind of a
    round number in terminal terms.
    
    Finally, this patch amends the reference output of iotest 082 to match
    the changes (and thus makes it pass again).
    
    Signed-off-by: default avatarMax Reitz <mreitz@redhat.com>
    Reviewed-by: default avatarMarc-André Lureau <marcandre.lureau@redhat.com>
    Signed-off-by: default avatarKevin Wolf <kwolf@redhat.com>
    63898712
    option: Make option help nicer to read
    Hanna Reitz authored
    
    
    This adds some whitespace into the option help (including indentation)
    and puts angle brackets around the type names.  Furthermore, the list
    name is no longer printed as part of every line, but only once in
    advance, and only if the caller did not print a caption already.
    
    This patch also restores the description alignment we had before commit
    9cbef9d6, just at 24 instead of 16 characters like we used to.
    This increase is because now we have the type and two spaces of
    indentation before the description, and with a usual type name length of
    three chracters, this sums up to eight additional characters -- which
    means that we now need 24 characters to get the same amount of padding
    for most options.  Also, 24 is a third of 80, which makes it kind of a
    round number in terminal terms.
    
    Finally, this patch amends the reference output of iotest 082 to match
    the changes (and thus makes it pass again).
    
    Signed-off-by: default avatarMax Reitz <mreitz@redhat.com>
    Reviewed-by: default avatarMarc-André Lureau <marcandre.lureau@redhat.com>
    Signed-off-by: default avatarKevin Wolf <kwolf@redhat.com>
Loading