[Xastir] Xastir / Solaris 10 Sparc

[Chris Bell]

> On Mon, 25 Apr 2005, Chris Bell wrote:
> 
> > I am going to see how many places it needs to be changed, and test
> > that while waiting to see if anyone else has any bright ideas...
> 
> Is there a way to do any #define's to fix it globally?
> 

I don't think so... it is too tricky.

> If not that, perhaps an encapsulation function that checks for
> NULL's in the parameters and then passes different parameters off to
> the *printf routines?
> 
> Just trying to think of an easier method than checking through
> thousands of lines of code.
> 

That's what I was thinking too... we already use "xastir_snprintf()"
in many places, so it would be possible to make that into a wrapper
instead of a #define, but that would not fix all the fprintfs and such
in ALL the debug statements, and others.  I think that is the right
angle to head though, there are many places where NULL is used as a
flag to test too, so setting it to "" will break the test... intead of
if(var==NULL) we would have to do something like
if(strlen(var)==0)... ick.  

My quick mods to fix the two places it was crashing for me (about a
dozen changes) has been running all night with no problems.  I am
going to look through again to see any other obvious places that need
to be fixed, and if it breaks anything before I check it in.  

Chris.