[asterisk-dev] [Code Review]: Opaquify remaining fields and move struct ast_channel into main/channel_internal_api.c

Terry Wilson reviewboard at asterisk.org
Mon Mar 12 19:45:41 CDT 2012



> On March 12, 2012, 2:06 p.m., rmudgett wrote:
> > You are missing the new channel_internal.h file in this diff.

Well crap, I suppose I will re-write those changes...


> On March 12, 2012, 2:06 p.m., rmudgett wrote:
> > /trunk/main/channel_internal_api.c, lines 1175-1179
> > <https://reviewboard.asterisk.org/r/1786/diff/2/?file=26000#file26000line1175>
> >
> >     It is a bit silly to have this function return anything since it is a setter function.

Eh, it was convenient since the only place that would *ever* call this function is the channel alloc function just before return. If it makes you a happy camper, I'll make it void and  add an extra line to the alloc function.


- Terry


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviewboard.asterisk.org/r/1786/#review5796
-----------------------------------------------------------


On March 9, 2012, 3:54 p.m., Terry Wilson wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviewboard.asterisk.org/r/1786/
> -----------------------------------------------------------
> 
> (Updated March 9, 2012, 3:54 p.m.)
> 
> 
> Review request for Asterisk Developers.
> 
> 
> Summary
> -------
> 
> This should finish up the first stage of opaquification.
> 
> 
> Diffs
> -----
> 
>   /trunk/main/channel.c 358725 
>   /trunk/main/channel_internal_api.c 358725 
>   /trunk/apps/app_channelredirect.c 358725 
>   /trunk/apps/app_chanspy.c 358725 
>   /trunk/apps/app_confbridge.c 358725 
>   /trunk/apps/app_dial.c 358725 
>   /trunk/apps/app_disa.c 358725 
>   /trunk/apps/app_dumpchan.c 358725 
>   /trunk/apps/app_externalivr.c 358725 
>   /trunk/apps/app_followme.c 358725 
>   /trunk/apps/app_macro.c 358725 
>   /trunk/apps/app_mixmonitor.c 358725 
>   /trunk/apps/app_parkandannounce.c 358725 
>   /trunk/apps/app_queue.c 358725 
>   /trunk/apps/app_speech_utils.c 358725 
>   /trunk/apps/app_stack.c 358725 
>   /trunk/apps/app_talkdetect.c 358725 
>   /trunk/bridges/bridge_multiplexed.c 358725 
>   /trunk/channels/chan_agent.c 358725 
>   /trunk/channels/chan_bridge.c 358725 
>   /trunk/channels/chan_dahdi.c 358725 
>   /trunk/channels/chan_local.c 358725 
>   /trunk/channels/chan_phone.c 358725 
>   /trunk/channels/chan_sip.c 358725 
>   /trunk/channels/chan_unistim.c 358725 
>   /trunk/channels/chan_vpb.cc 358725 
>   /trunk/channels/sig_analog.c 358725 
>   /trunk/include/asterisk/channel.h 358725 
>   /trunk/main/app.c 358725 
>   /trunk/main/autoservice.c 358725 
>   /trunk/main/bridging.c 358725 
>   /trunk/main/cli.c 358725 
>   /trunk/main/features.c 358725 
>   /trunk/main/file.c 358725 
>   /trunk/main/indications.c 358725 
>   /trunk/main/manager.c 358725 
>   /trunk/main/pbx.c 358725 
>   /trunk/main/rtp_engine.c 358725 
>   /trunk/res/res_agi.c 358725 
>   /trunk/res/res_musiconhold.c 358725 
>   /trunk/res/snmp/agent.c 358725 
> 
> Diff: https://reviewboard.asterisk.org/r/1786/diff
> 
> 
> Testing
> -------
> 
> unit tests pass. I think I ran the testsuite on it last night and it passed. I'm running it again right now just to make sure.
> 
> 
> Thanks,
> 
> Terry
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.digium.com/pipermail/asterisk-dev/attachments/20120313/c4ed4c2a/attachment-0001.htm>


More information about the asterisk-dev mailing list