Skip to content

Commit

Permalink
Removes move constructors from obu_base.h and ia_sequence_header.h - …
Browse files Browse the repository at this point in the history
…these are unused and do not allow for the updating of an IaSequenceHeaderObu instance.

PiperOrigin-RevId: 632541196
  • Loading branch information
Googler authored and jwcullen committed May 14, 2024
1 parent d7008a4 commit af7c4ba
Show file tree
Hide file tree
Showing 2 changed files with 4 additions and 7 deletions.
7 changes: 3 additions & 4 deletions iamf/obu/ia_sequence_header.h
Original file line number Diff line number Diff line change
Expand Up @@ -47,6 +47,8 @@ class IASequenceHeaderObu : public ObuBase {
primary_profile_(primary_profile),
additional_profile_(additional_profile) {}

IASequenceHeaderObu() = default;

/*!\brief Creates a `IASequenceHeaderObu` from a `ReadBitBuffer`.
*
* This function is designed to be used from the perspective of the decoder.
Expand All @@ -62,10 +64,7 @@ class IASequenceHeaderObu : public ObuBase {
const ObuHeader& header, ReadBitBuffer& rb);

/*\!brief Copy constructor.*/
IASequenceHeaderObu(IASequenceHeaderObu& other) = default;

/*\!brief Move constructor.*/
IASequenceHeaderObu(IASequenceHeaderObu&& other) = default;
IASequenceHeaderObu(const IASequenceHeaderObu& other) = default;

/*!\brief Destructor. */
~IASequenceHeaderObu() override = default;
Expand Down
4 changes: 1 addition & 3 deletions iamf/obu/obu_base.h
Original file line number Diff line number Diff line change
Expand Up @@ -38,9 +38,7 @@ class ObuBase {
* \param obu_type `obu_type` of the OBU.
*/
ObuBase(ObuType obu_type) : ObuBase(ObuHeader(), obu_type) {}

/*\!brief Move constructor.*/
ObuBase(ObuBase&& other) = default;
ObuBase() = default;

/*\!brief Copy constructor.*/
ObuBase(const ObuBase& other) = default;
Expand Down

0 comments on commit af7c4ba

Please sign in to comment.