[sync] SyncML Patch and Seg Fault

[Chuck Hagenbuch]

Quoting Anthony Mills <amills at gascard.net>:

> Here is a patch that gets syncml working with WBXML.  The code calls syncml
> rpc like it should.  One problem on line 434 of Encoder.php:  it seg faults.
> You know you did good when you can make a scripting language seg fault. :D  If
> we can fix this problem, then we should be done with WBXML, and can start on
> the guts of SyncML.  I suspect the problem lies with the "gvar" stream.  I
> can't imagine someone not finding this problem with the other streams.   I'm
> going to try this on another machine, and see if that helps, and I will try to
> fix on Wednesday, unless some else does it for me (nudge, nudge).

Can you provide a bit of explanation about the pieces of this patch and what
they do? Some of them look weird to me; some of them just add trailing
whitespace to the files, and some of them seem to just change private variable
names ($_var) to public ($var) (no, that's not enforced, but it's a visual
convention to let the programmer know the intention of the variable). So I'd
rather not commit this wholesale...

-chuck

--
Charles Hagenbuch, <chuck at horde.org>
"I'm really... I'm not too fascinated by green food." - Average Joe