min
/
boring2
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

Boring doesn't use function codes

This commit is contained in:
Kornel 2025-06-13 12:42:33 +01:00 committed by Kornel
parent c5045fb6b4
commit 8d77a5d40e
1 changed files with 4 additions and 16 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

20
boring/src/error.rs
View File

@ -208,20 +208,9 @@ impl Error {
ffi::ERR_GET_LIB(self.code) ffi::ERR_GET_LIB(self.code)
} }
/// Returns the name of the function reporting the error. /// Returns `None`. Boring doesn't use function codes.
#[must_use]
pub fn function(&self) -> Option<&'static str> { pub fn function(&self) -> Option<&'static str> {
if self.is_internal() { None
return None;
}
unsafe {
let cstr = ffi::ERR_func_error_string(self.code);
if cstr.is_null() {
return None;
}
let bytes = CStr::from_ptr(cstr as *const _).to_bytes();
str::from_utf8(bytes).ok()
}
} }
/// Returns the reason for the error. /// Returns the reason for the error.
@ -256,6 +245,8 @@ impl Error {
} }
/// Returns the line in the source file which encountered the error. /// Returns the line in the source file which encountered the error.
///
/// 0 if unknown
#[allow(clippy::unnecessary_cast)] #[allow(clippy::unnecessary_cast)]
#[must_use] #[must_use]
pub fn line(&self) -> u32 { pub fn line(&self) -> u32 {
@ -299,9 +290,6 @@ impl fmt::Debug for Error {
builder.field("library", &library); builder.field("library", &library);
} }
builder.field("library_code", &self.library_code()); builder.field("library_code", &self.library_code());
if let Some(function) = self.function() {
builder.field("function", &function);
}
if let Some(reason) = self.reason() { if let Some(reason) = self.reason() {
builder.field("reason", &reason); builder.field("reason", &reason);
} }