commit 1c85cbebb21d4da40f364e990be07e9aa002864c
author Yi Kong <yikong@google.com> Mon Dec 18 13:01:24 2023 +0900
committer Yi Kong <yikong@google.com> Mon Dec 18 13:07:33 2023 +0900
tree 7a847238578409ddb8f70012de337a09c429b067
parent e4eb1cb445acda18bda65a5a5d747fe2e07fe209

Reexport operator!= overloads in the derived type to avoid ambiguous base class lookups

clang-r510928 is becoming stricter on ambiguous lookups, and produces the following error:

frameworks/av/media/codec2/vndk/include/util/C2InterfaceHelper.h:606:34: error: member 'operator!=' found in multiple base classes of different types
  606 |                 if (helper.get() != *typedParam->get()) {
      |                                  ^
frameworks/av/media/codec2/vndk/include/util/C2InterfaceHelper.h:587:23: note: while substituting into a lambda expression here
  587 |             setSetter([typedParam, fn, &deps...](
      |                       ^
...
frameworks/av/media/codec2/core/include/C2Param.h:430:17: note: member found by ambiguous name lookup
  430 |     inline bool operator!=(const C2Param &o) const { return !operator==(o); }
      |                 ^
frameworks/av/media/codec2/core/include/C2Config.h:1320:10: note: member found by ambiguous name lookup
 1320 |     bool operator!=(const C2RectStruct &) = delete;
      |          ^

Test: presubmit
Bug: 316272684
Change-Id: If199a6df9a426e24a1f02ee99d5a1d8d454ed769

media/codec2/core/include/C2ParamDef.h

diff --git a/media/codec2/core/include/C2ParamDef.h b/media/codec2/core/include/C2ParamDef.h
index d578820..86dfe65 100644
--- a/media/codec2/core/include/C2ParamDef.h
+++ b/media/codec2/core/include/C2ParamDef.h
@@ -404,6 +404,7 @@
     /// Specialization for an input port parameter.
     struct input : public T, public S,
             public _C2StructCheck<S, ParamIndex, T::PARAM_KIND | T::Index::DIR_INPUT> {
+        using T::operator!=;
         _C2_CORE_INDEX_OVERRIDE(ParamIndex)
         /// Wrapper around base structure's constructor.
         template<typename ...Args>
@@ -416,6 +417,7 @@
     /// Specialization for an output port parameter.
     struct output : public T, public S,
             public _C2StructCheck<S, ParamIndex, T::PARAM_KIND | T::Index::DIR_OUTPUT> {
+        using T::operator!=;
         _C2_CORE_INDEX_OVERRIDE(ParamIndex)
         /// Wrapper around base structure's constructor.
         template<typename ...Args>
@@ -470,6 +472,7 @@
     /// Specialization for an input port parameter.
     struct input : public T,
             public _C2FlexStructCheck<S, ParamIndex, T::PARAM_KIND | T::Index::DIR_INPUT> {
+        using T::operator!=;
     private:
         /// Wrapper around base structure's constructor while also specifying port/direction.
         template<typename ...Args>
@@ -486,6 +489,7 @@
     /// Specialization for an output port parameter.
     struct output : public T,
             public _C2FlexStructCheck<S, ParamIndex, T::PARAM_KIND | T::Index::DIR_OUTPUT> {
+        using T::operator!=;
     private:
         /// Wrapper around base structure's constructor while also specifying port/direction.
         template<typename ...Args>
@@ -549,6 +553,7 @@
     struct input : public T, public S,
             public _C2StructCheck<S, ParamIndex,
                     T::PARAM_KIND | T::Index::IS_STREAM_FLAG | T::Type::DIR_INPUT> {
+        using T::operator!=;
         _C2_CORE_INDEX_OVERRIDE(ParamIndex)
 
         /// Default constructor. Stream-ID is undefined.
@@ -567,6 +572,7 @@
     struct output : public T, public S,
             public _C2StructCheck<S, ParamIndex,
                     T::PARAM_KIND | T::Index::IS_STREAM_FLAG | T::Type::DIR_OUTPUT> {
+        using T::operator!=;
         _C2_CORE_INDEX_OVERRIDE(ParamIndex)
 
         /// Default constructor. Stream-ID is undefined.
@@ -634,6 +640,7 @@
     struct input : public T,
             public _C2FlexStructCheck<S, ParamIndex,
                     T::PARAM_KIND | T::Index::IS_STREAM_FLAG | T::Type::DIR_INPUT> {
+        using T::operator!=;
     private:
         /// Default constructor. Stream-ID is undefined.
         inline input(size_t flexCount) : T(_Type::CalcSize(flexCount), input::PARAM_TYPE) { }
@@ -656,6 +663,7 @@
     struct output : public T,
             public _C2FlexStructCheck<S, ParamIndex,
                     T::PARAM_KIND | T::Index::IS_STREAM_FLAG | T::Type::DIR_OUTPUT> {
+        using T::operator!=;
     private:
         /// Default constructor. Stream-ID is undefined.
         inline output(size_t flexCount) : T(_Type::CalcSize(flexCount), output::PARAM_TYPE) { }