[vlc-devel] [PATCH] demux/ty: Demux: replace if-else with switch

Francois Cartegnie fcvlcdev at free.fr
Mon Mar 20 14:37:06 CET 2017


Le 20/03/2017 à 14:26, Filip Roséen a écrit :
> A switch should be easier to maintain, while also improving
> readability.


> +        case 0x01: /* closed caption */
> +        case 0x02: /* XDS */
> +            DemuxRecCc( p_demux, p_rec, p_block_in );
> +            break;

Misleading incorrect attribution of comments to single values.

Francois


More information about the vlc-devel mailing list