Skip to content

dleq: verify: return false instead of panic()ing on nil parameters - #641

Merged
bwesterb merged 1 commit into
mainfrom
bas/gw18
Jul 14, 2026
Merged

dleq: verify: return false instead of panic()ing on nil parameters#641
bwesterb merged 1 commit into
mainfrom
bas/gw18

Conversation

@bwesterb

@bwesterb bwesterb commented Jul 14, 2026 •
edited by devin-ai-integration Bot
Loading

Copy link
Copy Markdown
Member

@bwesterb bwesterb changed the title Bas/gw18 dleq: verify: return false instead of panic()ing on nil parameters Jul 14, 2026
devin-ai-integration[bot]
devin-ai-integration Bot reviewed Jul 14, 2026
View reviewed changes

@devin-ai-integration devin-ai-integration Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Devin Review found 2 potential issues.

View 2 additional findings in Devin Review.

Open in Devin Review

Comment thread zk/dleq/dleq.go
Comment thread zk/dleq/dleq.go
@bwesterb
bwesterb merged commit 0c3c386 into main Jul 14, 2026
14 checks passed
@bwesterb
bwesterb deleted the bas/gw18 branch July 14, 2026 20:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@devin-ai-integration devin-ai-integration[bot] devin-ai-integration[bot] left review comments
@cjpatton cjpatton cjpatton approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@bwesterb @cjpatton