[Asterisk-video] Question to H223MuxSDU::Pop()

[Sergio Garcia Murillo]

Yep, mee too, but I don't know why I was having problems compiling with
assert (don't remmember why), and as I said, I was in paranoid mode.. :)

BR
Sergio


On Mon, 2008-02-25 at 21:19 +0100, Michael (qq12345) wrote:
> Hi,
> 
> in that case, I prefer using
> 
> #include <assert.h>
> 
> assert( false ); 
> 
> or 
> 
> assert( ini > end ) ;
> 
> Nevertheless I try to file a bug.
> 
> Michael
> 
> > -----Original Message-----
> > From: asterisk-video-bounces at lists.digium.com 
> > [mailto:asterisk-video-bounces at lists.digium.com] On Behalf Of 
> > Sergio Garcia Murillo
> > Sent: Monday, February 25, 2008 8:21 PM
> > To: Development discussion of video media support in Asterisk
> > Subject: Re: [Asterisk-video] Question to H223MuxSDU::Pop()
> > 
> > 
> > On Mon, 2008-02-25 at 18:07 +0100, Michael (qq12345) wrote:
> > > Hi Sergio,
> > > 
> > > in the method below, the construct with the 0-Pointer, I have never 
> > > seen and it looks strange for me.
> > > Is that intended? Would you like to shed some light on this?
> > > 
> > > BYTE H223MuxSDU::Pop()
> > > {
> > > 	if(ini>=end)
> > > 		*((BYTE*)0)=0;
> > > 	//Exit
> > > 	return buffer[ini++];
> > > }
> > > 
> > 
> > Do you know a fastest way to get a core dump?? ;)
> > Should be an assertion check that I put in a paranoid day just to make
> > sure I was not getting memmory overwritten. As the condition is never
> > meet it would never cause a problem. I'll remove it anyway.
> > 
> > Could you put it on the tracker just to make sure I don't 
> > fotget to fix
> > it?
> > 
> > Best regards and thank you
> > Sergio
> > 
> > 
> > 
> > _______________________________________________
> > --Bandwidth and Colocation Provided by http://www.api-digital.com--
> > 
> > asterisk-video mailing list
> > To UNSUBSCRIBE or update options visit:
> >    http://lists.digium.com/mailman/listinfo/asterisk-video
> > 
> 
> 
> _______________________________________________
> --Bandwidth and Colocation Provided by http://www.api-digital.com--
> 
> asterisk-video mailing list
> To UNSUBSCRIBE or update options visit:
>    http://lists.digium.com/mailman/listinfo/asterisk-video