You've already forked FFmpeg
							
							
				mirror of
				https://github.com/FFmpeg/FFmpeg.git
				synced 2025-10-30 23:18:11 +02:00 
			
		
		
		
	avutil/channel_layout: return earlier on UNSPEC layouts in av_channel_layout_subset()
No point running all 64 iterations in the loop to never write anything to ret. Also make ambisonic layouts check its mask too while at it. Signed-off-by: James Almer <jamrial@gmail.com>
This commit is contained in:
		| @@ -989,12 +989,16 @@ uint64_t av_channel_layout_subset(const AVChannelLayout *channel_layout, | ||||
|     uint64_t ret = 0; | ||||
|     int i; | ||||
|  | ||||
|     if (channel_layout->order == AV_CHANNEL_ORDER_NATIVE) | ||||
|     switch (channel_layout->order) { | ||||
|     case AV_CHANNEL_ORDER_NATIVE: | ||||
|     case AV_CHANNEL_ORDER_AMBISONIC: | ||||
|         return channel_layout->u.mask & mask; | ||||
|  | ||||
|     for (i = 0; i < 64; i++) | ||||
|         if (mask & (1ULL << i) && av_channel_layout_index_from_channel(channel_layout, i) >= 0) | ||||
|             ret |= (1ULL << i); | ||||
|     case AV_CHANNEL_ORDER_CUSTOM: | ||||
|         for (i = 0; i < 64; i++) | ||||
|             if (mask & (1ULL << i) && av_channel_layout_index_from_channel(channel_layout, i) >= 0) | ||||
|                 ret |= (1ULL << i); | ||||
|         break; | ||||
|     } | ||||
|  | ||||
|     return ret; | ||||
| } | ||||
|   | ||||
		Reference in New Issue
	
	Block a user