[asterisk-dev] Re: [asterisk-commits] russell: trunk r41278 - in /trunk: apps/channels/ include/asterisk/ main/

[Paul Cadach]

Hello,

----- Original Message -----
From: <asterisk-commits at lists.digium.com>
To: <asterisk-commits at lists.digium.com>; <svn-commits at lists.digium.com>
Sent: Wednesday, August 30, 2006 2:50 AM
Subject: [asterisk-commits] russell: trunk r41278 - in /trunk: apps/channels/ include/asterisk/ main/


> Author: russell
> Date: Tue Aug 29 15:50:36 2006
> New Revision: 41278
>
> URL: http://svn.digium.com/view/asterisk?rev=41278&view=rev
> Log:
> Merge team/russell/frame_caching

[skipped]

> Modified: trunk/main/channel.c
> URL: http://svn.digium.com/view/asterisk/trunk/main/channel.c?rev=41278&r1=41277&r2=41278&view=diff
> ==============================================================================
> --- trunk/main/channel.c (original)
> +++ trunk/main/channel.c Tue Aug 29 15:50:36 2006

[skipped]

> @@ -1337,8 +1328,6 @@
>   trans = (dir == SPY_READ) ? &chan->spies->read_translator : &chan->spies->write_translator;
>
>   AST_LIST_TRAVERSE(&chan->spies->list, spy, list) {
> - struct ast_frame *last;
> - struct ast_frame *f1; /* the frame to append */
>   struct ast_channel_spy_queue *queue;
>
>   ast_mutex_lock(&spy->lock);
> @@ -1370,7 +1359,7 @@
>   break;
>   }
>   }
> - f1 = translated_frame;
> + AST_LIST_INSERT_TAIL(&queue->list, ast_frdup(f), frame_list);

Originally we have queued copy of translated_frame, currently we adds to queue original frame, not translated_frame. Is
it correct?


WBR,
Paul.