Home Bug Reports

Unecessary __operation in slice2cpp

joejoe Member
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

  • dwaynedwayne St. John's, NewfoundlandMember Dwayne BooneOrganization: ZeroC, Inc.Project: Internet Communications Engine
    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
  • joejoe Member
    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
Sign In or Register to comment.