Apply reviewers suggestion

membraneframework · Feb 23, 2024 · bf50ec9 · bf50ec9
1 parent 088936e
commit bf50ec9
Showing 1 changed file with 9 additions and 9 deletions.
diff --git a/lib/membrane/rtp/rtsp_decapsulator.ex b/lib/membrane/rtp/rtsp_decapsulator.ex
@@ -100,17 +100,17 @@ defmodule Membrane.RTP.RTSP.Decapsulator do
          channel_id,
          complete_packets
        ) do
-    if payload_length > byte_size(rest) do
-      {packets_binary, Enum.reverse(complete_packets)}
-    else
-      <<complete_packet_binary::binary-size(payload_length)-unit(8), rest::binary>> = rest
+    case rest do
+      <<complete_packet_binary::binary-size(payload_length)-unit(8), rest::binary>> ->
+        complete_packets =
+          if channel_id != received_channel_id,
+            do: complete_packets,
+            else: [complete_packet_binary | complete_packets]
 
-      complete_packets =
-        if channel_id != received_channel_id,
-          do: complete_packets,
-          else: [complete_packet_binary | complete_packets]
+        get_complete_packets(rest, channel_id, complete_packets)
 
-      get_complete_packets(rest, channel_id, complete_packets)
+      _incomplete_packet_binary ->
+        {packets_binary, Enum.reverse(complete_packets)}
     end
   end