Driving Mode Crashes CSipSimple

[GoogleCodeExporter]

What steps will reproduce the problem?
1. Turn on CSipSimple
2. Turn on TMobile Slide Driving Mode
3. Receive call

What is the expected output? What do you see instead?
Driving mode on the TMobile Slide turns off the ringtone and instead announces 
the name of the caller. When CSipSimple is running, it causes CSipSimple to 
crash on incoming call.

What version of the product are you using? On what operating system?
0.00-12-09. TMobile Slide 3G running Android 2.1-update1 with the latest 
firmware pushed out for the Slide.

Please provide any additional information below.

Attached is a logcat of CSipSimple crashing, as well as a call coming in 
without CSipSimple running.

Original issue reported on code.google.com by robert.b...@gmail.com on 10 Aug 2010 at 2:11

Attachments:

• csipsimple.txt
• normal.txt

[GoogleCodeExporter]

Regarding logs it seems to be linked to the fact ringtone is not found. 

Maybe in driving mode default ringtone is set to null, and so when csipsimple 
try to retrieve the default ringtone we get this null pointer exception.

I'll try to solve the null pointer exception in a first step. 
For announcing the caller name, I hope the fact that I publish the officially 
formatted intent about the fact there is an incoming call will be enough to 
make the Slide Driving Mode app announcing the caller name.

Original comment by r3gis...@gmail.com on 10 Aug 2010 at 9:32

• Changed state: Accepted

[GoogleCodeExporter]

0.00-12-10 includes a fix that should avoid the crash (the null pointer 
exception observed in the log).
Let me know if :
1- it doesn't crash anymore
2- the TMobile Slide Driving Mode app announce the caller name

Original comment by r3gis...@gmail.com on 10 Aug 2010 at 9:54

• Changed state: Started

[GoogleCodeExporter]

1. It did NOT crash, thank you!
2. The phone is silent until the regular phone picks up the call -- Driving 
Mode does not announce anything for the VOIP call.

Original comment by robert.b...@gmail.com on 10 Aug 2010 at 10:11

[GoogleCodeExporter]

> 2. Driving Mode does not announce anything for the VOIP call.

It would have been too simple :). Don't know if Driving Mode for the Slide 
source code is available.If so, I can maybe find out the correct intent to send 
in order to ask driving mode to announce the user name.

What I can only do for now is considering that Driving mode app for the Slide 
just break things for other apps that want to ring. (There is probably other 
app in this case that just remove phone ringing settings - for example sleeping 
app). 
Maybe, in the future each of this app can be supported case by case.

As for now, I think that I'll not spent a lot of time on these specials apps 
that break the "normal" phone behavior. 

-> But to make thing better for you, should it be acceptable if, while in 
driving mode, CSipSimple fallback to vibrator instead of ringing? (...If 
driving mode app doesn't break the vibrator feature too - I hope so).

Just by curiosity about this Driving mode application : if the user that call 
you has a custom ringtone, does it rings with custom ringtone or announce the 
caller id (or both?).

P.S. : for reference, the Nexus One driving mode doesn't break the ringtones (- 
probably because of the fact they do not announce the caller by text to speech 
and so doesn't need to prevent the a normal app to get back the default 
ringtone -)

Original comment by r3gis...@gmail.com on 10 Aug 2010 at 10:43

[GoogleCodeExporter]

Even with custom ringtones, the Driving Mode just announces the names. I'm 
pretty sure that I'm in the minority of people using CSipSimple on the Slide at 
this point, so I definitely understand this issue falling on the back burner. 
I'm just happy that it isn't crashing now.

If I set the phone to vibrate, it does vibrate when CSipSimple receives a call, 
as well as announcing the call when the phone actually picks it up. This is 
enough for me to get by.

Original comment by robert.b...@gmail.com on 10 Aug 2010 at 11:03

[GoogleCodeExporter]

Original comment by r3gis...@gmail.com on 13 Oct 2010 at 5:18

• Changed state: Fixed