Re: CF: Why is this line in readable.c, line 1357?

["Mark Wedel" <mark@icp.siemens.com>]

On Jan 19, 10:03pm, James Cameron wrote:
> Subject: Re: CF: Why is this line in readable.c, line 1357?
> Peter Mardahl wrote:
> > while (al == NULL);
>
> Code in context, 0.93.7 ...

 Its actually the same in 0.95.1

 However, there should probably be a retry count added, so we only do that loop
so many times before giving up.


>
>   /* lets determine what kind of artifact type randomly.
>    * Right now legal artifacts only come from those listed
>    * in art_name_array. Also, we check to be sure an artifactlist
>    * for that type exists!
>    */
>   do {
>      index = RANDOM()%(sizeof(art_name_array)/sizeof(arttypename));
>      type=art_name_array[index].type;
>      al = find_artifactlist(type);
>   } while(al==NULL);
>
> --
> James Cameron                                      (cameron@stl.dec.com)
>
> OpenVMS, Linux, Firewalls, Software Engineering, CGI, HTTP, X, C, FORTH,
> COBOL, BASIC, DCL, csh, bash, ksh, sh, Electronics, Microcontrollers,
> Disability Engineering, Netrek, Bicycles, Pedant, Farming, Home Control,
> Remote Area Power, Greek Scholar, Tenor Vocalist, Church Sound, Husband.
>
> "Specialisation is for insects." -- Robert Heinlein.
> -
> [you can put yourself on the announcement list only or unsubscribe altogether
> by sending an email stating your wishes to crossfire-request@ifi.uio.no]
>-- End of excerpt from James Cameron



-- 

-- Mark Wedel
mark@pyramid.com
-
[you can put yourself on the announcement list only or unsubscribe altogether
by sending an email stating your wishes to crossfire-request@ifi.uio.no]