Re: [PATCH] (u)int64 + string value transforms



On Wed, 6 Feb 2002, Andy Wingo wrote:

> Hi. This patch replaces my previous one.
> 
> * add (u)int64 to the type conversion routines in gvaluetransform.c
> * add value_transform_string_* routines
> 
> All of this has been tested in GStreamer in the XML serialization and
> restoration routines, but only in my local copy.
> 
> Please take a look at this at your earliest convenience.

the int64 part looks ok, but we're not going to support string->integer/float
conversions.
there's a lengthy thread on the reasons for that in the list archives, but
basically, it's that we're not going to force a specific syntax on people,
and that we're not able to handle derived types correctly. so please back out
that part and resend the patch (sending it to me privately on the second round
will suffice to get commit aproval).

as an aside, the ->string conversions shouldn't really be used in production
code, they are more of a convenience variant for debugging purposes and could
possibly change.

> 
> Best regards,
> 
> Andy Wingo

---
ciaoTJ




[Date Prev][Date Next]   [Thread Prev][Thread Next]   [Thread Index] [Date Index] [Author Index]