narcissus doesn't build on windows because mrbkap entrailed a js private function (js_GetDependentStringChars) using a js macro (JSSTRING_CHARS) in jsshell (Evaluate) which is an embedding consumer.
Created attachment 189807 [details] [diff] [review] use public apis timeless was concerned about JS_GetStringChars returning NULL, but I don't think that source can be a dependent string, so JS_GetStringChars shouldn't need to allocate anything and we should be safe.
Comment on attachment 189807 [details] [diff] [review] use public apis Brendan says r=him.
Furthermore (timeless!), JS_GetStringChars never returns null. If there's an OOM it returns a pointer into the dependent string's base string's chars, which lacks only a NUL backstop terminator. /be
Comment on attachment 189807 [details] [diff] [review] use public apis So this is not part of the build (unlike xpcshell), but a=me anyway -- get it in so we can all move on ;-). /be