Skip to content
This repository has been archived by the owner on Sep 23, 2023. It is now read-only.

fix panic handler in NewCompressor #987

Merged
merged 1 commit into from
May 8, 2023

Conversation

elee1766
Copy link
Contributor

@elee1766 elee1766 commented May 6, 2023

currently if newdecompressor panics, it will return nil, nil

this should make it return an error instead

@elee1766 elee1766 changed the title change behavior of panic handler in newdecompressor fix panic handler in NewCompressor May 6, 2023
@AskAlexSharov AskAlexSharov added this pull request to the merge queue May 8, 2023
Merged via the queue into ledgerwatch:main with commit 7b16970 May 8, 2023
calmbeing pushed a commit to calmbeing/bsc-erigon-lib that referenced this pull request Jul 10, 2023
currently if newdecompressor panics, it will return nil, nil

this should make it return an error instead
calmbeing pushed a commit to calmbeing/bsc-erigon-lib that referenced this pull request Jul 12, 2023
currently if newdecompressor panics, it will return nil, nil

this should make it return an error instead
blxdyx pushed a commit to blxdyx/bsc-erigon-lib that referenced this pull request Sep 13, 2023
currently if newdecompressor panics, it will return nil, nil

this should make it return an error instead
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants