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

[Kevin P. Fleming]

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
don't think that trying to find some way to 'safely' cast things back
and forth is the right course of action, it would be better to try to
find a way to avoid needing them at all.