WARNING - OLD ARCHIVES

This is an archived copy of the Xen.org mailing list, which we have preserved to ensure that existing links to archives are not broken. The live archive, which contains the latest emails, can be found at http://lists.xen.org/
   
 
 
Xen 
 
Home Products Support Community News
 
   
 

xen-api

Re: [Xen-API] [PATCH] Add function to Date module to assert ISO 8601 dat

To: Dave Scott <Dave.Scott@xxxxxxxxxxxxx>
Subject: Re: [Xen-API] [PATCH] Add function to Date module to assert ISO 8601 datetime values are UTC
From: Anil Madhavapeddy <anil@xxxxxxxxxx>
Date: Wed, 1 Sep 2010 17:25:12 +0100
Cc: Rob Hoes <Rob.Hoes@xxxxxxxxxx>, xen-api <xen-api@xxxxxxxxxxxxxxxxxxx>
Delivery-date: Wed, 01 Sep 2010 09:25:23 -0700
Envelope-to: www-data@xxxxxxxxxxxxxxxxxxx
In-reply-to: <81A73678E76EA642801C8F2E4823AD219330C427FF@xxxxxxxxxxxxxxxxxxxxxxxxx>
List-help: <mailto:xen-api-request@lists.xensource.com?subject=help>
List-id: Discussion of API issues surrounding Xen <xen-api.lists.xensource.com>
List-post: <mailto:xen-api@lists.xensource.com>
List-subscribe: <http://lists.xensource.com/mailman/listinfo/xen-api>, <mailto:xen-api-request@lists.xensource.com?subject=subscribe>
List-unsubscribe: <http://lists.xensource.com/mailman/listinfo/xen-api>, <mailto:xen-api-request@lists.xensource.com?subject=unsubscribe>
References: <E1OqmkK-00080d-0M@xxxxxxxxxxxxxxxxxxxxxxxx> <81A73678E76EA642801C8F2E4823AD219330C427FF@xxxxxxxxxxxxxxxxxxxxxxxxx>
Sender: xen-api-bounces@xxxxxxxxxxxxxxxxxxx
Bits of it aren't threadsafe (although this use might be), but in general it 
definitely isnt.  You could replace this use with a Scanf.sscanf easily enough 
I think?

-anil

On 1 Sep 2010, at 17:22, Dave Scott wrote:

> Hi Rob,
> 
> Is the Str module threadsafe? ISTR there was something dodgy about it in a 
> threaded environment.
> 
>> +let assert_utc x =
>> +    let r = Str.regexp "^[0-9][0-9][0-9][0-9][0-9][0-9][0-9][0-9]T[0-
>> 9][0-9]:[0-9][0-9]:[0-9][0-9]Z$" in
>> +    if not (Str.string_match r x 0) then
>> +            invalid_arg x
>> +
> 
> 
> Cheers,
> Dave
> 
> _______________________________________________
> xen-api mailing list
> xen-api@xxxxxxxxxxxxxxxxxxx
> http://lists.xensource.com/mailman/listinfo/xen-api
> 


_______________________________________________
xen-api mailing list
xen-api@xxxxxxxxxxxxxxxxxxx
http://lists.xensource.com/mailman/listinfo/xen-api