Fix issue #5444 - #5566
Conversation
reportImplementationExpectedError: The next node in the tree is not
necessarily consecutive. This happens due to syntax errors, e.g.
class C { foo(), foo(); }
|
Hi jeffreymorlan, I'm your friendly neighborhood Microsoft Pull Request Bot (You can call me MSBOT). Thanks for your contribution! TTYL, MSBOT; |
|
Can you please add unit tests. |
|
To be clear, Mohamed Hegazy (@mhegazy) does't mean a unit test, he means an end-to-end test. Check out Adding a Test in our CONTRIBUTING.md. |
There was a problem hiding this comment.
Can you add a comment that you need this test because garbage declarations may be introduced between overloads and cause issues according to our expectations
|
👍 Vladimir Matveev (@vladima) could you take a look? |
|
👍 after Daniel Rosenwasser (@DanielRosenwasser) note about the comment is addressed |
reportImplementationExpectedError: The next node in the tree is not
necessarily consecutive. This happens due to syntax errors, e.g.
Edit by Daniel Rosenwasser (@DanielRosenwasser): Fixes #5444