Drop unneeded capture groups

[CRS-migration-bot]

Issue for tracking original pull request created by user fgsch on date 2018-06-11 17:15:49.
Link to original PR: SpiderLabs/owasp-modsecurity-crs#1124.

HEAD is: 5dfb9b5af62eb52f163652996bf814093e9fd39e
BASE is: 5f01890
Follow up of #1111.

This covers all the remaining rules. Minor style changes while here.

For the curious, for this PR I wrote a small program that prints the number of capture groups using PCRE_INFO_CAPTURECOUNT and run it through all the patterns to have a list of rules that needed reviewing.

[CRS-migration-bot]

User csanders-git commented on date 2018-07-10 01:22:48:

clearly equivalent... G2G

[CRS-migration-bot]

User csanders-git commented on date 2018-07-10 01:22:53:

clearly equivalent... G2G

[CRS-migration-bot]

User csanders-git commented on date 2018-07-10 01:24:03:

Seems to be equivalent, good to go.

[CRS-migration-bot]

User csanders-git commented on date 2018-07-10 01:24:11:

Seems to be equivalent, good to go.

[CRS-migration-bot]

User csanders-git commented on date 2018-07-10 01:24:41:

this is a comment...?

[CRS-migration-bot]

User fgsch commented on date 2018-07-10 09:52:37:

Yes, this is to keep the comment in sync with the change below.

[CRS-migration-bot]

User franbuehler commented on date 2018-07-31 04:25:13:

Comment was in sync with the rule below. No change needed here.

[CRS-migration-bot]

User fgsch commented on date 2018-07-31 11:08:33:

franbuehler No, I changed the regexp. Before:

(?i:([...

Now:

(?i)[...

So I updated the comment to be in sync with my change.