Re: [xml] cleanup on randomization - inconsistent in master



On Sun, May 20, 2012 at 11:28:05AM +0300, Roumen Petrov wrote:
Oops,

Roumen Petrov wrote:
Hi All,

One of recent updates swicth from rand to rand_r but it seems to
me switch is incomplete:
- dict.c - #ifdef HAVE_RAND_R
- libxml.h -   +#if defined(HAVE_RAND)
- configure check for rand function but not for rand_r
Please to update configure script and libxml.h header to be consistent.

Post 2.8.0 could implement fail back solution to rand as not all
platforms provide rand_r.
Uh it seems to me I check only commit, and miss that rand_r is used
under condition.

So only check for rand_r is missing in configure script.

  Right I forgot to push that chunk because I have another change
there to bump to 2.8.0 ... oops, I'm fixing that :-)

   thanks !

Daniel

-- 
Daniel Veillard      | libxml Gnome XML XSLT toolkit  http://xmlsoft.org/
daniel veillard com  | Rpmfind RPM search engine http://rpmfind.net/
http://veillard.com/ | virtualization library  http://libvirt.org/



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