From a269766c27ea6d31860b165ef5cb045be323cf16 Mon Sep 17 00:00:00 2001
From: konsti <konstin@mailbox.org>
Date: Tue, 20 Feb 2024 18:32:01 +0100
Subject: [PATCH] If-let instead of unwrap (PR #1746 follow-up) (#1770)

Solves: https://github.com/astral-sh/uv/pull/1746#discussion_r1496139629
---
 crates/pep508-rs/src/lib.rs | 21 +++++++++++----------
 1 file changed, 11 insertions(+), 10 deletions(-)

diff --git a/crates/pep508-rs/src/lib.rs b/crates/pep508-rs/src/lib.rs
index e3b53747c..ede41701c 100644
--- a/crates/pep508-rs/src/lib.rs
+++ b/crates/pep508-rs/src/lib.rs
@@ -983,16 +983,17 @@ fn parse(cursor: &mut Cursor, working_dir: Option<&Path>) -> Result<Requirement,
     cursor.eat_whitespace();
     if let Some((pos, char)) = cursor.next() {
         if let Some(VersionOrUrl::Url(url)) = requirement_kind {
-            // Unwrap safety: The `VerbatimUrl` we just parsed has a string source.
-            if url.given().unwrap().ends_with(';') && marker.is_none() {
-                return Err(Pep508Error {
-                    message: Pep508ErrorSource::String(
-                        "Missing space before ';', the end of the URL is ambiguous".to_string(),
-                    ),
-                    start: requirement_end - ';'.len_utf8(),
-                    len: ';'.len_utf8(),
-                    input: cursor.to_string(),
-                });
+            if let Some(given) = url.given() {
+                if given.ends_with(';') && marker.is_none() {
+                    return Err(Pep508Error {
+                        message: Pep508ErrorSource::String(
+                            "Missing space before ';', the end of the URL is ambiguous".to_string(),
+                        ),
+                        start: requirement_end - ';'.len_utf8(),
+                        len: ';'.len_utf8(),
+                        input: cursor.to_string(),
+                    });
+                }
             }
         }
         let message = if marker.is_none() {