Skip to content

fix: bug in slebEncode#368

Merged
hansl merged 3 commits intomasterfrom
leb-fix
Feb 11, 2020
Merged

fix: bug in slebEncode#368
hansl merged 3 commits intomasterfrom
leb-fix

Conversation

@chenyan-dfinity
Copy link
Contributor

lebEncode and slebEncode for positive numbers are not the same. Refactor both functions to make the difference clear.

@chenyan-dfinity chenyan-dfinity marked this pull request as ready for review February 9, 2020 21:11
@chenyan-dfinity chenyan-dfinity requested a review from a team as a code owner February 9, 2020 21:11
@hansl hansl merged commit d4ef458 into master Feb 11, 2020
dfinity-bot added a commit that referenced this pull request Aug 28, 2020
## Changelog for advisory-db:
Branch: 
Commits: [rustsec/advisory-db@47d12507...c341943a](rustsec/advisory-db@47d1250...c341943)

* [`91aeb3b0`](rustsec/advisory-db@91aeb3b) Suggest lz-fear as an alternative to lz4-compress
* [`002af101`](rustsec/advisory-db@002af10) failure: add informational "unmaintained" advisory ([RustSec/advisory-db⁠#368](http://r.duckduckgo.com/l/?uddg=https://github.com/RustSec/advisory-db/issues/368))
* [`1d3ca115`](rustsec/advisory-db@1d3ca11) Assigned RUSTSEC-2020-0036 to failure ([RustSec/advisory-db⁠#369](http://r.duckduckgo.com/l/?uddg=https://github.com/RustSec/advisory-db/issues/369))
* [`df8a0c46`](rustsec/advisory-db@df8a0c4) RUSTSEC-2020-0036: add fehler ([RustSec/advisory-db⁠#370](http://r.duckduckgo.com/l/?uddg=https://github.com/RustSec/advisory-db/issues/370))
mergify bot pushed a commit that referenced this pull request Aug 28, 2020
## Changelog for advisory-db:
Branch: 
Commits: [rustsec/advisory-db@47d12507...c341943a](rustsec/advisory-db@47d1250...c341943)

* [`91aeb3b0`](rustsec/advisory-db@91aeb3b) Suggest lz-fear as an alternative to lz4-compress
* [`002af101`](rustsec/advisory-db@002af10) failure: add informational "unmaintained" advisory ([RustSec/advisory-db⁠#368](http://r.duckduckgo.com/l/?uddg=https://github.com/RustSec/advisory-db/issues/368))
* [`1d3ca115`](rustsec/advisory-db@1d3ca11) Assigned RUSTSEC-2020-0036 to failure ([RustSec/advisory-db⁠#369](http://r.duckduckgo.com/l/?uddg=https://github.com/RustSec/advisory-db/issues/369))
* [`df8a0c46`](rustsec/advisory-db@df8a0c4) RUSTSEC-2020-0036: add fehler ([RustSec/advisory-db⁠#370](http://r.duckduckgo.com/l/?uddg=https://github.com/RustSec/advisory-db/issues/370))
gobengo pushed a commit that referenced this pull request Sep 1, 2020
## Changelog for advisory-db:
Branch: 
Commits: [rustsec/advisory-db@47d12507...c341943a](rustsec/advisory-db@47d1250...c341943)

* [`91aeb3b0`](rustsec/advisory-db@91aeb3b) Suggest lz-fear as an alternative to lz4-compress
* [`002af101`](rustsec/advisory-db@002af10) failure: add informational "unmaintained" advisory ([RustSec/advisory-db⁠#368](http://r.duckduckgo.com/l/?uddg=https://github.com/RustSec/advisory-db/issues/368))
* [`1d3ca115`](rustsec/advisory-db@1d3ca11) Assigned RUSTSEC-2020-0036 to failure ([RustSec/advisory-db⁠#369](http://r.duckduckgo.com/l/?uddg=https://github.com/RustSec/advisory-db/issues/369))
* [`df8a0c46`](rustsec/advisory-db@df8a0c4) RUSTSEC-2020-0036: add fehler ([RustSec/advisory-db⁠#370](http://r.duckduckgo.com/l/?uddg=https://github.com/RustSec/advisory-db/issues/370))
@lwshang lwshang deleted the leb-fix branch July 29, 2022 18:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants