Use absl::Span in AudioFrameObu constructor.

PiperOrigin-RevId: 694211381
AOMediaCodec · Nov 11, 2024 · 1ec8bd5 · 1ec8bd5
1 parent 3db06a5
commit 1ec8bd5
Show file tree

Hide file tree

Showing 3 changed files with 7 additions and 3 deletions.
diff --git a/iamf/obu/BUILD b/iamf/obu/BUILD
@@ -56,6 +56,7 @@ cc_library(
         "@com_google_absl//absl/log",
         "@com_google_absl//absl/status",
         "@com_google_absl//absl/status:statusor",
+        "@com_google_absl//absl/types:span",
     ],
 )
 

diff --git a/iamf/obu/audio_frame.cc b/iamf/obu/audio_frame.cc
@@ -16,6 +16,7 @@
 
 #include "absl/log/log.h"
 #include "absl/status/status.h"
+#include "absl/types/span.h"
 #include "iamf/common/macros.h"
 #include "iamf/common/read_bit_buffer.h"
 #include "iamf/common/write_bit_buffer.h"
@@ -42,9 +43,9 @@ ObuType GetObuType(uint32_t substream_id) {
 
 AudioFrameObu::AudioFrameObu(const ObuHeader& header,
                              DecodedUleb128 substream_id,
-                             const std::vector<uint8_t>& audio_frame)
+                             absl::Span<const uint8_t> audio_frame)
     : ObuBase(header, GetObuType(substream_id)),
-      audio_frame_(audio_frame),
+      audio_frame_(audio_frame.begin(), audio_frame.end()),
       audio_substream_id_(substream_id) {}
 
 absl::StatusOr<AudioFrameObu> AudioFrameObu::CreateFromBuffer(

diff --git a/iamf/obu/audio_frame.h b/iamf/obu/audio_frame.h
@@ -17,6 +17,7 @@
 
 #include "absl/status/status.h"
 #include "absl/status/statusor.h"
+#include "absl/types/span.h"
 #include "iamf/common/read_bit_buffer.h"
 #include "iamf/common/write_bit_buffer.h"
 #include "iamf/obu/obu_base.h"
@@ -58,9 +59,10 @@ class AudioFrameObu : public ObuBase {
    *
    * \param header `ObuHeader` of the OBU.
    * \param substream_id Substream ID.
+   * \param audio_frame Audio frame.
    */
   AudioFrameObu(const ObuHeader& header, DecodedUleb128 substream_id,
-                const std::vector<uint8_t>& audio_frame);
+                absl::Span<const uint8_t> audio_frame);
 
   /*!\brief Destructor.*/
   ~AudioFrameObu() = default;