| apol | |
| kfunk |
| KDevelop |
I missed a possibility to open documents (from the commandline) at a specified cursor position and it was only after looking through the code that I realised that feature was already implemented.
I thought it useful to document the feature in the --help usage info; this changeset is a proposed draft of just that.
N/A
| Automatic diff as part of commit; lint not applicable. |
| Automatic diff as part of commit; unit tests not applicable. |
Wouldn't it be easier to write [FILE(:line)(:column)...]? It feels a bit too verbose to me.
Also since some time it supports directories that are opened as a project. This should probably be reflected too.
Aleix Pol Gonzalez wrote on 20170703::12:44:35 re: "D6477: KDevelop --help: document the possibility to open files at a specific cursor position"
Wouldn't it be easier to write `[FILE(:line)(:column)...]`? It feels a bit too verbose to me.
My initial idea was [FILE[:line[:column]]...] but that felt a bit too cryptic/illegible to me in practice.
If the surrounding square brackets indicate the optional nature of FILES we should probably stick to those brackets, and not use another form with the same intention.
Also since some time it supports directories that are opened as a project. This should probably be reflected too.
i18n("Files to load, or directories to load as projects"), "[FILE[:line[:column]]...] [DIRECTORY...]" ?
/me just learned about http://docopt.org/ -- cool -- so what about:
i18n("Files to load, or directories to load as projects"), "[FILE[:line[:column]] | DIRECTORY]..."?
I could live with that.
Bonus points for adding two example usages at the bottom of the help text in a new Examples section(?)
i18n("Files to load, or directories to load as projects"), "[FILE[:line[:column]] | DIRECTORY]..."?
I could live with that.
Will do.
Bonus points for adding two example usages at the bottom of the help text in a new Examples section(?)
What help text, the one printed by kdevelop --help?
> What help text, the one printed by `kdevelop --help`? Yes. But nevermind, not needed.
Good, because as far as I can see there are no hooks to do that. At least not where it makes sense to add this text (below the option list).
Please push to master branch.
Will do.