[PATCH] D68998: [docs][llvm-ar] Update llvm-ar command guide

Wed Oct 16 20:27:38 PDT 2019

ruiu accepted this revision.
ruiu added a comment.

LGTM

================
Comment at: llvm/docs/CommandGuide/llvm-ar.rst:55
+
+Long Options
+~~~~~~~~~~~~
----------------
gbreynoo wrote:
> ruiu wrote:
> > I see why you named this section "Long Options", but the section that would have been named "short options" is actually named "operations", and this section contains short options such as `-h`, so this section name seems a bit odd to me.
> I think the distinction between "Operation", "Modifier" and "Other" is helpful to the user but I'm not sure what to call this "Other" section. "Non key letter" feels long winded, and `-h` doesn't  fit that either. 
> 
> Maybe it should be called "Other" and put underneath the other two sections. `M` and `h` could be moved to the "Operation" section and `--help` would be considered a synonym. What do you think?
I think "Other" is a better name.

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D68998/new/

https://reviews.llvm.org/D68998