List Info

Thread: russell: trunk r41278 - in /trunk: apps/channels/ include/asterisk/ main/




russell: trunk r41278 - in /trunk: apps/channels/ include/asterisk/ main/
user name
2006-08-30 12:43:30
Hello,

----- Original Message -----
From: <asterisk-commitslists.digium.com>
To: <asterisk-commitslists.digium.com>;
<svn-commitslists.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.

_______________________________________________
--Bandwidth and Colocation provided by Easynews.com --

asterisk-dev mailing list
To UNSUBSCRIBE or update options visit:
   http://lists.digium.com/mailman/listinfo/asterisk-dev
russell: trunk r41278 - in /trunk: apps/channels/ include/asterisk/ main/
user name
2006-08-30 17:04:33
----- Paul Cadach <paulodt.east.telecom.kz>
wrote:
> Originally we have queued copy of translated_frame,
currently we adds
> to queue original frame, not translated_frame. Is
> it correct?

No, that is a mistake.  Very nice job catching that!  Thank
you!

-- 
Russell Bryant
Software Engineer
Digium, Inc.

_______________________________________________
--Bandwidth and Colocation provided by Easynews.com --

asterisk-dev mailing list
To UNSUBSCRIBE or update options visit:
   http://lists.digium.com/mailman/listinfo/asterisk-dev
[1-2]

about | contact  Other archives ( Real Estate discussion Medical topics )