Skip to content

Commit 59bdf9b

Browse files
Fix: Ensure ClearDefaultEventParameters is wrapped for internal SDK use
Corrects an issue in `analytics/src/swig/analytics.i` that caused a build failure (CS0117) because `FirebaseAnalyticsInternal.ClearDefaultEventParameters` was not being generated. The problem was that `%ignore firebase::analytics::ClearDefaultEventParameters;` was preventing SWIG from wrapping this function, even though it was later declared in `analytics.i` with the intent to be wrapped for internal C# SDK use. This commit removes the overly broad `%ignore` for `ClearDefaultEventParameters`. The function is now correctly wrapped by SWIG due to its explicit declaration in the SWIG interface file, making it available as `FirebaseAnalyticsInternal.ClearDefaultEventParameters()`. This allows `FirebaseAnalytics.SetDefaultParameters(null)` to call it as intended, without exposing `ClearDefaultEventParameters` as a new public API on `FirebaseAnalytics` itself.
1 parent a047cec commit 59bdf9b

File tree

1 file changed

+0
-1
lines changed

1 file changed

+0
-1
lines changed

analytics/src/swig/analytics.i

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -121,7 +121,6 @@ void SetDefaultEventParametersHelper(
121121
// Ignore SetDefaultEventParameters and ClearDefaultEventParameters, as we handle them
122122
// with a custom version or re-expose ClearDefaultEventParameters.
123123
%ignore firebase::analytics::SetDefaultEventParameters;
124-
%ignore firebase::analytics::ClearDefaultEventParameters;
125124
// Ignore the Parameter class, as we don't want to expose that to C# at all.
126125
%ignore firebase::analytics::Parameter;
127126

0 commit comments

Comments
 (0)