Was testing some LLD features on Ladybird and needed to add these small changes to make it compile.
Silences the -Wuser-defined-literals warning which is triggered by our
use of the sv suffix for StringView
Removes an unused captured this pointer [-Wunused-lambda-capture]
Changes a JSONArray.h include to JSONObject.h to get the definition
for JSONValue::serialize. This is needed because template functions
are not exported for dylibs on macOS. This is a hack; the JSON headers
should be refactored so that each one includes the definition of
the template functions it sees. -- Maybe we should build with
-fvisibility-inlines-hidden on Linux to catch issues like this?
Was testing some LLD features on Ladybird and needed to add these small changes to make it compile.
sv
suffix for StringViewthis
pointer [-Wunused-lambda-capture]JSONValue::serialize
. This is needed because template functions are not exported for dylibs on macOS. This is a hack; the JSON headers should be refactored so that each one includes the definition of the template functions it sees. -- Maybe we should build with -fvisibility-inlines-hidden on Linux to catch issues like this?