Skip to content

Commit

Permalink
[codec][vaapi] Add const to copy assignment
Browse files Browse the repository at this point in the history
llvm/llvm-project#89652 made
std::pair trivially copyable if its members are, and
it caused the following issue: "error: the parameter
for this explicitly-defaulted copy assignment operator
is const, but a member or base requires it to be
non-const". This patch adds const to copy assignment
operator to fix the issue.

Bug: 354627057
Change-Id: I10ee04c40da8820e33b80f51492e136a27e97972
Reviewed-on: https://fuchsia-review.googlesource.com/c/fuchsia/+/1086537
Fuchsia-Auto-Submit: Gulfem Savrun Yeniceri <[email protected]>
Commit-Queue: Auto-Submit <[email protected]>
Reviewed-by: John Bauman <[email protected]>
  • Loading branch information
gulfemsavrun authored and CQ Bot committed Jul 22, 2024
1 parent bef64e6 commit 3d310ae
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion src/media/codec/codecs/vaapi/codec_adapter_vaapi_encoder.h
Original file line number Diff line number Diff line change
Expand Up @@ -47,7 +47,7 @@ class VaApiEncoderOutput {
~VaApiEncoderOutput();

VaApiEncoderOutput& operator=(VaApiEncoderOutput&& other) noexcept;
VaApiEncoderOutput& operator=(VaApiEncoderOutput& other) noexcept = delete;
VaApiEncoderOutput& operator=(const VaApiEncoderOutput& other) noexcept = delete;

private:
uint8_t* base_address_ = nullptr;
Expand Down

0 comments on commit 3d310ae

Please sign in to comment.