summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorPhilip Chimento <philip.chimento@gmail.com>2022-11-06 14:02:26 -0800
committerPhilip Chimento <philip.chimento@gmail.com>2023-02-20 23:11:31 -0800
commit35e1d81ce97045c6009998dc46b6bc6159e6eb15 (patch)
treea781b42af3e227d43d6796fe6cbe367fc721cab9
parentd11a2a8bb4c1a47cf0104fc198e0126aa35b06f8 (diff)
downloadgjs-35e1d81ce97045c6009998dc46b6bc6159e6eb15.tar.gz
function: Add missing include
This one is tricky, and not caught by IWYU, because constructing JS::RootedValueVector requires computing sizeof(JS::Value). It was also not caught in CI because the build error doesn't occur when using precompiled headers, and all of the CI build jobs currently do. Work around it by writing out the template parameter explicitly, instead of relying on the typedef. Closes: #503
-rw-r--r--gi/function.h3
1 files changed, 2 insertions, 1 deletions
diff --git a/gi/function.h b/gi/function.h
index 5db20ec8..92b73397 100644
--- a/gi/function.h
+++ b/gi/function.h
@@ -21,6 +21,7 @@
#include <js/GCVector.h>
#include <js/RootingAPI.h>
#include <js/TypeDecls.h>
+#include <js/Value.h>
#include "gi/closure.h"
#include "gjs/jsapi-util.h"
@@ -100,7 +101,7 @@ class GjsFunctionCallState {
public:
std::unordered_set<GIArgument*> ignore_release;
JS::RootedObject instance_object;
- JS::RootedValueVector return_values;
+ JS::RootedVector<JS::Value> return_values;
GjsAutoError local_error;
GICallableInfo* info;
uint8_t gi_argc = 0;