From 40c13bd1b0ad69eb94d0ab465d548b825f0893e8 Mon Sep 17 00:00:00 2001 From: Tilghman Lesher Date: Fri, 28 Aug 2009 20:14:39 +0000 Subject: Merged revisions 214701 via svnmerge from https://origsvn.digium.com/svn/asterisk/branches/1.4 ........ r214701 | tilghman | 2009-08-28 15:13:32 -0500 (Fri, 28 Aug 2009) | 8 lines Modify comment to be a bit more accurate. We have kept this comment around long enough, that it's pretty clear that we're keeping the code, because changing the code would require a pretty fundamental architectural shift. We've also taken criticism in some quarters, because it was believed that it was referring to the code being nasty. No, the code isn't nasty, just the operation itself is rather odd. Fixed for eternity (probably not). ........ git-svn-id: https://origsvn.digium.com/svn/asterisk/trunk@214702 65c4cc65-6c06-0410-ace0-fbb531ad65f3 --- main/channel.c | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-) (limited to 'main') diff --git a/main/channel.c b/main/channel.c index 8573570e0..b525d257b 100644 --- a/main/channel.c +++ b/main/channel.c @@ -5045,10 +5045,10 @@ int ast_do_masquerade(struct ast_channel *original) char masqn[AST_CHANNEL_NAME]; char zombn[AST_CHANNEL_NAME]; - /* XXX This is a seriously wacked out operation. We're essentially putting the guts of - the clone channel into the original channel. Start by killing off the original - channel's backend. I'm not sure we're going to keep this function, because - while the features are nice, the cost is very high in terms of pure nastiness. XXX */ + /* XXX This operation is a bit odd. We're essentially putting the guts of + * the clone channel into the original channel. Start by killing off the + * original channel's backend. While the features are nice, which is the + * reason we're keeping it, it's still awesomely weird. XXX */ /* We need the clone's lock, too */ ast_channel_lock(clonechan); -- cgit v1.2.3