Skip to content

Commit

Permalink
Fix IGWarnAssert macro to remove unneeded condition parameter
Browse files Browse the repository at this point in the history
Summary:
I'm looking at updating IGKitKit's test suite for Xcode 15, but when I went to do a build, it failed at the `IGWarnAssert` macro. Turns out that because we set the condition in the macro to always return `(NO)`, there's no need to expose `frmt` to test for the condition.

This diff removes `frmt` from the define, allowing the tests to build again.

Reviewed By: fabiomassimo

Differential Revision:
D49628618

Privacy Context Container: L1203725

fbshipit-source-id: b0456a149bf5be96c8401a21866880ccd72ebbcb
  • Loading branch information
TimOliver authored and facebook-github-bot committed Sep 26, 2023
1 parent 25d1887 commit 2019083
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion Source/IGListDiffKit/IGListAssert.h
Original file line number Diff line number Diff line change
Expand Up @@ -14,7 +14,7 @@
#endif // IGWarn

#ifndef IGWarnAssert
#define IGWarnAssert(frmt, ... ) IGAssert( (NO) , ##__VA_ARGS__)
#define IGWarnAssert( ... ) IGAssert( (NO) , ##__VA_ARGS__)
#endif // IGWarnAssert

#ifndef IGFailAssert
Expand Down

0 comments on commit 2019083

Please sign in to comment.