Fix `parse-c-struct'.

* module/system/foreign.scm (parse-c-struct): Update use of
  `pointer->bytevector' to the new API.

* test-suite/tests/foreign.test ("structs"): New test prefix.

diff --git a/module/system/foreign.scm b/module/system/foreign.scm
index f074998..121db60 100644 (file)
--- a/module/system/foreign.scm
+++ b/module/system/foreign.scm
@@ -18,6 +18,7 @@
 
 (define-module (system foreign)
   #:use-module (rnrs bytevectors)
+  #:use-module (srfi srfi-1)
   #:export (void
             float double
             int unsigned-int long unsigned-long size_t
@@ -124,4 +125,8 @@
     (bytevector->pointer bv)))
 
 (define (parse-c-struct foreign types)
-  (read-c-struct (pointer->bytevector foreign) 0 types))
+  (let ((size (fold (lambda (type total)
+                      (+ (sizeof type) total))
+                    0
+                    types)))
+    (read-c-struct (pointer->bytevector foreign size) 0 types)))

diff --git a/test-suite/tests/foreign.test b/test-suite/tests/foreign.test
index 2cc1e21..eb12360 100644 (file)
--- a/test-suite/tests/foreign.test
+++ b/test-suite/tests/foreign.test
@@ -77,3 +77,13 @@
                        (+ byte (* 256 address)))
                      0
                      bytes)))))
+
+\f
+(with-test-prefix "structs"
+
+  (pass-if "parse-c-struct"
+    (let ((layout (list int64 uint8))
+          (data   (list -300 43)))
+      (equal? (parse-c-struct (make-c-struct layout data)
+                              layout)
+              data))))