Archived
This forum has been archived. Please start a new discussion on GitHub.
Unecessary __operation in slice2cpp
in Bug Reports
Hi ZeroC,
Minor point: seems like slice2cpp\Gen.cpp line 4154 is redundant. The generated static const std::string __operation isn't used: the global static const that's generated via line 4147 is used instead.
Nothing major clearly...
Minor point: seems like slice2cpp\Gen.cpp line 4154 is redundant. The generated static const std::string __operation isn't used: the global static const that's generated via line 4147 is used instead.
try { static const ::std::string __operation("op1"); /*redundant from 4154*/ __prepare(__prx, AMI__Test1Mdl__TestItf__op1_name /*defined on 4147*/, ::Ice::Normal, __ctx); //... __os->endWriteEncaps(); }
Nothing major clearly...
0
Comments
-
Hi,
The extra line has already been removed in our mainline for our upcoming 3.1.0 release of Ice. We were hoping noone would have noticed it
Regards,
Dwayne0 -
Hi Dwayne,
Cool. I wouldn't have looked but was hunting for a way to do true async calls by wrapping __invoke in a functor...
Cheers0