Page MenuHomePhabricator
Feed Advanced Search

Aug 22 2023

sken updated sken.
Aug 22 2023, 14:53

Mar 21 2020

sken added a comment to D5490: [SECP256K1] turn NativeSecp256k1 class into a regular class.

It doesn't replace finalize, but its the recommended way of handling resources. IMO, anyone with java experience would understand this, and treat this class as such. we can make it clear on the release notes and the comment

Mar 21 2020, 02:32
sken updated the diff for D5490: [SECP256K1] turn NativeSecp256k1 class into a regular class.

requested changes

Mar 21 2020, 02:30

Mar 20 2020

sken updated the diff for D5490: [SECP256K1] turn NativeSecp256k1 class into a regular class.

Forgot to add this last minute change

Mar 20 2020, 22:54
sken updated the diff for D5490: [SECP256K1] turn NativeSecp256k1 class into a regular class.

make requested changes and implement Autocloseable into the class

Mar 20 2020, 22:52
sken added inline comments to D5490: [SECP256K1] turn NativeSecp256k1 class into a regular class.
Mar 20 2020, 22:51

Mar 19 2020

sken updated the diff for D5490: [SECP256K1] turn NativeSecp256k1 class into a regular class.

Make requested changes

Mar 19 2020, 19:36
sken added a comment to D5490: [SECP256K1] turn NativeSecp256k1 class into a regular class.

I responded to some of those comments and made the requested changes regardless. we can roll some of them back. as for isEnabled name im open to suggestions, i cant think of something more concise ;). Also, i added isValid to indicate the instance has a valid context. as you said, users dont need to know what context is, only if it works or it doesnt

Mar 19 2020, 19:33
sken updated the diff for D5490: [SECP256K1] turn NativeSecp256k1 class into a regular class.

clean all resources in test

Mar 19 2020, 03:26
sken updated the diff for D5490: [SECP256K1] turn NativeSecp256k1 class into a regular class.

make requested changes and add proposed ones

Mar 19 2020, 02:30
sken updated the diff for D5490: [SECP256K1] turn NativeSecp256k1 class into a regular class.

make requested changes and add proposed ones

Mar 19 2020, 02:05
sken updated the diff for D5490: [SECP256K1] turn NativeSecp256k1 class into a regular class.

rolling back diff (hopefully)

Mar 19 2020, 01:40
sken added a comment to D5490: [SECP256K1] turn NativeSecp256k1 class into a regular class.

arc diff just did somethign strange to that py file. is there a way to revert that?

Mar 19 2020, 01:28
sken updated the diff for D5490: [SECP256K1] turn NativeSecp256k1 class into a regular class.

make requested changes and also add the proposed ones

Mar 19 2020, 01:25
sken added a comment to D5490: [SECP256K1] turn NativeSecp256k1 class into a regular class.

the context is set to -1 at "cleanup" (line 200~) when the context in the native lib is destroyed. We can set it up to -1 before assigning it at the constructor call. that way we can handle the case where the exception thrown at construction is caught for some dumb reason. Perhaps we should also check if the context is not -1 before every lib call. Theres also fixing cloneContext, which IMO should result in returning a new NativeSecp256k1 class with the cloned native lib context in, but i was hoping to fix that on another ticket

Mar 19 2020, 01:02

Mar 18 2020

sken updated the diff for D5490: [SECP256K1] turn NativeSecp256k1 class into a regular class.

Remove Secp256k1Context and move the loading to NativeSecp256k1

Mar 18 2020, 06:28
sken added inline comments to D5490: [SECP256K1] turn NativeSecp256k1 class into a regular class.
Mar 18 2020, 06:14
sken updated the diff for D5490: [SECP256K1] turn NativeSecp256k1 class into a regular class.

Roll back formating changes

Mar 18 2020, 06:07

Mar 17 2020

sken updated the diff for D5490: [SECP256K1] turn NativeSecp256k1 class into a regular class.

checking if library is enabled, otherwise throw exception

Mar 17 2020, 21:00
sken added inline comments to D5490: [SECP256K1] turn NativeSecp256k1 class into a regular class.
Mar 17 2020, 16:54
sken added a comment to D5490: [SECP256K1] turn NativeSecp256k1 class into a regular class.

The getContext used to have the benefit of checking that the context is properly initialized. Now there is no such check, so if cleanup is called, then something else is, you'll get an horrible crash instead. I do not think this is somethign java devs expect.

Mar 17 2020, 16:40
sken updated the summary of D5490: [SECP256K1] turn NativeSecp256k1 class into a regular class.
Mar 17 2020, 06:22
sken updated the summary of D5490: [SECP256K1] turn NativeSecp256k1 class into a regular class.
Mar 17 2020, 03:30
sken updated the diff for D5490: [SECP256K1] turn NativeSecp256k1 class into a regular class.

Fix spacing issue

Mar 17 2020, 03:26
sken updated the diff for D5490: [SECP256K1] turn NativeSecp256k1 class into a regular class.

fixing typo

Mar 17 2020, 03:22
sken created D5490: [SECP256K1] turn NativeSecp256k1 class into a regular class.
Mar 17 2020, 03:18