[asterisk-dev] [svn-commits] rizzo: trunk r89426 - in /trunk: include/ include/asterisk/ utils/

[Olle E Johansson]

19 nov 2007 kl. 21.01 skrev Luigi Rizzo:

> On Mon, Nov 19, 2007 at 08:47:09PM +0100, Johansson Olle E wrote:
>>
>> 19 nov 2007 kl. 20.36 skrev SVN commits to the Digium repositories:
>>
>>> Author: rizzo
>>> Date: Mon Nov 19 13:36:32 2007
>>> New Revision: 89426
>>>
>>> URL: http://svn.digium.com/view/asterisk?view=rev&rev=89426
>>> Log:
>>> move the declaration of struct ast_channel ast_frame and ast_module
>>> to compat.h so it is always available - hopefully this will let
>>> us reduce the number of inclusions of channel.h and frame.h
>>>
>> Well, at some point you start removing stuff to optimize so hard so
>> you'll end
>> up obfuscating instead. Not having ast_channel declared in channel.h
>> doesn't make
>> any sense at all to me, just adding to the confusion.
>
> i didn't even touch channel.h !
>
> in any case channel.h contains the full definition of the stucture,  
> i.e.
>
> 	struct ast_channel { ... a ton of fields ... };
>
> what i moved (from asterisk.h to compat.h) is only the forward  
> declaration,
>
> 	struct ast_channel;	/* opaque type at this point */

Ahh, sorry. Overreacted. Spent too much time in the bug tracker.
My apologies.

/O :-)