mirror of
https://github.com/zeek/zeek.git
synced 2025-10-02 14:48:21 +00:00
Merge remote-tracking branch 'origin/topic/bbannier/coverity-fixes'
* origin/topic/bbannier/coverity-fixes: Prefer `std::move` over copy
This commit is contained in:
commit
31f51f7a87
4 changed files with 8 additions and 3 deletions
4
CHANGES
4
CHANGES
|
@ -1,3 +1,7 @@
|
||||||
|
8.0.0-dev.309 | 2025-06-02 10:17:14 +0200
|
||||||
|
|
||||||
|
* Prefer `std::move` over copy (Benjamin Bannier, Corelight)
|
||||||
|
|
||||||
8.0.0-dev.307 | 2025-06-02 10:12:14 +0200
|
8.0.0-dev.307 | 2025-06-02 10:12:14 +0200
|
||||||
|
|
||||||
* minor BTest maintenance updates for -O gen-C++ (Vern Paxson, Corelight)
|
* minor BTest maintenance updates for -O gen-C++ (Vern Paxson, Corelight)
|
||||||
|
|
2
VERSION
2
VERSION
|
@ -1 +1 @@
|
||||||
8.0.0-dev.307
|
8.0.0-dev.309
|
||||||
|
|
|
@ -1,5 +1,6 @@
|
||||||
%%{
|
%%{
|
||||||
#include <string>
|
#include <string>
|
||||||
|
#include <utility>
|
||||||
|
|
||||||
#include "zeek/IPAddr.h"
|
#include "zeek/IPAddr.h"
|
||||||
#include "zeek/cluster/Backend.h"
|
#include "zeek/cluster/Backend.h"
|
||||||
|
@ -218,7 +219,7 @@ function Cluster::__listen_websocket%(options: WebSocketServerOptions%): bool
|
||||||
}
|
}
|
||||||
|
|
||||||
struct ServerOptions server_options{
|
struct ServerOptions server_options{
|
||||||
listen_addr,
|
std::move(listen_addr),
|
||||||
static_cast<uint16_t>(options_rec->GetField<zeek::PortVal>("listen_port")->Port()),
|
static_cast<uint16_t>(options_rec->GetField<zeek::PortVal>("listen_port")->Port()),
|
||||||
};
|
};
|
||||||
|
|
||||||
|
|
|
@ -134,7 +134,7 @@ std::optional<zeek::logging::detail::LogWriteBatch> detail::BinarySerializationF
|
||||||
|
|
||||||
fmt.EndRead();
|
fmt.EndRead();
|
||||||
|
|
||||||
return logging::detail::LogWriteBatch{header, std::move(records)};
|
return logging::detail::LogWriteBatch{std::move(header), std::move(records)};
|
||||||
}
|
}
|
||||||
|
|
||||||
#include "zeek/ID.h"
|
#include "zeek/ID.h"
|
||||||
|
|
Loading…
Add table
Add a link
Reference in a new issue