[Asterisk-code-review] format: Register format-attribute module with cached formats. (asterisk[master])

[Joshua Colp]

Joshua Colp has posted comments on this change.

Change subject: format: Register format-attribute module with cached formats.
......................................................................


Patch Set 2: Code-Review-1

(1 comment)

https://gerrit.asterisk.org/#/c/1600/2/main/format.c
File main/format.c:

Line 261: 	if (!interface) {
        : 		struct format_interface *format_interface = ao2_find(interfaces, format2->codec->name, OBJ_SEARCH_KEY);
        : 		if (format_interface) {
        : 			interface = format_interface->interface;
        : 			ao2_ref(format_interface, -1);
        : 		}
        : 	}
You don't need to do this twice. The codec can't differ between them.


-- 
To view, visit https://gerrit.asterisk.org/1600
To unsubscribe, visit https://gerrit.asterisk.org/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: I1ea1f0483e5261e2a050112e4ebdfc22057d1354
Gerrit-PatchSet: 2
Gerrit-Project: asterisk
Gerrit-Branch: master
Gerrit-Owner: Alexander Traud <pabstraud at compuserve.com>
Gerrit-Reviewer: Alexander Traud <pabstraud at compuserve.com>
Gerrit-Reviewer: Anonymous Coward #1000019
Gerrit-Reviewer: Joshua Colp <jcolp at digium.com>
Gerrit-HasComments: Yes