Review fixes

Author: CraftSpider

crates/engine_bibtex/src/c_api/cite.rs

@@ -127,17 +127,6 @@ fn less_than(arg1: &CiteNumber, arg2: &CiteNumber) -> Ordering {
         let str2 = entries.strs(ptr2);
 
         Ord::cmp(str1, str2)
-        // for (){
-        //     match (char1, char2) {
-        //         (127, 127) => return arg1.cmp(arg2),
-        //         (127, _) => return Ordering::Less,
-        //         (_, 127) => return Ordering::Greater,
-        //         (char1, char2) if char1 != char2 => return char1.cmp(&char2),
-        //         _ => (),
-        //     }
-        //
-        //     char_ptr += 1;
-        // }
     })
 }
 

crates/engine_bibtex/src/c_api/exec.rs

@@ -69,7 +69,7 @@ pub struct ExecCtx {
     pub(crate) lit_stack: Box<XBuf<ExecVal>>,
     pub lit_stk_ptr: usize,
     pub mess_with_entries: bool,
-    /// Pointer to the current top of the string pool, used to optimized certain string operations
+    /// Pointer to the current top of the string pool, used to optimize certain string operations
     pub bib_str_ptr: StrNumber,
 }
 
@@ -227,8 +227,7 @@ pub fn bst_ex_warn_print(ctx: &ExecCtx, pool: &StringPool) -> Result<(), BibtexE
     }
 
     write_logs("\nwhile executing-");
-    // SAFETY: The `Bibtex` value pointed to by `ExecCtx` is guaranteed valid
-    bst_ln_num_print(unsafe { &*ctx.glbl_ctx }, pool)?;
+    bst_ln_num_print(ctx.glbl_ctx(), pool)?;
     mark_error();
     Ok(())
 }