Re: [Ekiga-devel-list] Win32 Opal datasize, please help

[Damien Sandras <dsandras seconix com>]

Le mardi 14 juillet 2009 à 18:06 +0200, Michael Rickmann a écrit :
> Damien Sandras schrieb:
> > Le lundi 13 juillet 2009 à 23:32 +0200, Michael Rickmann a écrit :
> >> Damien Sandras schrieb:
> >>> Le dimanche 12 juillet 2009 à 22:40 +0200, Michael Rickmann a écrit :
> >>>> I was stuck testing the latest changes to Ekiga since about 10 days. 
> >>>> Once a call to 500 ekiga net had been established windows were piling up 
> >>>>   and rendered WinXP and Win7 unusable. Today I found the reason: commit 
> >>>> 23036 to opal trunk which coresponds to  23028 opal v3_6 and is 
> >>>> contained in the archives needed by Ekiga 3.2.5.
> >>>> Attached patch fixes this for ekiga head - it just reverses the opal 
> >>>> commit - no final solution. I am really no opal expert and a bit lost 
> >>>> here. Why has only Win32 been hit here?
> >>>> Good news is that "Split ekiga into an exec and helper libs" - it 
> >>>> happend at about the same time as the opal change - seems to work for Win32.
> >>> Would you have a -d 4 output of the problem when it occurs without your
> >>> patch?
> >>>
> >>> Robert will examine it.
> >> Thanks a lot. I produced the -d 4 logs with my Win32 version of current 
> >> Ekiga stable, i.e. Ekiga 3.2.5, Opal 3.6.4 and Ptlib 2.6.4. So commit 
> >> 23028 to opal v3_6 seems to cause the error. The logs are rather long, 
> >> you find them at 
> >> http://wwwuser.gwdg.de/~mrickma/ekiga/data-size-logs.tar.gz. 
> >> 500 ekiganet-ekiga-killed-stderr txt is when calling 500 ekiga net  As 
> >> soon as the call was accepted empty windows were piling up and I had to 
> >> kill Ekiga. fritzbox-nosound-ekiga-stderr.txt is when calling my 
> >> audio-only SIP-phone-box at home. Ekiga told me that it was unable to 
> >> open the audio driver, exit gracefully. The programs which I used can be 
> >> found at http://wwwuser.gwdg.de/~mrickma :
> >> buggy-ekiga-setup-3.2.5-release.exe fails, ekiga-setup-3.2.5-release.exe 
> >> is ok with patch reversed, ekiga_build-3.2.5.tgz documents how I built 
> >> the stuff.
> > 
> > Can you try with the following patch ?
> > The error will stay, but we will have more information and you won't
> > have windows popping up again and again.
> > 
> > You will also have an indication of what the problem can be.
> > 
> > Robert thinks the problem could be in Ekiga, but due to the API change.
> > 
> > 
> I applied the patch, the Windows went and Win32 Ekiga is better than it 
> ever was. The large audio delay (> 1.5 sec) is nearly gone, must have 
> been the changes to Opal. However, as you said, the error stayed - 
> somehow I like it, audio is next to perfect. In the logs we get now 
> lines saying
> Media Patch:3096	GMAudioInputManager_ptlib	Encountered error while 
> trying to read data, read 160 instead of 320
> The complete -d 4 log you find at 
> http://wwwuser.gwdg.de/~mrickma/ekiga/ekiga-stderr.txt.gz .
> Attached backtrace shows what happens when calling 500 ekiga net and 
> where the request for 320 comes from.
> Thanks for the help so far.
> Michael

The purpose of that fix (from Robert) was to improve the audio quality.

Now, even if it works, we need to determine where those 320 and 160
values come from and which one is correct (I suppose 160 is correct as
it works).
-- 
 _     Damien Sandras
(o-      
//\    Ekiga Softphone : http://www.ekiga.org/
v_/_   Be IP           : http://www.beip.be/
       FOSDEM          : http://www.fosdem.org/
       SIP Phone       : sip:dsandras ekiga net