LongYinan
a0522986b3
Merge pull request #1331 from napi-rs/pass-in-buffer-leak
...
fix(napi): should also delete the reference while dropping the Buffer
2022-10-02 12:50:33 +08:00
LongYinan
6ff69a1dab
test(memory-testing): adjust buffer tests
2022-10-02 12:28:56 +08:00
LongYinan
c76be2f1e0
ci: do not throw if docker container stop failed
2022-10-02 11:14:52 +08:00
LongYinan
d3fc689269
ci: remove clear cargo cache in MSRV job
2022-10-02 10:31:36 +08:00
LongYinan
47de6301ee
fix(napi): should also delete the reference while dropping the Buffer
2022-10-02 10:14:25 +08:00
messense
4279291f4b
feat(cli): parse Cargo.toml using cargo metadata ( #1330 )
2022-10-01 22:06:14 +08:00
user.tax
e54c37a0b1
feat(napi): add support for Vec<(std::string::String, u16)> and some other small change ( #1320 )
2022-09-20 12:13:07 +08:00
user.tax
0d49b45ea9
feat(napi): add impl<T: Into<Vec<u8>>> From<T> for Uint8Array ( #1317 )
...
Co-authored-by: any <any@user.tax>
2022-09-16 23:15:29 +08:00
LongYinan
ea18170779
fix(napi): propagation error in function call ( #1315 )
2022-09-14 19:30:43 +08:00
Devon Govett
5ba70b0e1a
fix(napi): improve error propagation ( #1303 )
2022-09-14 17:03:11 +08:00
LongYinan
d5a6445bee
chore(cli): increase ava timeout ( #1314 )
2022-09-14 17:01:53 +08:00
messense
30accad110
Add arm64 macOS CI ( #1313 )
2022-09-14 10:34:04 +08:00
Wodann
f7c26cccbf
fix(cli): custom Cargo (build) target directories ( #1300 )
2022-09-09 18:54:53 +08:00
Hana
22adc885d6
chore: fix ci for forked repo ( #1302 )
2022-09-09 18:54:23 +08:00
LongYinan
f3c203d46e
Release independent packages
...
napi@2.9.1
napi-derive@2.9.1
2022-09-08 17:27:35 +08:00
Dennis Duda
fc63ba8b52
fix(napi): some of the unsoundness in Buffer ( #1294 )
...
* fix leaked napi refcount in `Buffer` when cloning
Cloning unconditionally increased the refcount in `Buffer::clone`, but only called `napi_reference_unref` on dropping the last Buffer (the one with `strong_count == 1`). This means that the refcount will never drop back to zero after cloning, leaking the Buffer.
This commit changes it to also unconditionally unref the buffer.
* fix multiple sources of UB in `Buffer`
- `slice::from_raw_parts` may never be created with a null pointer, but `napi_get_buffer_info` was not sufficiently checked → UB when passing an empty Buffer
- `&'static mut [u8],` is invalid, as it certainly doesn't live for `'static`
Switching to `NonNull<u8>` and a `len` field fixes both of these.
- I also don't really understand how the `impl ToNapiValue for &mut Buffer` could have been sound. It creates an entirely new `Arc`, but reuses the same `Vec` allocation, leading to... a double free of the `Vec` on drop? I have replaced it with a simple call to `clone` instead.
* remove overcomplicated bool and drop impl
As far as I can tell, by just removing the bool and letting the drop code do its thing we clean up correctly in all cases. Because `napi_create_external_buffer` gets an owned `Buffer` attached to it via the Box, we can rely on `from_raw` retrieving it in the `drop_buffer` function.
2022-09-05 13:04:43 +08:00
mat-if
26f6c926d3
chore(napi-derive): pin minimum usable version of dependency syn ( #1293 )
2022-09-01 10:50:48 +08:00
LongYinan
76fd7b7526
Merge pull request #1291 from napi-rs/fix-this-types
...
fix(napi-derive): should not generate this types for Constructor/Getter/Setter
2022-08-30 17:04:23 +08:00
LongYinan
32a039aad0
fix(napi-derive): should not generate this types for Constructor/Getter/Setter
2022-08-30 16:39:51 +08:00
LongYinan
7e3b1ef330
Merge pull request #1290 from napi-rs/zst
...
fix(napi): remove previous reference if value_ref existed
2022-08-25 22:17:47 +08:00
LongYinan
525d881590
fix(napi): remove previous reference if value_ref existed
...
Usually happens while using ZST
2022-08-25 21:51:06 +08:00
LongYinan
767c040d94
Release independent packages
...
- napi@2.9.0
- napi-derive@2.9.0
2022-08-23 22:27:53 +08:00
LongYinan
58f0d82554
Merge pull request #1287 from napi-rs/upgrade-dependencies
...
chore: upgrade npm dependencies
2022-08-23 21:00:46 +08:00
LongYinan
184c4af588
chore: upgrade npm dependencies
2022-08-23 20:18:25 +08:00
LongYinan
4153c03a9f
Merge pull request #1286 from napi-rs/f32-to-napi-value
...
feat(napi): implement ToNapiValue for f32
2022-08-23 17:03:01 +08:00
LongYinan
f7c00c9a90
feat(napi): implement as_unknown and validate for Either types ( #1285 )
2022-08-23 17:02:51 +08:00
LongYinan
49ec8917df
feat(napi): implement as_object and validate for ClassInstance ( #1284 )
2022-08-23 17:02:41 +08:00
LongYinan
36275438ad
chore(napi): misc cleanup
2022-08-23 16:18:08 +08:00
LongYinan
03e80361d9
feat(napi): implement ToNapiValue for f32
2022-08-23 16:18:08 +08:00
messense
99e17c7294
fix(napi): segfault when ThreadsafeFunction's callback closure captures data ( #1281 )
2022-08-20 22:40:26 +08:00
LongYinan
b7a3103f0c
feat(napi-derive): catch_unwind attribute ( #1280 )
2022-08-19 23:36:36 +08:00
Fy
bc69e15efb
fix(napi-derive): fix macro expansion naming shadow ( #1278 )
...
Co-authored-by: fengyu <fengyu.shelby@bytedance.com>
2022-08-19 11:05:09 +08:00
LongYinan
5ba603459b
Release independent packages
...
napi@2.8.0
napi-derive@2.8.0
2022-08-17 23:12:05 +08:00
LongYinan
40d46f4b2a
Merge pull request #1275 from napi-rs/anyhow
...
feat(napi): `error_anyhow` feature
2022-08-17 23:10:40 +08:00
iuser
13996c1864
feat(napi): error_anyhow feature
2022-08-17 22:50:52 +08:00
LongYinan
c4eb51b509
Merge pull request #1274 from napi-rs/weak-reference
...
feat(napi): add get and get_mut method on `WeakReference`
2022-08-17 22:12:02 +08:00
LongYinan
1cd4fda01e
feat(napi): add get and get_mut method on WeakReference
2022-08-17 21:29:03 +08:00
LongYinan
242b292079
Merge pull request #1273 from napi-rs/do-not-restrict-this
...
feat(napi-derive): support inject This<Value> into raw function
2022-08-17 18:46:53 +08:00
LongYinan
528b1d21cb
style: clippy fix
2022-08-17 18:16:01 +08:00
LongYinan
0ef482c6ca
feat(napi-derive): support inject This<Value> into raw function
2022-08-17 18:16:00 +08:00
LongYinan
5030cfb8fb
Merge pull request #1272 from napi-rs/support-instanceof
...
feat(napi-derive): implement instance_of for Class
2022-08-17 16:00:46 +08:00
LongYinan
711372e175
feat(napi-derive): implement instance_of for Class
2022-08-17 15:18:43 +08:00
LongYinan
8d0045f5b6
Merge pull request #1270 from napi-rs/custom-finalize
...
feat(napi): allow implement custom finalize logic for Class
2022-08-17 14:25:51 +08:00
LongYinan
2385b52a72
feat(napi): allow implement custom finalize logic for Class
2022-08-17 13:24:40 +08:00
LongYinan
304bddeae9
chore: publish
...
- @napi-rs/cli@2.11.4
2022-08-13 01:11:37 +08:00
LongYinan
6bd682279d
Merge pull request #1263 from xhuz/main
...
feat(napi): support rust array to js array
2022-08-12 21:04:43 +08:00
huzz
fd191a4586
feat(napi): support rust array to js array
2022-08-12 17:53:36 +08:00
LongYinan
8784ed9459
Merge pull request #1265 from liby/feature/update-ci-template
...
chore(napi): use recommended `vmactions/freebsd-vm@v0`
2022-08-12 17:40:10 +08:00
LongYinan
2d2e4425b4
Merge pull request #1266 from napi-rs/zig-cross-fix
...
fix(cli): zig cross to *-apple-darwin target
2022-08-12 17:39:49 +08:00
LongYinan
d9f5245432
ci: fix FreeBSD check CI
2022-08-12 16:56:45 +08:00
