rnveach

Looking at the bytecode, there is no difference between `:` and `->`.

Basically a duplicate of #13345. https://github.com/checkstyle/checkstyle/blob/b0961ea6a54e1d3b920857e708cfc1cac8c3734d/src/xdocs-examples/java/com/puppycrawl/tools/checkstyle/checks/regexp/RegexpMultilineCheckExamplesTest.java#L27-L28 https://github.com/checkstyle/checkstyle/blob/b0961ea6a54e1d3b920857e708cfc1cac8c3734d/src/xdocs-examples/java/com/puppycrawl/tools/checkstyle/checks/regexp/RegexpMultilineCheckExamplesTest.java#L94 Examples is disable and is not guaranteed to be correct. Issue will be resolved under #13345 .

This is not blocking you from fixing this if you desire. It is just a duplicate of the other issue since the test is disabled and has no guarantees to...

Just so you are aware, the data you are gathering and processing will be in doubt until https://github.com/checkstyle/checkstyle/issues/13345 is resolved.

I am good to approve.

> gone through our default regression project list, but didn't see any valid test case Report is fine, just that if original regression showed nothing for this change I would...

> I noticed that we are not using local-checkstyle|local| config in the default project list. And if I use that, there will be some random differences. > If I run...

If regression prints too many violations then it will run out of memory. That is most likely the case here. If you ran this before with no issues (same config/project),...

> please share your opinion. Which am I sharing on?

Here is my answer - https://github.com/checkstyle/checkstyle/pull/14673#discussion_r1544998932 Let me know when we have decided on a group decision.