clippy

Hendrik van Antwerpen · Hendrik van Antwerpen · commit a83b25071373 · 2024-10-04T17:39:07.000+02:00
diff --git a/crates/bpe-openai/build.rs b/crates/bpe-openai/build.rs
@@ -23,10 +23,11 @@ fn main() {
 }
 
 fn serialize_tokens(name: &str, bpe: &CoreBPE, num_tokens: usize, hash_factor: u64) {
-    let mut path = PathBuf::from(env::var("OUT_DIR").unwrap());
+    let mut path = PathBuf::from(env::var("OUT_DIR").expect("OUT_DIR is set during build"));
     path.push(format!("bpe_{name}.dict"));
-    let file = File::create(path).unwrap();
+    let file = File::create(path).expect("can create output file");
     let mut serializer = rmp_serde::Serializer::new(file);
     let bpe = BytePairEncoding::from_tiktoken(bpe, num_tokens, Some(hash_factor));
-    bpe.serialize(&mut serializer).unwrap();
+    bpe.serialize(&mut serializer)
+        .expect("serialization succeeds");
 }

Original file line number	Diff line number	Diff line change
`@@ -23,10 +23,11 @@ fn main() {`
`23`	`23`	`}`
`24`	`24`
`25`	`25`	`fn serialize_tokens(name: &str, bpe: &CoreBPE, num_tokens: usize, hash_factor: u64) {`
`26`		`- let mut path = PathBuf::from(env::var("OUT_DIR").unwrap());`
	`26`	`+ let mut path = PathBuf::from(env::var("OUT_DIR").expect("OUT_DIR is set during build"));`
`27`	`27`	`path.push(format!("bpe_{name}.dict"));`
`28`		`- let file = File::create(path).unwrap();`
	`28`	`+ let file = File::create(path).expect("can create output file");`
`29`	`29`	`let mut serializer = rmp_serde::Serializer::new(file);`
`30`	`30`	`let bpe = BytePairEncoding::from_tiktoken(bpe, num_tokens, Some(hash_factor));`
`31`		`- bpe.serialize(&mut serializer).unwrap();`
	`31`	`+ bpe.serialize(&mut serializer)`
	`32`	`+ .expect("serialization succeeds");`
`32`	`33`	`}`