[asterisk-commits] dlee: branch dlee/stasis-router r383053 - /team/dlee/stasis-router/main/

SVN commits to the Asterisk project asterisk-commits at lists.digium.com
Thu Mar 14 09:09:35 CDT 2013


Author: dlee
Date: Thu Mar 14 09:09:32 2013
New Revision: 383053

URL: http://svnview.digium.com/svn/asterisk?view=rev&rev=383053
Log:
Premature optimization, and all that

Modified:
    team/dlee/stasis-router/main/stasis_message_router.c

Modified: team/dlee/stasis-router/main/stasis_message_router.c
URL: http://svnview.digium.com/svn/asterisk/team/dlee/stasis-router/main/stasis_message_router.c?view=diff&rev=383053&r1=383052&r2=383053
==============================================================================
--- team/dlee/stasis-router/main/stasis_message_router.c (original)
+++ team/dlee/stasis-router/main/stasis_message_router.c Thu Mar 14 09:09:32 2013
@@ -101,7 +101,8 @@
 		/* We don't expect many message types, so a simple loop should
 		 * be adequate, even if the complexity is O(n). Sorting the list
 		 * would be an easy way to bring that down to O(log(n)). Using a
-		 * hashtable/ao2_container could be even better.
+		 * hashtable/ao2_container could be even better. Just be sure to
+		 * profile before you optimize!
 		 */
 		route = router->default_route;
 		for (i = 0; i < router->num_routes_current; ++i) {




More information about the asterisk-commits mailing list