Austin Henriksen

Currently our error handling cases in the Slice grammar amount to 'continue absorbing tokens until we reach a `;` character, or a `)` character when parsing an operation's parameters. This...

Replace the 'format(...)' Metadata with 'sliced-format'

1

Slice currently supports 3 different arguments for the `format` metadata: `default`, `compact`, and `sliced`. In 4.0 `compact` will become the new default, cutting this list down to 2 options. Having...

Currently, when we use `@see` attributes in Slice doc comments, we don't end them in periods. For example: ``` /// The remote invocation metrics map. /// @see RemoteMetrics MetricsMap remotes;...

Doxygen prints lots of warnings about misnamed or missing parameters in the C++ documentation that we should either fix or suppress. For example: ``` ice/cpp/include/generated/Ice/Router.h:456: warning: argument 'sent' of command...

Hello! I'm the maintainer of the Slice language! A few months ago, we released a new 'edition' of our language's syntax, so there are now 2 separate syntaxes for Slice....

Java's `Ice/translator` Tests are not Run (and are out of date)

5

These tests seem to be checking the java mapping, with an emphasis on the `module`->`package` mapping. But, these tests are clearly never run. They aren't in the gradle file: https://github.com/zeroc-ice/ice/blob/main/java/test/slice.gradle...

I was fixing some of our `errorDetection` tests and noticed that the line number are off. For example, take `WarningInvalidMetaData.ice` In the source we have: https://github.com/zeroc-ice/ice/blob/60c41eab191ebdc522ac402ff4e0ec96f3c9351a/cpp/test/Slice/errorDetection/WarningInvalidMetaData.ice#L55-L59 But in the expected...

Doxygen Theme Tweaks

2

I think we all agree this theme is an overall improvement over the current Doxygen theme, but I think there are some things that could be improved, or that used...

When regenerating the Slice scanner (located at `cpp/src/Slice/Scanner.l`), the following warning is emitted: ``` src/Slice/Scanner.l:550: warning, -s option given but default rule can be matched ``` This message means that...

Always Place Spaces after Commas in Macro Match Arms

15

This PR fixes #5573 by ensuring a space is always placed after commas in macro match arms, and updates a test that exhibited the mistake.