Re: [docs] [PATCH] manuals: simplify colon usage

["Quentin Schulz" <foss@0leil.net>]

On April 15, 2021 5:10:04 PM UTC, Michael Opdenacker <michael.opdenacker@bootlin.com> wrote:
>Hi Quentin
>
>On 4/15/21 6:55 PM, Quentin Schulz wrote:
>> On Thu, Apr 15, 2021 at 06:14:33PM +0200, Michael Opdenacker wrote:
>>> On 4/15/21 6:09 PM, Michael Opdenacker wrote:
>>>> - This replaces instances of ": ::" by "::", which
>>>>   generates identical HTML output
>> Could you explain how you did the changes? I'm pretty sure you are not
>> crazy enough to do all those changes by hand :)
>
>
>Yes, I am!
>I was using:
>
>git grep -Ovi ": ::"
>
>This takes at most a few minutes to go through all the matches and
>substitute them all in the same way. Actually, when the number of
>changes is still not prohibitive such as here, my personal preference is
>to check each instance before making the substitution (just alternating
>between "n" and "." in vim) to make sure it's not a special case that
>would be different. Otherwise, I'd have to inspect each line in the
>generated patch which would be an equivalent effort, wouldn't it?
>

I would craft a command and then check the patch. But honestly, I don't know if it's any better. It's anyway time consuming.

>By the way, which remaining occurrences did you find?
>

I used ag -R ":\n\s*::" to find them. It'll probably work fine with git grep instead of ag. Or :%s/:\n\s*::/::/gc in vim I guess?

Cheers,
Quentin