Conversation
mharis001
left a comment
There was a problem hiding this comment.
I dislike this since it does not correctly support the following situations:
- Multi-line comments that start each line with
//. Lines that continued the previous line's sentence are forced to start with a capital, which IMO is a worse style and less readable. - Comments that start with command names or config entry names are forced to start with a capital, which makes their casing incorrect.
Yep, this sounds reasonable, and easily done. If comments are chained one after another, then, if the previous comment doesn't end with a
For this I don't have a very good solution apart from loading everything from SQF into the script, which would make things very ugly. I'd guess it would be easier to just change the sentence structure so that the commands don't start first. If the first point would be implemented it would, for the most part, remove the requirement for this point. |
Yeah, that should work well.
Could get the command names from the wiki and parse them using a simple regex. |
When merged this pull request will:
*.sqf,*.cppand*.hppfiles (only//comments are checked)http/https)For the future (will not be included in this PR):