Skip to content

Commit

Permalink
cleanup
Browse files Browse the repository at this point in the history
  • Loading branch information
realFlowControl committed Jan 8, 2024
1 parent ecc4a5d commit 1ab66ae
Show file tree
Hide file tree
Showing 3 changed files with 4 additions and 2 deletions.
2 changes: 1 addition & 1 deletion profiling/build.rs
Original file line number Diff line number Diff line change
Expand Up @@ -100,7 +100,7 @@ fn build_zend_php_ffis(
"../zend_abstract_interface/sandbox/php8/sandbox.c"
};

// Profiling only needs config and its dependencies.
// Profiling only needs config, exceptions and its dependencies.
let zai_c_files = [
"../zend_abstract_interface/config/config_decode.c",
"../zend_abstract_interface/config/config_ini.c",
Expand Down
2 changes: 2 additions & 0 deletions profiling/src/php_ffi.h
Original file line number Diff line number Diff line change
Expand Up @@ -18,6 +18,8 @@
// Profiling needs ZAI config for INI support.
#include <config/config.h>

// Exception profiling needs to get the message of the exception (and ZAI
// provides `zai_exception_message()`)
#include <exceptions/exceptions.h>

// Used to communicate strings from C -> Rust.
Expand Down
2 changes: 1 addition & 1 deletion profiling/src/profiling/mod.rs
Original file line number Diff line number Diff line change
Expand Up @@ -754,7 +754,7 @@ impl Profiler {

labels.push(Label {
key: "exception message",
value: LabelValue::Str(message.clone().into()),
value: LabelValue::Str(message.into()),
});

let n_labels = labels.len();
Expand Down

0 comments on commit 1ab66ae

Please sign in to comment.