At 10:01 PM 4/29/2001 +0200, Jani Taskinen wrote:
>On Sun, 29 Apr 2001, Andi Gutmans wrote:
>
> >At 09:42 PM 4/29/2001 +0200, Jani Taskinen wrote:
> >
> >>This is the line in question:
> >>
> >>php_srand(time(0) * getpid() * (php_combined_lcg() * 10000.0));
> >>
> >>This is now (in my patch) in RINIT and thus it is a different pid
> >>it doesn't matter if the RINIT happens the same second..
> >>Or did I misunderstood something? And I know it works now. :)
> >>And it didn't work before. As the PID was always the same like Andi said.
> >>
> >>I attached the patch in case you missed it before..
> >
> >It's OK but in multi-threaded mode it should use "(long) tsrm_thread_id()"
> >because getpid() will usually be the same. I hope the case is good enough
> >and will work in all multi-threaded environments.
>
>hmm..I wonder. That php_srand() line is the same it was before my patch..
>It has had that getpid() in it for a long time. How could it have worked
>on multi-threaded environments?? Something else is different on those?
No reason for it not to have worked. If two threads ran at the same time
they would each reinitialize the seed. It's not really a problem. Actually
come to think of it we don't need to fix threaded environments :)
Andi
--
PHP Development Mailing List <http://www.php.net/>
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]
To contact the list administrators, e-mail: [EMAIL PROTECTED]
