zeroc-ice 3.8.1 → 3.8.3

data/dist/ice/cpp/src/Ice/generated/RemoteLogger.cpp

@@ -1,6 +1,6 @@
 // Copyright (c) ZeroC, Inc.
 
-// slice2cpp version 3.8.1
+// slice2cpp version 3.8.3
 // <auto-generated>Generated from Slice file 'RemoteLogger.ice'.</auto-generated>
 
 #ifndef ICE_API_EXPORTS
@@ -33,7 +33,7 @@
 #   if ICE_INT_VERSION % 100 >= 50
 #       error Beta header file detected
 #   endif
-#   if ICE_INT_VERSION % 100 < 1
+#   if ICE_INT_VERSION % 100 < 3
 #       error Ice patch level mismatch!
 #   endif
 #endif

data/dist/ice/cpp/src/Ice/generated/ReplyStatus.cpp

@@ -1,6 +1,6 @@
 // Copyright (c) ZeroC, Inc.
 
-// slice2cpp version 3.8.1
+// slice2cpp version 3.8.3
 // <auto-generated>Generated from Slice file 'ReplyStatus.ice'.</auto-generated>
 
 #ifndef ICE_API_EXPORTS
@@ -34,7 +34,7 @@
 #   if ICE_INT_VERSION % 100 >= 50
 #       error Beta header file detected
 #   endif
-#   if ICE_INT_VERSION % 100 < 1
+#   if ICE_INT_VERSION % 100 < 3
 #       error Ice patch level mismatch!
 #   endif
 #endif
@@ -64,6 +64,8 @@ Ice::operator<<(std::ostream& os, ReplyStatus value)
             return os << "InvalidData";
         case ReplyStatus::Unauthorized:
             return os << "Unauthorized";
+        case ReplyStatus::NotSupported:
+            return os << "NotSupported";
         default:
             return os << static_cast<std::int32_t>(value);
     }

data/dist/ice/cpp/src/Ice/generated/Router.cpp

@@ -1,6 +1,6 @@
 // Copyright (c) ZeroC, Inc.
 
-// slice2cpp version 3.8.1
+// slice2cpp version 3.8.3
 // <auto-generated>Generated from Slice file 'Router.ice'.</auto-generated>
 
 #ifndef ICE_API_EXPORTS
@@ -33,7 +33,7 @@
 #   if ICE_INT_VERSION % 100 >= 50
 #       error Beta header file detected
 #   endif
-#   if ICE_INT_VERSION % 100 < 1
+#   if ICE_INT_VERSION % 100 < 3
 #       error Ice patch level mismatch!
 #   endif
 #endif

data/dist/ice/cpp/src/Ice/generated/SliceChecksumDict.cpp

@@ -1,6 +1,6 @@
 // Copyright (c) ZeroC, Inc.
 
-// slice2cpp version 3.8.1
+// slice2cpp version 3.8.3
 // <auto-generated>Generated from Slice file 'SliceChecksumDict.ice'.</auto-generated>
 
 #ifndef ICE_API_EXPORTS
@@ -33,7 +33,7 @@
 #   if ICE_INT_VERSION % 100 >= 50
 #       error Beta header file detected
 #   endif
-#   if ICE_INT_VERSION % 100 < 1
+#   if ICE_INT_VERSION % 100 < 3
 #       error Ice patch level mismatch!
 #   endif
 #endif

data/dist/ice/cpp/src/Ice/generated/Version.cpp

@@ -1,6 +1,6 @@
 // Copyright (c) ZeroC, Inc.
 
-// slice2cpp version 3.8.1
+// slice2cpp version 3.8.3
 // <auto-generated>Generated from Slice file 'Version.ice'.</auto-generated>
 
 #ifndef ICE_API_EXPORTS
@@ -34,7 +34,7 @@
 #   if ICE_INT_VERSION % 100 >= 50
 #       error Beta header file detected
 #   endif
-#   if ICE_INT_VERSION % 100 < 1
+#   if ICE_INT_VERSION % 100 < 3
 #       error Ice patch level mismatch!
 #   endif
 #endif

data/dist/ice/cpp/src/IceDiscovery/LookupI.cpp

@@ -30,6 +30,7 @@ IceDiscovery::Request::retry()
 void
 IceDiscovery::Request::invoke(const string& domainId, const vector<pair<LookupPrx, LookupReplyPrx>>& lookups)
 {
+    ++_generation;
     _lookupCount = lookups.size();
     _failureCount = 0;
     Ice::Identity id;
@@ -42,8 +43,15 @@ IceDiscovery::Request::invoke(const string& domainId, const vector<pair<LookupPr
 }
 
 bool
-IceDiscovery::Request::exception()
+IceDiscovery::Request::exception(size_t generation)
 {
+    // Ignore a delayed failure from an earlier invocation round: it must not count against the current round, which
+    // reset _failureCount and may still have outstanding lookups.
+    if (generation != _generation)
+    {
+        return false;
+    }
+
     // If all the invocations on all the lookup proxies failed, report it to the locator.
     if (++_failureCount == _lookupCount)
     {
@@ -69,7 +77,17 @@ AdapterRequest::AdapterRequest(const LookupIPtr& lookup, const std::string& adap
 bool
 AdapterRequest::retry()
 {
-    return _proxies.empty() && --_retryCount >= 0;
+    if (_proxies.empty() && --_retryCount >= 0)
+    {
+        // We only reach here with _proxies empty, i.e. no replica-group response arrived. _latency is set only
+        // together with inserting into _proxies, so the latency timer is not running.
+        assert(_latency == chrono::nanoseconds::zero());
+
+        // Restart the replica-group latency window so it's measured from this round rather than from request creation.
+        _start = chrono::steady_clock::now();
+        return true;
+    }
+    return false;
 }
 
 bool
@@ -77,6 +95,7 @@ AdapterRequest::response(const ObjectPrx& proxy, bool isReplicaGroup)
 {
     if (isReplicaGroup)
     {
+        _proxies.insert(proxy);
         if (_latency == chrono::nanoseconds::zero())
         {
             _latency = chrono::duration_cast<chrono::nanoseconds>(chrono::steady_clock::now() - _start) *
@@ -84,7 +103,6 @@ AdapterRequest::response(const ObjectPrx& proxy, bool isReplicaGroup)
             _lookup->timer()->cancel(shared_from_this());
             _lookup->timer()->schedule(shared_from_this(), _latency);
         }
-        _proxies.insert(proxy);
         return false;
     }
     finished(proxy);
@@ -127,7 +145,8 @@ AdapterRequest::invokeWithLookup(const string& domainId, const LookupPrx& lookup
         _id,
         lookupReply,
         nullptr,
-        [self = shared_from_this()](exception_ptr ex) { self->_lookup->adapterRequestException(self, ex); });
+        [self = shared_from_this(), generation = _generation](exception_ptr ex)
+        { self->_lookup->adapterRequestException(self, ex, generation); });
 }
 
 void
@@ -155,7 +174,8 @@ ObjectRequest::invokeWithLookup(const string& domainId, const LookupPrx& lookup,
         _id,
         lookupReply,
         nullptr,
-        [self = shared_from_this()](exception_ptr ex) { self->_lookup->objectRequestException(self, ex); });
+        [self = shared_from_this(), generation = _generation](exception_ptr ex)
+        { self->_lookup->objectRequestException(self, ex, generation); });
 }
 
 void
@@ -173,6 +193,24 @@ LookupI::LookupI(LocatorRegistryIPtr registry, const LookupPrx& lookup, const Ic
       _domainId(properties->getIceProperty("IceDiscovery.DomainId")),
       _timer(IceInternal::getInstanceTimer(lookup->ice_getCommunicator()))
 {
+    if (_timeout <= chrono::milliseconds::zero())
+    {
+        throw PropertyException{__FILE__, __LINE__, "property 'IceDiscovery.Timeout' must be greater than 0"};
+    }
+    if (_retryCount < 0)
+    {
+        throw PropertyException{
+            __FILE__,
+            __LINE__,
+            "property 'IceDiscovery.RetryCount' must be greater than or equal to 0"};
+    }
+    if (_latencyMultiplier < 1)
+    {
+        throw PropertyException{
+            __FILE__,
+            __LINE__,
+            "property 'IceDiscovery.LatencyMultiplier' must be greater than or equal to 1"};
+    }
 }
 
 void
@@ -390,7 +428,7 @@ LookupI::objectRequestTimedOut(const ObjectRequestPtr& request)
 }
 
 void
-LookupI::adapterRequestException(const AdapterRequestPtr& request, exception_ptr ex)
+LookupI::adapterRequestException(const AdapterRequestPtr& request, exception_ptr ex, size_t generation)
 {
     lock_guard lock(_mutex);
     auto p = _adapterRequests.find(request->getId());
@@ -399,7 +437,7 @@ LookupI::adapterRequestException(const AdapterRequestPtr& request, exception_ptr
         return;
     }
 
-    if (request->exception())
+    if (request->exception(generation))
     {
         if (_warnOnce)
         {
@@ -448,7 +486,7 @@ LookupI::adapterRequestTimedOut(const AdapterRequestPtr& request)
 }
 
 void
-LookupI::objectRequestException(const ObjectRequestPtr& request, exception_ptr ex)
+LookupI::objectRequestException(const ObjectRequestPtr& request, exception_ptr ex, size_t generation)
 {
     lock_guard lock(_mutex);
     auto p = _objectRequests.find(request->getId());
@@ -457,7 +495,7 @@ LookupI::objectRequestException(const ObjectRequestPtr& request, exception_ptr e
         return;
     }
 
-    if (request->exception())
+    if (request->exception(generation))
     {
         if (_warnOnce)
         {

data/dist/ice/cpp/src/IceDiscovery/LookupI.h

@@ -22,7 +22,7 @@ namespace IceDiscovery
 
         virtual bool retry();
         void invoke(const std::string&, const std::vector<std::pair<LookupPrx, LookupReplyPrx>>&);
-        bool exception();
+        bool exception(size_t generation);
         [[nodiscard]] std::string getRequestId() const;
 
         virtual void finished(const std::optional<Ice::ObjectPrx>&) = 0;
@@ -35,6 +35,11 @@ namespace IceDiscovery
         int _retryCount;
         size_t _lookupCount{0};
         size_t _failureCount{0};
+
+        // Incremented on each invoke() (i.e. each retry round). The exception callbacks capture the value current when
+        // they were sent, so a delayed failure from an earlier round can be ignored instead of counting against the
+        // current round.
+        size_t _generation{0};
     };
     using RequestPtr = std::shared_ptr<Request>;
 
@@ -128,9 +133,9 @@ namespace IceDiscovery
         void foundAdapter(const std::string&, const std::string&, const Ice::ObjectPrx&, bool);
 
         void adapterRequestTimedOut(const AdapterRequestPtr&);
-        void adapterRequestException(const AdapterRequestPtr&, std::exception_ptr);
+        void adapterRequestException(const AdapterRequestPtr&, std::exception_ptr, size_t generation);
         void objectRequestTimedOut(const ObjectRequestPtr&);
-        void objectRequestException(const ObjectRequestPtr&, std::exception_ptr);
+        void objectRequestException(const ObjectRequestPtr&, std::exception_ptr, size_t generation);
 
         const IceInternal::TimerPtr& timer() { return _timer; }
 

data/dist/ice/cpp/src/IceDiscovery/generated/Lookup.cpp

@@ -1,6 +1,6 @@
 // Copyright (c) ZeroC, Inc.
 
-// slice2cpp version 3.8.1
+// slice2cpp version 3.8.3
 // <auto-generated>Generated from Slice file 'Lookup.ice'.</auto-generated>
 
 #define ICE_BUILDING_GENERATED_CODE
@@ -30,7 +30,7 @@
 #   if ICE_INT_VERSION % 100 >= 50
 #       error Beta header file detected
 #   endif
-#   if ICE_INT_VERSION % 100 < 1
+#   if ICE_INT_VERSION % 100 < 3
 #       error Ice patch level mismatch!
 #   endif
 #endif

data/dist/ice/cpp/src/IceLocatorDiscovery/PluginI.cpp

@@ -2,6 +2,7 @@
 
 #include "../Ice/Network.h" // For getInterfacesForMulticast
 #include "../Ice/Timer.h"
+#include "Ice/LocalExceptions.h"
 #include "Ice/LoggerUtil.h"
 #include "IceLocatorDiscovery/IceLocatorDiscovery.h"
 #include "IceLocatorDiscovery/Lookup.h"
@@ -76,16 +77,16 @@ namespace
 
         vector<Ice::LocatorPrx> getLocators(const string&, const chrono::milliseconds&);
 
-        void exception(std::exception_ptr);
+        void exception(std::exception_ptr, size_t generation);
 
     private:
         void runTimerTask() final;
 
         LookupPrx _lookup;
         vector<pair<LookupPrx, LookupReplyPrx>> _lookups;
-        chrono::milliseconds _timeout;
-        int _retryCount;
-        chrono::milliseconds _retryDelay;
+        const chrono::milliseconds _timeout;
+        const int _retryCount;
+        const chrono::milliseconds _retryDelay;
         const IceInternal::TimerPtr _timer;
         const int _traceLevel;
 
@@ -99,6 +100,10 @@ namespace
         bool _pending{false};
         int _pendingRetryCount{0};
         size_t _failureCount{0};
+        // Incremented at the start of each lookup round. The findLocatorAsync exception callbacks capture the value
+        // current when they were sent, so a delayed failure from an earlier round is ignored instead of counting
+        // against the current round.
+        size_t _generation{0};
         bool _warnOnce{true};
         vector<RequestPtr> _pendingRequests;
         std::mutex _mutex;
@@ -316,7 +321,7 @@ Request::invoke(const Ice::LocatorPrx& l)
                     }
                     else
                     {
-                        outPair.first = &outParams[0];
+                        outPair.first = outParams.data();
                         outPair.second = outPair.first + outParams.size();
                     }
                     self->response(ok, outPair);
@@ -377,6 +382,11 @@ Request::exception(std::exception_ptr ex)
     catch (...)
     {
         _exception = ex;
+        // This catch-all is reached only from the asynchronous ice_invokeAsync completion (which runs off
+        // LocatorI::_mutex), never synchronously from Request::invoke: the local exceptions ice_invokeAsync can throw
+        // synchronously (caught by Request::invoke's own catch clause and passed to exception() in-line) are all
+        // handled by the specific catch clauses above. That matters because the retry below re-enters
+        // LocatorI::invoke and re-locks the non-recursive LocatorI::_mutex.
         _locator->invoke(_locatorPrx, shared_from_this()); // Retry with new locator proxy
     }
 }
@@ -396,17 +406,26 @@ LocatorI::LocatorI(
       _locator(lookup->ice_getCommunicator()->getDefaultLocator()),
       _voidLocator(std::move(voidLocator))
 {
-    if (_timeout < chrono::milliseconds::zero())
+    if (_timeout <= chrono::milliseconds::zero())
     {
-        _timeout = chrono::milliseconds(300);
+        throw Ice::PropertyException{
+            __FILE__,
+            __LINE__,
+            "property 'IceLocatorDiscovery.Timeout' must be greater than 0"};
     }
     if (_retryCount < 0)
     {
-        _retryCount = 0;
+        throw Ice::PropertyException{
+            __FILE__,
+            __LINE__,
+            "property 'IceLocatorDiscovery.RetryCount' must be greater than or equal to 0"};
     }
     if (_retryDelay < chrono::milliseconds::zero())
     {
-        _retryDelay = chrono::milliseconds::zero();
+        throw Ice::PropertyException{
+            __FILE__,
+            __LINE__,
+            "property 'IceLocatorDiscovery.RetryDelay' must be greater than or equal to 0"};
     }
 }
 
@@ -533,8 +552,8 @@ LocatorI::foundLocator(const Ice::LocatorPrx& locator)
             out << "received Ice locator with different instance name:\n";
             out << "using = '" << _locator->ice_getIdentity().category << "'\n";
             out << "received = '" << locator->ice_getIdentity().category << "'\n";
-            out << "This is typically the case if multiple Ice locators with different";
-            out << "instance names are deployed and the property `IceLocatorDiscovery.InstanceName' ";
+            out << "This is typically the case if multiple Ice locators with different ";
+            out << "instance names are deployed and the property 'IceLocatorDiscovery.InstanceName' ";
             out << "is not set.";
         }
         return;
@@ -624,6 +643,7 @@ LocatorI::invoke(const optional<Ice::LocatorPrx>& locator, const RequestPtr& req
         {
             _pending = true;
             _failureCount = 0;
+            ++_generation;
             _pendingRetryCount = _retryCount;
             try
             {
@@ -642,7 +662,8 @@ LocatorI::invoke(const optional<Ice::LocatorPrx>& locator, const RequestPtr& req
                         _instanceName,
                         l.second,
                         nullptr,
-                        [self = shared_from_this()](exception_ptr ex) { self->exception(ex); });
+                        [self = shared_from_this(), generation = _generation](exception_ptr ex)
+                        { self->exception(ex, generation); });
                 }
                 _timer->schedule(shared_from_this(), _timeout);
             }
@@ -672,9 +693,15 @@ LocatorI::invoke(const optional<Ice::LocatorPrx>& locator, const RequestPtr& req
 }
 
 void
-LocatorI::exception(std::exception_ptr ex)
+LocatorI::exception(std::exception_ptr ex, size_t generation)
 {
     lock_guard lock(_mutex);
+    // Ignore a delayed failure from an earlier lookup round: it must not count against the current round, which reset
+    // _failureCount and may still have outstanding lookups.
+    if (generation != _generation)
+    {
+        return;
+    }
     if (++_failureCount == _lookups.size() && _pending)
     {
         //
@@ -757,13 +784,15 @@ LocatorI::runTimerTask()
                 }
             }
             _failureCount = 0;
+            ++_generation;
             for (const auto& l : _lookups)
             {
                 l.first->findLocatorAsync(
                     _instanceName,
                     l.second,
                     nullptr,
-                    [self = shared_from_this()](exception_ptr ex) { self->exception(ex); });
+                    [self = shared_from_this(), generation = _generation](exception_ptr ex)
+                    { self->exception(ex, generation); });
             }
             _timer->schedule(shared_from_this(), _timeout);
             return;

data/dist/ice/cpp/src/IceLocatorDiscovery/generated/Lookup.cpp

@@ -1,6 +1,6 @@
 // Copyright (c) ZeroC, Inc.
 
-// slice2cpp version 3.8.1
+// slice2cpp version 3.8.3
 // <auto-generated>Generated from Slice file 'Lookup.ice'.</auto-generated>
 
 #define ICE_BUILDING_GENERATED_CODE
@@ -30,7 +30,7 @@
 #   if ICE_INT_VERSION % 100 >= 50
 #       error Beta header file detected
 #   endif
-#   if ICE_INT_VERSION % 100 < 1
+#   if ICE_INT_VERSION % 100 < 3
 #       error Ice patch level mismatch!
 #   endif
 #endif

data/dist/ice/cpp/src/Slice/DocCommentParser.cpp

@@ -60,9 +60,9 @@ DocCommentFormatter::formatCode(const string& rawText)
 }
 
 string
-DocCommentFormatter::formatParamRef(const string& param)
+DocCommentFormatter::formatParamRef(const string& paramName, const ParameterPtr&)
 {
-    return formatCode(param);
+    return formatCode(paramName);
 }
 
 string
@@ -210,25 +210,19 @@ namespace
         {
             // Then we perform additional parsing to extract the name...
 
-            auto nameStart = line.find_first_not_of(ws, tag.size());
+            auto nameStart = doc.find_first_not_of(ws);
             if (nameStart == string::npos)
             {
-                return false; // Malformed line, ignore it.
+                return false; // Malformed line, missing the name part after the tag. Ignore this line.
             }
 
-            auto nameEnd = line.find_first_of(ws, nameStart);
-            if (nameEnd == string::npos)
-            {
-                return false; // Malformed line, ignore it.
-            }
-            name = line.substr(nameStart, nameEnd - nameStart);
+            // If there's no whitespace after the name, that means the name runs to the end of the line.
+            auto nameEnd = doc.find_first_of(ws, nameStart);
+            name = (nameEnd == string::npos) ? doc.substr(nameStart) : doc.substr(nameStart, nameEnd - nameStart);
 
-            // Store whatever remains of the doc comment in the `doc` string.
-            auto docSplitPos = line.find_first_not_of(ws, nameEnd);
-            if (docSplitPos != string::npos)
-            {
-                doc = line.substr(docSplitPos);
-            }
+            // If there's any non-whitespace characters after the name, store them in the `doc` string.
+            auto docStart = doc.find_first_not_of(ws, nameEnd);
+            doc = (docStart == string::npos) ? "" : doc.substr(docStart);
 
             return true;
         }
@@ -291,12 +285,12 @@ namespace
         openingEnd = (openingEnd == string::npos ? line.size() : openingEnd);
         auto count = openingEnd - pos;
 
-        // Find the closing delimeter, starting for the end of the opening delimeter.
+        // Find the closing delimiter, starting for the end of the opening delimiter.
         auto delimiter = string(count, '`');
         auto closingStart = line.find(delimiter, openingEnd);
         if (closingStart == string::npos)
         {
-            // No matching delimeter was found. Emit a warning, then skip to the next line. This one is broken.
+            // No matching delimiter was found. Emit a warning, then skip to the next line. This one is broken.
             const string msg = "unterminated code span: missing closing backtick delimiter";
             p->unit()->warning(p->file(), p->line(), InvalidComment, msg);
 
@@ -325,12 +319,24 @@ namespace
         const string linkTag = "{@link ";
 
         auto endpos = line.find('}', pos);
-        if (endpos != string::npos)
+        if (endpos == string::npos)
+        {
+            // No closing brace was found. Emit a warning, then skip to the next line. This one is broken.
+            const string msg = "unterminated link tag: missing closing '}'";
+            p->unit()->warning(p->file(), p->line(), InvalidComment, msg);
+
+            pos = line.size();
+        }
+        else
         {
-            // Extract the linked-to identifier.
+            // Extract the linked-to identifier, trimming any whitespace around it.
+            string linkText;
             auto identStart = line.find_first_not_of(" \t", pos + linkTag.size());
-            auto identEnd = line.find_last_not_of(" \t", endpos);
-            string linkText = line.substr(identStart, identEnd - identStart);
+            if (identStart < endpos)
+            {
+                auto identEnd = line.find_last_not_of(" \t", endpos - 1);
+                linkText = line.substr(identStart, identEnd - identStart + 1);
+            }
 
             // Then erase the entire '{@link foo}' tag from the comment.
             line.erase(pos, endpos - pos + 1);
@@ -397,19 +403,19 @@ namespace
         // Extract the parameter's name and check if matches an operation parameter.
         // If it does, make sure to use the mapped name instead of the Slice name.
         string parameterName = line.substr(nameStart, nameEnd - nameStart);
+        ParameterPtr parameterPtr = nullptr;
         if (auto operationTarget = dynamic_pointer_cast<Operation>(p))
         {
-            bool doesParameterExist = false;
             for (const auto& param : operationTarget->parameters())
             {
                 if (param->name() == parameterName)
                 {
                     parameterName = param->mappedName();
-                    doesParameterExist = true;
+                    parameterPtr = param;
                     break;
                 }
             }
-            if (!doesParameterExist)
+            if (parameterPtr == nullptr)
             {
                 string opName = operationTarget->name();
                 string msg = "No parameter named '" + parameterName + "' exists on operation '" + opName + "'";
@@ -422,7 +428,7 @@ namespace
         }
 
         // Format the parameter's name and insert it in-place of the original '@p name'.
-        const string formattedParamName = formatter.formatParamRef(parameterName);
+        const string formattedParamName = formatter.formatParamRef(parameterName, parameterPtr);
         line.erase(pos, nameEnd - pos);
         line.insert(pos, formattedParamName);
 

data/dist/ice/cpp/src/Slice/DocCommentParser.h

@@ -27,12 +27,14 @@ namespace Slice
         [[nodiscard]] virtual std::string formatCode(const std::string& rawText);
 
         /// This function is called by the doc-comment parser to map '@p' tags into each language's syntax.
-        /// @param param The mapped name of the parameter that is being referenced.
-        /// @return A properly formatted parameters reference in the target language. The doc-comment parser will
+        /// @param paramName If @p paramPtr is non-null, this is the mapped name of the parameter being referenced.
+        /// Otherwise, this is the identifier after '@p', taken verbatim from the doc-comment.
+        /// @param paramPtr A pointer to the parameter that is being referenced, or `nullptr` if it doesn't exist.
+        /// @return A properly formatted parameter reference in the target language. The doc-comment parser will
         /// replace the entire "@p <rawParamName>" string with the returned value.
         //
         // By default we just emit the parameter's name in code formatting.
-        [[nodiscard]] virtual std::string formatParamRef(const std::string& param);
+        [[nodiscard]] virtual std::string formatParamRef(const std::string& paramName, const ParameterPtr& paramPtr);
 
         /// This function is called by the doc-comment parser to map doc-links ('{@link <rawLink>}') into each
         /// language's syntax.
@@ -46,7 +48,7 @@ namespace Slice
 
         /// This function is called by the doc-comment parser to map see-also references ('@see <rawLink>') into each
         /// language's syntax.
-        /// @param rawLink The references's raw text, taken verbatim from the doc-comment.
+        /// @param rawLink The reference's raw text, taken verbatim from the doc-comment.
         /// @param source A pointer to the Slice element that the doc-comment (and reference) are written on.
         /// @param target A pointer to the Slice element that is being referenced, or `nullptr` if it doesn't exist.
         /// @return A properly formatted see-also-link in the target language. This value will be stored in the

data/dist/ice/cpp/src/Slice/FileTracker.cpp

@@ -3,6 +3,7 @@
 #include "FileTracker.h"
 #include "../Ice/ConsoleUtil.h"
 #include "../Ice/FileUtil.h"
+#include "Util.h"
 
 #include <cassert>
 
@@ -29,20 +30,11 @@ Slice::FileTracker::instance()
 void
 Slice::FileTracker::setSource(const string& source)
 {
-    _source = source;
     pair<map<string, list<string>>::iterator, bool> p = _generated.insert(make_pair(source, list<string>()));
     assert(p.second);
     _curr = p.first;
 }
 
-void
-Slice::FileTracker::error()
-{
-    assert(_curr != _generated.end());
-    _generated.erase(_curr);
-    _curr = _generated.end();
-}
-
 void
 Slice::FileTracker::addFile(const string& file)
 {
@@ -84,10 +76,10 @@ Slice::FileTracker::dumpxml()
     {
         if (!p.second.empty())
         {
-            consoleOut << endl << "  <source name=\"" << p.first << "\">";
+            consoleOut << endl << "  <source name=\"" << escapeXMLAttribute(p.first) << "\">";
             for (const auto& q : p.second)
             {
-                consoleOut << endl << "    <file name=\"" << q << "\"/>";
+                consoleOut << endl << "    <file name=\"" << escapeXMLAttribute(q) << "\"/>";
             }
             consoleOut << endl << "  </source>";
         }

data/dist/ice/cpp/src/Slice/FileTracker.h

@@ -30,13 +30,11 @@ namespace Slice
         void setSource(const std::string& source);
         void addFile(const std::string& file);
         void addDirectory(const std::string& dir);
-        void error();
         void cleanup();
         void dumpxml();
 
     private:
         std::list<std::pair<std::string, bool>> _files;
-        std::string _source;
         std::map<std::string, std::list<std::string>> _generated;
         std::map<std::string, std::list<std::string>>::iterator _curr;
     };