[asterisk-dev] Re: [asterisk-commits] rizzo: trunk r47652 - in /trunk: include/asterisk/cli.h main/cli.c

[Luigi Rizzo]

On Wed, Nov 15, 2006 at 02:07:06PM -0600, Kevin P. Fleming wrote:
> asterisk-commits at lists.digium.com wrote:
> > Author: rizzo
> > Date: Wed Nov 15 08:11:28 2006
> > New Revision: 47652
> > 
> > URL: http://svn.digium.com/view/asterisk?view=rev&rev=47652
> > Log:
> > update the internal cli api following comments from kevin.
> > This change basically simplifies the interface of the
> > new-style handler removing almost all the tricks used in
> > the previous implementation to achieve backward compatibility
> > (which is still present and guaranteed.)
> 
> There is still a problem here; you are casting between 'int' and 'char
> *', and on a 64-bit platform these types are not the same size. This is
> why I was concerned about the use of these casts in the first place... I

yes this was also my concern, but the only use for this now is to map
the 3 return values from the new-style handlers back to the old-style,
and this is going away tomorrow (one place and 3 definitions).

please have a bit of patience :)

	cheers
	luigi