Find Boxees endless loop about boxeeremote

[GoogleCodeExporter]

I'm looking into this and there are indications that this is a null pointer 
exception. 

I was in the process of tracking the issue down when life interrupted me. I 
will post 
back with more details and perhaps a patch if I can get that far.

Original comment by [email protected] on 1 Apr 2009 at 5:31

[GoogleCodeExporter]

Oh, wow, I didn't think you were going to try to fix it! I'll try to take a 
look at
this later, but I think you're right that I send null if the receive times out 
and
then don't check for null in BoxeeRemote.addAnnouncedServers.

Original comment by [email protected] on 1 Apr 2009 at 5:39

[GoogleCodeExporter]

Very cool. I will work with you on this issue. I find this application useful 
and I 
am interested in seeing it mature.

I have the latest revision (R58 at the moment) running in the emulator so if I 
get 
anywhere with a fix I will surely notify you. I also have some other issues in 
the 
application which I want to report and work on.

How would you like to receive code submissions? Should I email a patch, attach 
it as 
a file to this Issue, or what?

Original comment by [email protected] on 1 Apr 2009 at 6:21

[GoogleCodeExporter]

It would be great if you can upload it to
http://codereview.appspot.com/, as that makes it easy for me to read
and comment on. You can get a script from
http://code.google.com/p/rietveld/wiki/CodeReviewHelp that uploads a
diff from the svn tree automatically. If that's too much trouble, I
don't mind getting short diffs as a patch.

Original comment by [email protected] on 1 Apr 2009 at 6:34

[GoogleCodeExporter]

It was a small patch, so I sent you an email at 
andrew-[at]-andrewchatham-[d0t]-com 
with the patch as an attachment. The subject line is "BoxeeRemote Patch (Issue 
4)"

It is currently running on my phone and doesn't seem to break anything.

Original comment by [email protected] on 2 Apr 2009 at 3:16

[GoogleCodeExporter]

Original comment by [email protected] on 3 Apr 2009 at 3:01

• Changed state: Fixed