Bernard Normier

This draft PR shows the MATLAB optional proxy bugs.

The Slice compiler should reject optional parameters and fields with classes, just like slicec does. Rationale: despite our best efforts, we could not find a way to encode correctly such...

We should switch to nullable (`?`) for the mapping of Slice optional to C#. The Java-like Optional we provide is odd and not convenient to use. We should also drop...

#1633 shows 2 bugs: - an unset optional proxy is not == Ice.Unset - the generated code code an operation with optional proxies fails when these parameters are not set

Deprecate Object used as Value

1

Passing interfaces by value is deprecated as of Ice 3.7, however, `Object` remains a non-deprecated way to specify `Value`. For example: ``` module M { interface Intf { Intf f();...

Incorrect escaping in C++ generated code

1

The generated code for test Slice/escape/Key.ice does not compile if you make the local interface in that file non-local. Bogus code: ```cpp /// \cond INTERNAL bool _cpp_and::_cpp_friend::_iceD_goto(::IceInternal::Incoming& inS, const ::Ice::Current&...

For Ice 3.8, the min C++ standard for our entire C++ source base should be c++17. As a result, we can use c++17 features (including standard library features such as...

We support 'tag' as an alias for 'optional' in the Slice grammar, however, the standard term remains 'optional' and therefore that's what error messages should produce. Unfortunately, https://github.com/zeroc-ice/ice/pull/1590 incorrectly replaced...

We should add noexcept to all C++ proxy functions that do not throw, and document this "noexcept" in the Manual. See https://doc.zeroc.com/ice/3.7/client-side-features/proxies/proxy-methods

The recommended shutdown procedure for a server is to shut it down before destroying it. In some demos, the server is correctly shut down, for example: https://github.com/zeroc-ice/ice-demos/blob/7d29cb7548a7c805d2a5fc612f1e7b2417e49bff/cpp11/Ice/hello/Server.cpp#L37 In other demos,...