Prism C++ incorrectly colors overloaded operator< and operator> in sequence

[AlexLearnCpp]

Repro steps:

• Go to https://prismjs.com/test.html#language=cpp
• Paste in this code:

bool operator< (const Foo& f1, const Foo& f2)
{
    return f1.foo < f2.foo;
}

bool operator> (const Foo& f1, const Foo& f2)
{
    return f1.foo > f2.foo;
}

Note that foo < f2.foo from the top function and the > in operator> in the bottom function are the wrong color.

[AlexLearnCpp]

Issue still outstanding

[LeaVerou]

Hi there, this is what I see:

I see the issue with foo < f2.foo, but operator> in the second function seems fine?

[AlexLearnCpp]

Agreed. It appears this has been partially fixed since it was originally filed. Thanks for reviewing.

[ferdnyc]

@AlexLearnCpp

It appears this has been partially fixed since it was originally filed.

Actually, I don't think it has. If anything, it's even more broken, despite the results appearing better.

If you turn on "Show tokens", you can see that the only reason the body of operator> highlights "correctly" is that Prism is matching the > in operator> with the < from the previous function, instead of recognizing it as a new function declaration!

(It's that three-level nested boxiness around everything from the foo in f1.foo to the end of operator> that's the troubling part.)

[LeaVerou]

It's neither fixed nor more broken folks — Prism was feature frozen before this bug was filed so we could work on v2 and the maintainer who was leading that effort got swamped with other things. My plan is currently to take over work on v2 this fall.