Skip to content

Commit

Permalink
cleanup: remove init in the configuration constructor
Browse files Browse the repository at this point in the history
This `init({});` in the falco_configuration constructor is unnecessary
since when we call the action `load_config`, if we don't have a config
file, we will call the same `init` we have just removed. This cleanup
avoids calling `falco_configuration::init` 2 times.

Signed-off-by: Andrea Terzolo <[email protected]>
  • Loading branch information
Andreagit97 authored and poiana committed Nov 21, 2023
1 parent 66a122d commit a1d5bb7
Showing 1 changed file with 0 additions and 1 deletion.
1 change: 0 additions & 1 deletion userspace/falco/configuration.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -78,7 +78,6 @@ falco_configuration::falco_configuration():
m_metrics_convert_memory_to_mb(true),
m_metrics_include_empty_values(false)
{
init({});
}

void falco_configuration::init(const std::vector<std::string>& cmdline_options)
Expand Down

0 comments on commit a1d5bb7

Please sign in to comment.