Skip to content

Commit aa5a78c

Browse files
christophpurrermeta-codesync[bot]
authored andcommitted
Fix unused parameter warnings in TimerManager (#56516)
Summary: Pull Request resolved: #56516 Fixed clang-diagnostic-unused-parameter warnings in TimerManager.cpp by commenting out the unused 'runtime' parameter names in two functions: - deleteTimer() at line 135 - deleteRecurringTimer() at line 152-153 Both functions accept a jsi::Runtime& parameter for API compatibility but don't use it internally. The fix maintains the function signatures while suppressing the warnings by commenting out the parameter names. Changelog: [Internal] Reviewed By: zeyap Differential Revision: D101108602 fbshipit-source-id: 95554385c6d36e724fdc1de4a3b20e1e1014a39c
1 parent acb85ce commit aa5a78c

1 file changed

Lines changed: 4 additions & 2 deletions

File tree

packages/react-native/ReactCommon/react/runtime/TimerManager.cpp

Lines changed: 4 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -132,7 +132,9 @@ TimerHandle TimerManager::createRecurringTimer(
132132
return timerID;
133133
}
134134

135-
void TimerManager::deleteTimer(jsi::Runtime& runtime, TimerHandle timerHandle) {
135+
void TimerManager::deleteTimer(
136+
jsi::Runtime& /* runtime */,
137+
TimerHandle timerHandle) {
136138
if (timerHandle < 0) {
137139
/**
138140
* Do nothing for negative values to match web spec.
@@ -150,7 +152,7 @@ void TimerManager::deleteTimer(jsi::Runtime& runtime, TimerHandle timerHandle) {
150152
}
151153

152154
void TimerManager::deleteRecurringTimer(
153-
jsi::Runtime& runtime,
155+
jsi::Runtime& /* runtime */,
154156
TimerHandle timerHandle) {
155157
if (timerHandle < 0) {
156158
/**

0 commit comments

Comments
 (0)