Archived

This forum has been archived. Please start a new discussion on GitHub.

Unecessary __operation in slice2cpp

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.
    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...

Comments

  • dwayne
    dwayne St. John's, Newfoundland
    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,
    Dwayne
  • 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...

    Cheers