Fix nqp::eoffh. Implement nqp::seekfh.

pmurias · Dec 3, 2014 · 1cf4ad3 · 1cf4ad3
1 parent 5a9057c
commit 1cf4ad3
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 1 deletion.
diff --git a/src/vm/js/QAST/Compiler.nqp b/src/vm/js/QAST/Compiler.nqp
@@ -374,6 +374,7 @@ class QAST::OperationsJS {
     add_simple_op('open', $T_OBJ, [$T_STR, $T_STR], :sideffects);
 
     add_simple_op('tellfh', $T_INT, [$T_OBJ], :sideffects);
+    add_simple_op('seekfh', $T_INT, [$T_OBJ, $T_INT, $T_INT], :sideffects);
     add_simple_op('eoffh', $T_INT, [$T_OBJ], :sideffects);
     add_simple_op('readlinefh', $T_STR, [$T_OBJ], :sideffects);
     add_simple_op('readallfh', $T_STR, [$T_OBJ], :sideffects);

diff --git a/src/vm/js/nqp-runtime/io.js b/src/vm/js/nqp-runtime/io.js
@@ -120,7 +120,7 @@ op.eoffh = function(fh) {
   // I haven't found a way to implement this directly in node.js
   var current = fs.seekSync(fh.fd, 0, 1);
   var end = fs.seekSync(fh.fd, 0, 2);
-  fs.seekSync(fh.fd, 0, current);
+  fs.seekSync(fh.fd, current, 0);
   return current == end ? 1 : 0;
 };
 
@@ -170,6 +170,10 @@ op.readallfh = function(fh) {
   return iconv.decode(all, fh.encoding);
 };
 
+op.seekfh = function(fh, offset, whence) {
+  fs.seekSync(fh.fd, offset, whence);
+};
+
 op.closefh = function(fh) {
   fh.closefh();
   return fh;