Bug 27567 – False positives in namespace indentation checks

• Component: Tools / Tests
• Status: RESOLVED FIXED

[Jakob Petsovits]

Yong Li discovered a false positive for the namespace indentation check in implementation files, triggered by this piece of code:

"""
namespace WebCore {

PlatformMenuDescriptionType::PlatformMenuDescriptionType()
    : hMenu(::CreatePopupMenu())
    , itemCount(0)
{
}
"""

It turns out that the regular expression skipping goto labels was too permissive, and thus treated the line with the constructor's signature as label, skipping it and (then correctly) causing the error for the indented second line. Not cool, sorry for that bug.

This patch improves on the concerned code, makes the unit tests more demanding, and fixes the bug. Please review!

[Jakob Petsovits]

Created attachment 33298 [details]
Fix false positives in namespace indentation checks

[Adam Treat]

Comment on attachment 33298 [details]
Fix false positives in namespace indentation checks

Next time put more detail in the ChangeLog as you did in the bug report.

Also, I'd like a comment for the added regex.

[Jakob Petsovits]

Created attachment 33300 [details]
Fix false positives in namespace indentation checks (try 2)

Better patch, more verbosely describing what is being done. (No changes in functionality, though.)

[Adam Treat]

Landed with r46248.

[Maxime Simon]

*** Bug 27559 has been marked as a duplicate of this bug. ***