rtp: fix dead code and check for impossible values

nal_type is the index for a GstH265NalUnitType enum. There are two types of dead
code here:
First, after checking if nal_type is >= 39 there are two OR conditionals that
check if the value is in ranges higher than that number, so if nal_type >= 39
falls in the True branch those other conditions aren't checked and if it falls
in the False branch and they are checked, they will always also be False. They
are redundant.
Second, the enum has a range of 0 to 40. So the checks for ranges higher than 41
should never be True.
Removing this redundant checks.

CID 1249684
This commit is contained in:
Luis de Bethencourt 2015-01-08 13:47:09 +00:00 committed by Tim-Philipp Müller
parent 544c0d75ce
commit d215b18a20

View file

@ -914,8 +914,7 @@ gst_rtp_h265_depay_handle_nal (GstRtpH265Depay * rtph265depay, GstBuffer * nal,
}
complete = TRUE;
} else if ((nal_type >= 32 && nal_type <= 35)
|| nal_type >= 39 || (nal_type >= 41 && nal_type <= 44)
|| (nal_type >= 48 && nal_type <= 55)) {
|| nal_type >= 39) {
/* VPS, SPS, PPS, SEI, ... terminate an access unit */
complete = TRUE;
}