ALWAYS_INLINE for json_match_keyword()

eregon · byroot · commit 7b284d4c1d5a · 2026-06-05T23:38:51.000+02:00
diff --git a/ext/json/ext/parser/parser.c b/ext/json/ext/parser/parser.c
@@ -1454,7 +1454,7 @@ static inline void json_value_completed(json_frame *frame)
     frame->phase = (enum json_frame_phase) frame->type;
 }
 
-static inline bool json_match_keyword(JSON_ParserState *state, const char *keyword, size_t offset)
+ALWAYS_INLINE(static) bool json_match_keyword(JSON_ParserState *state, const char *keyword, size_t offset)
 {
     // It is assumed that since `keyword` is always a literal, the compiler is able to constantize this
     // `strlen` and several other computations in that routine, such as eliminating the `if (resumable)` branch.

Original file line number	Diff line number	Diff line change
`@@ -1454,7 +1454,7 @@ static inline void json_value_completed(json_frame *frame)`
`1454`	`1454`	`frame->phase = (enum json_frame_phase) frame->type;`
`1455`	`1455`	`}`
`1456`	`1456`
`1457`		`-static inline bool json_match_keyword(JSON_ParserState state, const char keyword, size_t offset)`
	`1457`	`+ALWAYS_INLINE(static) bool json_match_keyword(JSON_ParserState state, const char keyword, size_t offset)`
`1458`	`1458`	`{`
`1459`	`1459`	// It is assumed that since `keyword` is always a literal, the compiler is able to constantize this
`1460`	`1460`	// `strlen` and several other computations in that routine, such as eliminating the `if (resumable)` branch.