Mirror/zeek
1
0
Fork 0
mirror of https://github.com/zeek/zeek.git synced 2025-10-02 06:38:20 +00:00
Code Issues Projects Releases Packages Wiki Activity Actions 1

Fix some clang-tidy findings in generated BIF code

Browse source
This commit is contained in:
Tim Wojtulewicz 2025-08-13 13:26:54 -07:00
parent f194b14727
commit a1f5f0a40e
5 changed files with 16 additions and 14 deletions
Download patch file Download diff file Expand all files Collapse all files

6
src/strings.bif
View file

@ -1251,7 +1251,7 @@ function hexdump%(data_str: string%) : string
function reverse%(str: string%) : string function reverse%(str: string%) : string
%{ %{
string s = str->ToStdString(); string s = str->ToStdString();
reverse(s.begin(), s.end()); std::ranges::reverse(s);
return zeek::make_intrusive<zeek::StringVal>(s.length(), (const char*)s.c_str()); return zeek::make_intrusive<zeek::StringVal>(s.length(), (const char*)s.c_str());
%} %}
@ -1308,8 +1308,8 @@ static int64_t do_find_str(zeek::StringVal* str, zeek::StringVal* sub, int64_t s
if ( ! case_sensitive ) if ( ! case_sensitive )
{ {
transform(s.begin(), s.end(), s.begin(), ::tolower); std::ranges::transform(s, s.begin(), ::tolower);
transform(sb.begin(), sb.end(), sb.begin(), ::tolower); std::ranges::transform(sb, sb.begin(), ::tolower);
} }
if ( rfind ) if ( rfind )

2
src/telemetry/telemetry_functions.bif
View file

@ -73,7 +73,7 @@ bool labels_valid(std::span<const zeek::telemetry::LabelView> labels,
return std::find(keys.begin(), keys.end(), x.first) != keys.end(); return std::find(keys.begin(), keys.end(), x.first) != keys.end();
}; };
return labels.size() == label_names.size() return labels.size() == label_names.size()
&& std::all_of(labels.begin(), labels.end(), key_in_label_names); && std::ranges::all_of(labels, key_in_label_names);
} }
template <class IntOrDbl> template <class IntOrDbl>

12
src/zeek.bif
View file

@ -17,6 +17,7 @@
#include <cstring> #include <cstring>
#include <ctime> #include <ctime>
#include <vector> #include <vector>
#include <numbers>
#include "zeek/digest.h" #include "zeek/digest.h"
#include "zeek/Reporter.h" #include "zeek/Reporter.h"
@ -4222,12 +4223,13 @@ function blocking_lookup_hostname%(host: string%) : addr_set
## .. zeek:see:: haversine_distance_ip ## .. zeek:see:: haversine_distance_ip
function haversine_distance%(lat1: double, long1: double, lat2: double, long2: double%): double function haversine_distance%(lat1: double, long1: double, lat2: double, long2: double%): double
%{ %{
const double PI = 3.14159; constexpr double RADIUS = 3958.7615; // Mean radius of the Earth in miles.
const double RADIUS = 3958.8; // Earth's radius in miles. constexpr double PI_360 = std::numbers::pi / 360.0;
constexpr double PI_180 = std::numbers::pi / 180.0;
double s1 = sin((lat2 - lat1) * PI/360); double s1 = pow(sin((lat2 - lat1) * PI_360), 2);
double s2 = sin((long2 - long1) * PI/360); double s2 = pow(sin((long2 - long1) * PI_360), 2);
double a = s1 * s1 + cos(lat1 * PI/180) * cos(lat2 * PI/180) * s2 * s2; double a = s1 + (cos(lat1 * PI_180) * cos(lat2 * PI_180) * s2);
double distance = 2 * RADIUS * asin(sqrt(a)); double distance = 2 * RADIUS * asin(sqrt(a));
return zeek::make_intrusive<zeek::DoubleVal>(distance); return zeek::make_intrusive<zeek::DoubleVal>(distance);

8
testing/btest/Baseline/bifs.haversine_distance/out
View file

@ -1,8 +1,8 @@
### BTest baseline data generated by btest-diff. Do not edit. Use "btest -U/-u" to update. Requires BTest >= 0.63. ### BTest baseline data generated by btest-diff. Do not edit. Use "btest -U/-u" to update. Requires BTest >= 0.63.
5.8481e+03 5.8480e+03
5.8481e+03 5.8480e+03
1.9193e-02 1.9193e-02
1.5136e-02 1.5124e-02
9.2419e-01 9.0762e-01
1.2437e+04 1.2437e+04
1.2437e+04 1.2437e+04

2
testing/btest/bifs/haversine_distance.zeek
View file

@ -18,7 +18,7 @@ event zeek_init()
test(.0001388889, 0, -.0001388889, 0); test(.0001388889, 0, -.0001388889, 0);
# Distance of one second of longitude (crossing the prime meridian). # Distance of one second of longitude (crossing the prime meridian).
test(38, 0.000138999, 38, -0.000138999); test(38, 0.000138889, 38, -0.000138889);
# Distance of one minute of longitude (test extreme longitude values). # Distance of one minute of longitude (test extreme longitude values).
test(38, 180, 38, -179.98333); test(38, 180, 38, -179.98333);