Austin Henriksen

@joshtriplett @scampi I recently opened a PR to fix the opposite issue: #5573, where it was removing a space that should be there: ```diff macro_rules! test_macro { - ($child:expr, Module)...

We already reject these in string literals: ``` /* Matches a universal character code that isn't the correct size, or uses incorrect characters. */ "\\u"{hex}{0,3}[g-zG-Z]* ... report an error ```...

Fixed by #2437 in Java. And Javascript has no mutex locking.

This was fixed in #2288.

But I'm leaving this open, because it's still broken in other languages. Once it's 'un-broken' it sounds like we want to deprecate: `Glacier2::Router::refreshSession` and `IceGrid::Admin::keepAlive` I suspect we'll also want...

I'd be fine with adding this, although I'm not sure how much value it'll provide.

So, we aren't even consistent ourselves. The metadata you use to mark something deprecated is `deprecate` But the string you pass in to ignore deprecation warnings is `deprecated` Should we...

This was implemented in #2166.

This was 'accidentally' fixed by #2770, which removed the `IceUtilInternal` package, fixing this warning.

Just for the record, when I try to compile these Slice files manually, they indeed generate a bunch of syntax errors.