revised API and implementation #37

hhas · 2022-03-10T16:59:26Z

significantly improves performance by caching parse method types and wrapper functions on first use
JS<->ObjC type bridging is fully integrated with ref-napi's JS<->C type descriptors and uses ObjC-style type encoding strings throughout
redesigned APIs for creating blocks, structs, subclasses, and inout arguments
ns, js functions support custom handling of unconverted values
updated example scripts and documentation

TO DO:

inout arguments in blocks and subclass methods
array, union, bitwise types
update test.js
optionally replace ffi-napi with C++ extension to improve performance
clean up code comments

… also now caches method wrappers (ObjC's method introspection APIs are heinously slow, so this caching provides ~1000× speedup on repeated method calls!); caution: this is all *very* Work In Progress (lots of stuff is incomplete/inconsistent/broken/mis-commented/to be redone; including JS↔︎NS type conversions, inout args, unit tests, docs, etc, etc)

…speed is much improved without breaking behavior)

… we finish codecs redesign) * renamed `selectorForMethodName` to `Selector.fromjs` (this allows users to create Selector objects from both "foo:bar:" and "foo_bar_" strings) * various rejigging of internal API designs

* added `Selector.tojs()` * `callObjCMethod` now includes object and method name in error messages * added extra debug logging/error handling to `introspectMethod`

* temporary fixes for `void*` inout arguments (pass a Buffer for now), argument encodings containing inout/out/etc modifiers

…sses a third argument to handler’s `get` which is the proxy object itself, so use that—*not* `this`, which is the handler object only—to call additional ObjC methods on the object)

…ns are TBD, but these should do for now)

….inspect(objc)` result * sorting out (touch-wood) `util.inspect.custom` support

* revised `ns`, `js` functions take function/value as optional 2nd argument, allowing handling of unconverted values to be customized * updated examples

…[util.inspect.custom]` and `Proxy` behave sensibly together

…ons, streamlining packing and unpacking of method arguments and results

… dog-slow overall; which might be ffi overhead) * moved `swizzle` back to its own module, updating implementation (not yet tested) * redesigned block API, dividing it into `BlockType` and `Block` (not yet tested, and `BlockType` is not yet fully ref-napi compatible)

…from `ref-struct-di`, with `objc.structs.define` function for defining new struct types, and already-created struct types being cached and exported as `objc.structs.NAME` (note: `ref*` types may eventually go away, or at least be relegated to 'slow fallback' as there is are severe performance bottlenecks in `ref-napi` and `ffi-napi`; thus users should use ObjC encoding strings rather than `objc.types...` when defining new struct types)

…se are currently publicly exported as `objc.blocks` and `objc.structs` but should eventually merge into top-level `objc` namespace) * various renamings of public classes for consistency (TBD: should they all have `ObjC` prefix or not; on one hand it may make troubleshooting easier as their names are otherwise very generic; on the other, name prefixing is verbose and clumsy) * reworked CommonJS `require` and `module.exports` usage to allow for circular dependencies (i.e. 1. attach to `module.exports` instead of replacing it; and 2. bind the whole module returned by `require`, not parts of it)

…ion, example script, and docs to use current APIs (its own API is still TBD) * struct and block types should now be accessible via `objc`, although this is currently kludgy and doesn’t detect/prevent ObjC namespace collisions (which is a problem in current `objc.blocks` and `objc.structs` API design)

…`objc` object) * simplified subclassing API (encodings object is now passed as argument before methods object) * tidying code, comments, docs

…red parser object isn't thread safe) * added `objc.auto()` convenience function, equivalent to `@autoreleasepool {...}` * tidied comments

# Conflicts: # src/instance.js # src/objctypes.js # test/test-inout.js

hhas added 28 commits January 23, 2022 16:53

added quick-n-dirty test script (mostly to check that method calling …

99e2b92

…speed is much improved without breaking behavior)

bump

d6c8e5b

* fixed bugs in ObjCObject[toPrimitive]

0e2bd2e

* added `Selector.tojs()` * `callObjCMethod` now includes object and method name in error messages * added extra debug logging/error handling to `introspectMethod`

* testing, debugging, rearranging internal APIs

685fead

* temporary fixes for `void*` inout arguments (pass a Buffer for now), argument encodings containing inout/out/etc modifiers

more testing, debugging, straightening out kinks (of note: Proxy pa…

b16c84c

…sses a third argument to handler’s `get` which is the proxy object itself, so use that—*not* `this`, which is the handler object only—to call additional ObjC methods on the object)

disentangling JS representations of ObjC objects (final representatio…

e007c88

…ns are TBD, but these should do for now)

* moved objc.types to objc.__internal__.types; decluttering `util…

70d0785

….inspect(objc)` result * sorting out (touch-wood) `util.inspect.custom` support

* fixed [hopefully] util.inspect.custom support in ObjCObjects

1849a67

* revised `ns`, `js` functions take function/value as optional 2nd argument, allowing handling of unconverted values to be customized * updated examples

fixed an error in the neverending whackamole that is trying to make `…

dc89010

…[util.inspect.custom]` and `Proxy` behave sensibly together

updated readme

333f0ca

* rewrote ObjC type bridging to use ref-napi-compatible type definiti…

9707831

…ons, streamlining packing and unpacking of method arguments and results

fixed regression; method wrapper throws more descriptive error message

c826505

bug fixes in isClass, isInstance

66b1ee4

unbroke the build

29ee162

* simplified block and struct API (all types now appear on top-level …

8be2533

…`objc` object) * simplified subclassing API (encodings object is now passed as argument before methods object) * tidying code, comments, docs

tweaking, tidying

2166589

* create a new instance of ObjCTypeEncodingParser for each use (sha…

37576ab

…red parser object isn't thread safe) * added `objc.auto()` convenience function, equivalent to `@autoreleasepool {...}` * tidied comments

API tweaks

78c3df3

Merge branch 'master' of https://github.com/hhas/objc

d673d79

# Conflicts: # src/instance.js # src/objctypes.js # test/test-inout.js

remove stray crud that broke package.json

9bb5c34

bugfix in objctypes

59b6d12

updated version to 0.24.1

1e809f2

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

revised API and implementation #37

revised API and implementation #37

hhas commented Mar 10, 2022

revised API and implementation #37

Are you sure you want to change the base?

revised API and implementation #37

Conversation

hhas commented Mar 10, 2022