search

tromey / emacs-ffi

FFI for Emacs
99 stars 17 forks source link

Fixes some problems I've found using ffi.el #13

Closed zevlg closed 7 years ago

zevlg commented 7 years ago

[fix] define-ffi-struct: make it work for definitions like

(define-ffi-struct PointInfo
  (x :type :double)
  (y :type :double))

[fix] with-ffi-string/with-ffi-temporary: execute body under `progn', so multiexpression body won't screw things up

tromey commented 7 years ago

Thanks! I think this looks pretty good, but would you mind adding a test case?

zevlg commented 7 years ago

I figured out about symbol-name. It definitely was just a typo, symbol-value should be used instead.

fixed, and I also added test cases https://github.com/tromey/emacs-ffi/pull/13/commits/db05bb4cdaceeebd01bf5f47c5d13755e87c4099

tromey commented 7 years ago

Thanks!