Re: [Flightgear-devel] [PATCH] Fix fgviewer segfault

2010-05-24 Thread Martin Spott
Jeff Taylor wrote: > Is Terragear still being maintained? The current version doesn't even > compile without errors. I have a couple of fixes there. I'll just > attach them to this message for now. In the future, should I make > separate e-mails for each? I will likely keep posting patches as

Re: [Flightgear-devel] [PATCH] Fix fgviewer segfault

2010-05-23 Thread Jeff Taylor
On Sun, May 23, 2010 at 3:20 PM, Martin Spott wrote: > Gijs de Rooy wrote: > >> If you feel happy creating patches for fgviewer, here's another issue that >> really >> should be looked at IMO... >> >> http://code.google.com/p/flightgear-bugs/issues/detail?id=117 > I'm working on something... My

Re: [Flightgear-devel] [PATCH] Fix fgviewer segfault

2010-05-23 Thread Martin Spott
Gijs de Rooy wrote: > If you feel happy creating patches for fgviewer, here's another issue that > really > should be looked at IMO... > > http://code.google.com/p/flightgear-bugs/issues/detail?id=117 Please find the corresponding thread, starting here: http://www.mail-archive.com/flightgea

Re: [Flightgear-devel] [PATCH] Fix fgviewer segfault

2010-05-23 Thread Gijs de Rooy
> Jeff wrote: > fgviewer crashes when reading channel options (which it doesn't use). > Here is a patch which fixes this by adding a pointer check. If you feel happy creating patches for fgviewer, here's another issue that really should be looked at IMO... http://code.google.com/p/flightgear-bu

Re: [Flightgear-devel] [PATCH] Fix fgviewer segfault

2010-05-23 Thread Tim Moore
On Sat, May 22, 2010 at 10:30 PM, Jeff Taylor wrote: > Hello, > > fgviewer crashes when reading channel options (which it doesn't use). > Here is a patch which fixes this by adding a pointer check. > > This is my first ever patch to any open source project, so feedback > would be appreciated. :) >

[Flightgear-devel] [PATCH] Fix fgviewer segfault

2010-05-22 Thread Jeff Taylor
Hello, fgviewer crashes when reading channel options (which it doesn't use). Here is a patch which fixes this by adding a pointer check. This is my first ever patch to any open source project, so feedback would be appreciated. :) Jeff Taylor --- src/Main/options.cxx |8 1 files ch