oden-js/src/module/loader.rs

@@ -9,7 +9,7 @@ pub enum ModuleSource<'a> {
 }
 
 pub trait ModuleLoader {
-    fn load(&self, context: &ContextRef, name: &str) -> Result<ModuleSource>;
+    fn load(&mut self, context: &ContextRef, name: &str) -> Result<ModuleSource>;
 }
 
 pub struct DefaultModuleLoader {}
@@ -21,7 +21,7 @@ impl DefaultModuleLoader {
 }
 
 impl ModuleLoader for DefaultModuleLoader {
-    fn load(&self, _context: &ContextRef, name: &str) -> Result<ModuleSource> {
+    fn load(&mut self, _context: &ContextRef, name: &str) -> Result<ModuleSource> {
         // Attempt to open the file.
         let path = Path::new(name);
         match std::fs::read_to_string(path) {
@@ -34,7 +34,7 @@ impl ModuleLoader for DefaultModuleLoader {
 pub(crate) fn load_module(
     context: &ContextRef,
     name: &str,
-    loader: &Box<dyn ModuleLoader>,
+    loader: &mut Box<dyn ModuleLoader>,
 ) -> *mut sys::JSModuleDef {
     match loader.load(context, name) {
         Ok(ModuleSource::Native(native)) => native.module,

oden-js/src/runtime.rs

@@ -3,11 +3,10 @@ use crate::ContextRef;
 use oden_js_sys as sys;
 use std::cell::RefCell;
 use std::ffi::CStr;
-use std::sync::Arc;
 
 struct PrivateState {
     refs: u64,
-    loader: Arc<Box<dyn ModuleLoader>>,
+    loader: Box<dyn ModuleLoader>,
 }
 
 impl PrivateState {
@@ -21,16 +20,9 @@ impl PrivateState {
             Err(_) => return std::ptr::null_mut(),
         };
 
-        let loader = unsafe {
+        let state = opaque as *mut PrivateState;
-            let ptr = opaque as *const RefCell<PrivateState>;
-            ptr.as_ref()
-                .expect("We already know this runtime is one of ours!")
-                .borrow()
-                .loader
-                .clone()
-        };
         let context = ContextRef::from_raw(ctx);
-        load_module(&context, path, &loader)
+        load_module(&context, path, &mut (*state).loader)
     }
 }
 
@@ -47,7 +39,7 @@ impl Runtime {
     pub fn with_loader<TLoader: ModuleLoader + 'static>(loader: TLoader) -> Runtime {
         let state = Box::new(RefCell::new(PrivateState {
             refs: 1,
-            loader: Arc::new(Box::new(loader)),
+            loader: Box::new(loader),
         }));
         let rt = unsafe {
             let rt = sys::JS_NewRuntime();

src/graphics.js

@@ -1,15 +1,15 @@
 import * as core from "graphics-core";
 
-export function cls(r, g, b) {
+function cls(r, g, b) {
   core.cls(r, g, b);
 }
 
-export function print(...args) {
+function print(...args) {
   core.print(args.join(" "));
 }
 
-export function spr(x, y, w, h, sx, sy, sw = undefined, sh = undefined) {
+function spr(x, y, w, h, sx, sy, sw = undefined, sh = undefined) {
   sw = sw | w;
   sh = sh | h;
-  core.spr(x, y, w, h, sx, sy, sw, sh);
+  core.spr(xy, w, h, sx, sy, sw, sh);
 }

src/main.js

@@ -1,4 +1,4 @@
-import { cls, print, spr } from "./src/graphics.js";
+import { cls, print, spr } from "./src/graphics";
 
 export function init() {
   print("Hello world!");