Re: [libvirt] [PATCH v3.1] json: fix interface locale dependency

2012-08-13 Thread Eric Blake
On 08/12/2012 01:55 AM, Martin Kletzander wrote: libvirt creates invalid commands if wrong locale is selected. For example with locale that uses comma as a decimal point, JSON commands created with decimal numbers are invalid because comma separates the entries in JSON. Fortunately even when

Re: [libvirt] [PATCH v3.1] json: fix interface locale dependency

2012-08-13 Thread Martin Kletzander
On 08/13/2012 05:51 PM, Eric Blake wrote: Of the two alternatives, I prefer this one for now. If we ever have need for further enhancements down the road, we can add them at that time. No need to add code that will be unused by being more flexible than our current needs. I see I've sent

[libvirt] [PATCH v3.1] json: fix interface locale dependency

2012-08-12 Thread Martin Kletzander
libvirt creates invalid commands if wrong locale is selected. For example with locale that uses comma as a decimal point, JSON commands created with decimal numbers are invalid because comma separates the entries in JSON. Fortunately even when decimal point is affected, thousands grouping is not,