After some internal discussion we agreed that we can simplify things and get consistent behaviour by using dot everywhere in diagnostics.

LGTM but my review was fairly superficial.

We reconsidered this in light of the policy - thanks for pointing that out Richard!
Just to be sure that I understand it - the policy is meant for CLI and not serialized diagnostics, right?

Eric, thanks for the context. I am now clarifying internally and if possible would like to simplify the whole format business by using just one delimiter everywhere. Would that make sense to you or do you think we should respect delimiter used in sources?

Sorry for me being dense here - since the output format is determined by input source code there's more work to do.

Hi Bruno, I just noticed couple of implementation details.

I am not 100% sure it's the best thing to set printing style at the point of parsing version tuples but I am not sure it's bad either. Unless someone convinces me otherwise I would rather not do any major changes.

Fixed the test. It turned out that version component separator for availability is set during parsing which is why all other tests (including my bad one) are passing.

Sorry, my bad. I tried to get rid of dependency on Foundation.h and didn't check the test is relevant for the fix after that.

Volodymyr, could you please confirm that the non-anonymous vs non-inline logic makes sense to you?

Addressed comments + proposal for refactoring of the namespace-related logic.

I forgot to mention review in commit message.

[ultranit] Sorry, just to be sure - did you really mean any number of dashes

By "Comments should be allowed at the end of test files." I meant that we shouldn't log it as an exceptional state. Should've worded that better.

Include the correct test.

Hi Sam, could you please take a look at this minor fix?

Hi Sam, could you please take a look at this minor fix?