Merge remote-tracking branch 'origin/topic/johanna/remove-build-unique'

* origin/topic/johanna/remove-build-unique:
  Replace build_unique with make_unique
This commit is contained in:
Jon Siwek 2019-10-29 09:39:10 -07:00
commit 7b9a27c96a
6 changed files with 21 additions and 14 deletions

12
CHANGES
View file

@ -1,4 +1,16 @@
3.1.0-dev.227 | 2019-10-29 09:39:10 -0700
* Replace build_unique with make_unique (Johanna Amann, Corelight)
This was a rarely used convenience function from when we did not yet
have c++17 support.
* GH-626: Revert "Fix compilation on OS-X." (Johanna Amann, Corelight)
Reverts workaround in cde28074a169212aa8f38fdac225ecbeac4e642d
which depended on C++14 features at a time when we used only C++11.
3.1.0-dev.222 | 2019-10-28 20:18:15 -0700 3.1.0-dev.222 | 2019-10-28 20:18:15 -0700
* Install cmake3 from EPEL on CentOS CI system (Jon Siwek, Corelight) * Install cmake3 from EPEL on CentOS CI system (Jon Siwek, Corelight)

View file

@ -1 +1 @@
3.1.0-dev.222 3.1.0-dev.227

@ -1 +1 @@
Subproject commit fa3b6009c78ef739b5e4f5f4daabe9d9f009b4e0 Subproject commit 6c2b36193e47490e61f22ce6de233af7ed3101b1

View file

@ -1,5 +1,7 @@
// See the file "COPYING" in the main distribution directory for copyright. // See the file "COPYING" in the main distribution directory for copyright.
#include <memory>
#include "OpaqueVal.h" #include "OpaqueVal.h"
#include "NetVar.h" #include "NetVar.h"
#include "Reporter.h" #include "Reporter.h"
@ -1073,7 +1075,7 @@ bool ParaglobVal::DoUnserialize(const broker::data& data)
try try
{ {
this->internal_paraglob = build_unique<paraglob::Paraglob>(std::move(iv)); this->internal_paraglob = std::make_unique<paraglob::Paraglob>(std::move(iv));
} }
catch (const paraglob::underflow_error& e) catch (const paraglob::underflow_error& e)
{ {
@ -1093,7 +1095,7 @@ Val* ParaglobVal::DoClone(CloneState* state)
{ {
try { try {
return new ParaglobVal return new ParaglobVal
(build_unique<paraglob::Paraglob>(this->internal_paraglob->serialize())); (std::make_unique<paraglob::Paraglob>(this->internal_paraglob->serialize()));
} }
catch (const paraglob::underflow_error& e) catch (const paraglob::underflow_error& e)
{ {

View file

@ -1,5 +1,7 @@
// See the file "COPYING" in the main distribution directory for copyright. // See the file "COPYING" in the main distribution directory for copyright.
#include <memory>
#include "zeek-config.h" #include "zeek-config.h"
#include "Var.h" #include "Var.h"
@ -510,7 +512,7 @@ void end_func(Stmt* body)
std::unique_ptr<function_ingredients> gather_function_ingredients(Scope* scope, Stmt* body) std::unique_ptr<function_ingredients> gather_function_ingredients(Scope* scope, Stmt* body)
{ {
auto ingredients = build_unique<function_ingredients>(); auto ingredients = std::make_unique<function_ingredients>();
ingredients->frame_size = scope->Length(); ingredients->frame_size = scope->Length();
ingredients->inits = scope->GetInits(); ingredients->inits = scope->GetInits();

View file

@ -563,15 +563,6 @@ void bro_strerror_r(int bro_errno, char* buf, size_t buflen);
*/ */
char* zeekenv(const char* name); char* zeekenv(const char* name);
/**
* Small convenience function. Does what std::make_unique does in C++14. Will not
* work on arrays.
*/
template <typename T, typename ... Args>
std::unique_ptr<T> build_unique (Args&&... args) {
return std::unique_ptr<T>(new T(std::forward<Args>(args)...));
}
/** /**
* Escapes bytes in a string that are not valid UTF8 characters with \xYY format. Used * Escapes bytes in a string that are not valid UTF8 characters with \xYY format. Used
* by the JSON writer and BIF methods. * by the JSON writer and BIF methods.