Closed edwardmp closed 1 year ago
@mwcw could you maybe take a look? Unless you have feedback I'd like to merge this upstream as soon as possible
I am looking at it today.
MW
Hi,
Sorry I am going to be unable to get to finish it today, I have had a look, I need to get hold of some more test vectors.
MW
@mwcw no problem, keep me posted
@mwcw what do you mean by more test vectors? Isn't the encryption/decryption test I added enough?
Hi,
Thanks this has been merged.
MW
@mwcw great, but it doesn't actually seem to have merged?
Sorry, I am in UTC+11 and I didn't finish it off in time.
Hi @mwcw , any way I can help prepare for the next release so I no longer need to use my own fork to have this functionality ;) ?
Prior to this commit, there was no mechanism to provide encoding parameters. This commit adds that ability.
Any feedback is welcome.
Fixes #174