[rrd-developers] [PATCH] use rrd_random() for PRNG

[Sebastian Harl]

Hi Kevin,

On Sun, Apr 12, 2009 at 03:57:44AM -0500, kevin brintnall wrote:
> diff --git a/src/librrd.sym.in.in b/src/librrd.sym.in.in
> index b4e8484..0a7a699 100644
> --- a/src/librrd.sym.in.in
> +++ b/src/librrd.sym.in.in
> @@ -32,6 +32,7 @@ rrd_new_context
>  rrd_open
>  rrd_parsetime
>  rrd_proc_start_end
> +rrd_random
>  rrd_read
>  rrd_resize
>  rrd_restore

> diff --git a/src/rrd_tool.h b/src/rrd_tool.h
> index f58cd78..ddcd2a9 100644
> --- a/src/rrd_tool.h
> +++ b/src/rrd_tool.h
> @@ -119,6 +119,8 @@ int rrd_fetch_fn_libdbi(char *filename, enum cf_en cf_idx,
>      char *a,
>      char *b);
>  
> +    long rrd_random(void);
> +
>  #endif /* _RRD_TOOL_H */
>  
>  #ifdef  __cplusplus

Why did you include rrd_random in librrd.sym (which tells the compiler
to export that symbol) but then added the definition to rrd_tool.h which
is supposed to be a "private" header?

Cheers,
Sebastian

-- 
Sebastian "tokkee" Harl +++ GnuPG-ID: 0x8501C7FC +++ http://tokkee.org/

Those who would give up Essential Liberty to purchase a little Temporary
Safety, deserve neither Liberty nor Safety.         -- Benjamin Franklin

-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 197 bytes
Desc: Digital signature
Url : http://lists.oetiker.ch/pipermail/rrd-developers/attachments/20090413/c426d751/attachment.bin