On 8/18/21 3:13 PM, Iain Sandoe wrote:
Hi,I have found it useful when working with modules stuff to have the completed set of command/responses available (some people working with the interfaces for more sophisticated tools are using them). This message is a hand-shake telling the server that a CMI has been built, and for the simplistic server implementation doesn’t need to do anything. This just replies with "OK”. Tested on x86_64-darwin. OK for master? thanks Iain c++tools/ChangeLog: * resolver.cc (module_resolver::ModuleCompiledRequest): Add a simple handler. * resolver.h: Declare handler for ModuleCompiledRequest. --- c++tools/resolver.cc | 7 +++++++ c++tools/resolver.h | 4 ++++ 2 files changed, 11 insertions(+) diff --git a/c++tools/resolver.cc b/c++tools/resolver.cc index edd4624b121..f862161095d 100644 --- a/c++tools/resolver.cc +++ b/c++tools/resolver.cc @@ -307,3 +307,10 @@ module_resolver::IncludeTranslateRequest (Cody::Server *s, Cody::Flags, return 0; }
Could use a comment here. OK with that added.
+int +module_resolver::ModuleCompiledRequest (Cody::Server *s, Cody::Flags, + std::string &) +{ + s->OKResponse(); + return 0; +} diff --git a/c++tools/resolver.h b/c++tools/resolver.h index b2f4381b4fa..c1ce9564e7f 100644 --- a/c++tools/resolver.h +++ b/c++tools/resolver.h @@ -96,6 +96,10 @@ public: std::string &include) override;+ using parent::ModuleCompiledRequest;+ virtual int ModuleCompiledRequest (Cody::Server *s, Cody::Flags Flags, + std::string &Module) override; + private: using parent::GetCMISuffix; virtual char const *GetCMISuffix () override;
