[PATCH] smbreadline: switch to new-style readline typedef

[Jeremy Allison]

On Mon, Mar 03, 2014 at 08:51:19PM -0300, Gustavo Zacarias wrote:
> On 03/03/2014 05:57 PM, Jeremy Allison wrote:
> 
> > A question - rl_hook_func_t is defined here:
> > 
> > http://www.delorie.com/gnu/docs/readline/rlman_26.html
> > 
> > as 'typedef int rl_hook_func_t (void);'
> > 
> > However, callback is defined in the Samba code as :
> > void (*callback)(void).
> > 
> > Should we also update the callback() definitions ?
> 
> Incomplete docs :)
> The callback is just a pointer to a function, used like this in line 456
> of readline-6.3/input.c:
> 
> (*rl_event_hook) ();
> 
> Any return value is ignored and unassigned, so it's good as it is and
> may cause compiler warnings if it's not void.

Fair enough - can you resend your patch please ?

Jeremy.