If we can eliminate .primary(), then we can use that name for this function... Maybe we should do that now anyway. We want to encourage people to use the fallible option anyway. #1828
Comments
larseggert
added a commit
to larseggert/neqo
that referenced
this issue
Apr 16, 2024
And deal with the fallout. Fixes mozilla#1828 Fixes mozilla#1813
github-merge-queue bot
pushed a commit
that referenced
this issue
Apr 17, 2024
…le()` (#1833) * fix: `Paths::primary()` is now `Paths::primary_fallible()` And deal with the fallout. Fixes #1828 Fixes #1813 * Fix merge * Update neqo-transport/src/connection/mod.rs Co-authored-by: Martin Thomson <[email protected]> Signed-off-by: Lars Eggert <[email protected]> * Update neqo-transport/src/connection/mod.rs Co-authored-by: Martin Thomson <[email protected]> Signed-off-by: Lars Eggert <[email protected]> * Update neqo-transport/src/connection/mod.rs Co-authored-by: Martin Thomson <[email protected]> Signed-off-by: Lars Eggert <[email protected]> * Update neqo-transport/src/connection/mod.rs Co-authored-by: Martin Thomson <[email protected]> Signed-off-by: Lars Eggert <[email protected]> * Update neqo-transport/src/connection/mod.rs Co-authored-by: Martin Thomson <[email protected]> Signed-off-by: Lars Eggert <[email protected]> * Update neqo-transport/src/connection/mod.rs Co-authored-by: Martin Thomson <[email protected]> Signed-off-by: Lars Eggert <[email protected]> * Fixups --------- Signed-off-by: Lars Eggert <[email protected]> Co-authored-by: Martin Thomson <[email protected]>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Originally posted by @martinthomson in #1814 (comment)
The text was updated successfully, but these errors were encountered: