Naming convention for function that leaves 2 or more objects on the cleanup stack

[payu.sergey]

Hi!

Does anybody know the naming rule for the function that leaves 2 or more objects on the cleanup stack?

For example, I need a function that will create 2 HBufC objects and leaves them on the cleanup stack. Would it be right to name it like this:

void GiveSomeStuffLC2(HBufC*& aBuf1, HBufC*& aBuf2)
{
aBuf1 = HBufC::NewLC(0);
aBuf2 = HBUfC::NewLC(0);
}

[ltomuta]

An allocation or construction function which places data on the cleanup stack ends with ...LC()

So it is not about how much data you leave on the stack but the fact that you are leaving the data there. However you need to make sure to document the method properly...

[payu.sergey]

But Ii think it can be little confusing, if I will place this into library and end the name with single 'LC'.

Thanks anyway!

[mubx2000]

The Best Way:

void GiveSomeStuffLC2(TDesC& aBuf1, TDesC& aBuf2)
{
HBufC* Buf1 = HBufC::NewL(aBuf1.Length());
HBufC* Buf2 = HBUfC::NewL(aBuf1.Length());
//--------------------->
// Do what ever her
//---------------------->
delete Buf1;
delete Buf2;

}

Good lack.

[payu.sergey]

Hi mubx2000,

The question was about naming convention, not about the way of code organizing. I'm sure to use the function that leaves 2 objects on the cleanup stack.

Thanks,
Sergey

[wizard_hu_]

I'm sure to use the function that leaves 2 objects on the cleanup stack.

Although having CBlah *&aBlah parameters is perfectly valid, C++ methods have one return value in general.
Anyway, the Symbian source uses LCC (or even LCCC) in this case.

[wizard_hu_]

The Best Way:

void GiveSomeStuffLC2(TDesC& aBuf1, TDesC& aBuf2)
{
HBufC* Buf1 = HBufC::NewL(aBuf1.Length());
HBufC* Buf2 = HBUfC::NewL(aBuf1.Length());
//--------------------->
// Do what ever her
//---------------------->
delete Buf1;
delete Buf2;

}

Good lack.

Yeah, this code really lacks of good parts. Possible leak of Buf1 and improper naming - not a single object pushed on the Cleanup Stack.