[gjs: 6/9] arg-cache: Only use one structure for fallback interface types
- From: Philip Chimento <pchimento src gnome org>
- To: commits-list gnome org
- Cc:
- Subject: [gjs: 6/9] arg-cache: Only use one structure for fallback interface types
- Date: Mon, 10 Jan 2022 01:47:47 +0000 (UTC)
commit 064da408a2e04862a3c687354935c4ab02063cbd
Author: Marco Trevisan (TreviƱo) <mail 3v1n0 net>
Date: Mon Oct 26 15:02:48 2020 +0100
arg-cache: Only use one structure for fallback interface types
They're just the same thing, but with different definitions, so we can
just use one type. Simplifying differences.
gi/arg-cache.cpp | 27 ++++-----------------------
1 file changed, 4 insertions(+), 23 deletions(-)
---
diff --git a/gi/arg-cache.cpp b/gi/arg-cache.cpp
index 91d00c8c..e85a260b 100644
--- a/gi/arg-cache.cpp
+++ b/gi/arg-cache.cpp
@@ -487,17 +487,6 @@ struct FallbackInterfaceIn : FallbackIn, RegisteredType {
using RegisteredType::RegisteredType;
};
-struct FallbackInterfaceInstanceIn : RegisteredIn, TypeInfo {
- using RegisteredIn::RegisteredIn;
-
- bool in(JSContext* cx, GjsFunctionCallState*, GIArgument* arg,
- JS::HandleValue value) override {
- return gjs_value_to_g_argument(cx, value, &m_type_info, m_arg_name,
- GJS_ARGUMENT_ARGUMENT, m_transfer,
- flags(), arg);
- }
-};
-
struct BoxedInTransferNone : RegisteredIn {
using RegisteredIn::RegisteredIn;
bool in(JSContext*, GjsFunctionCallState*, GIArgument*,
@@ -1838,12 +1827,8 @@ void ArgsCache::build_interface_in_arg(uint8_t gi_index, GITypeInfo* type_info,
if (arg_cache::is_gdk_atom(interface_info)) {
// Fall back to the generic marshaller
- if constexpr (ArgKind == Arg::Kind::INSTANCE)
- set_argument_auto<Arg::FallbackInterfaceInstanceIn,
- ArgKind>(common_args);
- else
- set_argument_auto<Arg::FallbackInterfaceIn, ArgKind>(
- common_args);
+ set_argument_auto<Arg::FallbackInterfaceIn, ArgKind>(
+ common_args);
return;
}
@@ -1873,12 +1858,8 @@ void ArgsCache::build_interface_in_arg(uint8_t gi_index, GITypeInfo* type_info,
}
if (g_type_is_a(gtype, G_TYPE_PARAM)) {
- if constexpr (ArgKind == Arg::Kind::INSTANCE)
- set_argument_auto<Arg::FallbackInterfaceInstanceIn,
- ArgKind>(common_args);
- else
- set_argument_auto<Arg::FallbackInterfaceIn, ArgKind>(
- common_args);
+ set_argument_auto<Arg::FallbackInterfaceIn, ArgKind>(
+ common_args);
return;
}
[
Date Prev][
Date Next] [
Thread Prev][
Thread Next]
[
Thread Index]
[
Date Index]
[
Author Index]