This avoids maintaining the same options in two different places.
|3 ↗||(On Diff #214288)|
I bet that if you adopt my changes to not use the classes below, this is unnecessary.
|15 ↗||(On Diff #214288)|
You can avoid the need for the class here by rewording slightly to say "Allow the tool to..."
|21 ↗||(On Diff #214288)|
Here and in the disable case, you can avoid the class by rewording slightly to something like "when operating on archives".
|41 ↗||(On Diff #214288)|
Again, I think you can remove the reference to the tool name entirely here i.e. "GNU's --strip-all".
Git is smart. The top of the patch says "copy from" / "copy to" https://reviews.llvm.org/file/data/ujocforfsc2wd7mdfbdg/PHID-FILE-gbjfb3ce67cfiabcoyar/D65991.id214422.diff. The manpage git-diff(1) explains the patch format but does not describe git's copying detection algorithm. Does anyone know? I'd love to find out more. @MaskRay, you have sparked my interest! :)