Skip to content

Commit

Permalink
update coda-oss and nitro (#526)
Browse files Browse the repository at this point in the history
* latest from coda-oss

* xml.lite changes from coda-oss

* need updates to Python bindings too

* trying to fine the right macro for SWIG

* more xml.lite updates from coda-oss

* "private" is part of name mangling

* test_xmlParser might be be running in coda-oss

* xml.lite tweaks from coda-oss

* coda-oss release 2021-12-13

* nitro release 2.10.7

* Squashed commit of the following:

commit 474b7f2
Author: Dan Smith <[email protected]>
Date:   Mon Dec 13 13:00:02 2021 -0500

    xml.lite changes that need to get back to coda-oss

commit c9c0ec3
Author: Dan Smith <[email protected]>
Date:   Mon Dec 13 11:45:39 2021 -0500

    Squashed commit of the following:

    commit 33bd170
    Author: Dan Smith <[email protected]>
    Date:   Mon Dec 13 11:44:54 2021 -0500

        nitro release 2.10.7

    commit ef19ad2
    Author: Dan Smith <[email protected]>
    Date:   Mon Dec 13 11:42:09 2021 -0500

        coda-oss release 2021-12-13

    commit 86c1856
    Author: Dan Smith <[email protected]>
    Date:   Fri Dec 10 11:57:26 2021 -0500

        xml.lite tweaks from coda-oss

    commit f21f79d
    Author: Dan Smith <[email protected]>
    Date:   Thu Dec 9 17:36:09 2021 -0500

        test_xmlParser might be be running in coda-oss

    commit 3526cd0
    Author: Dan Smith <[email protected]>
    Date:   Thu Dec 9 16:09:17 2021 -0500

        "private" is part of name mangling

    commit 351a45f
    Author: Dan Smith <[email protected]>
    Date:   Thu Dec 9 15:54:13 2021 -0500

        more xml.lite updates from coda-oss

    commit 1c72ba2
    Author: Dan Smith <[email protected]>
    Date:   Thu Dec 9 15:15:34 2021 -0500

        trying to fine the right macro for SWIG

    commit 8d6e11d
    Author: Dan Smith <[email protected]>
    Date:   Thu Dec 9 14:51:34 2021 -0500

        need updates to Python bindings too

    commit e208dd4
    Author: Dan Smith <[email protected]>
    Date:   Thu Dec 9 14:41:12 2021 -0500

        xml.lite changes from coda-oss

    commit 169df75
    Author: Dan Smith <[email protected]>
    Date:   Wed Dec 8 14:32:37 2021 -0500

        latest from coda-oss

commit 1a74d40
Author: Dan Smith <[email protected]>
Date:   Fri Dec 10 12:26:10 2021 -0500

    more use of new xml.lite infrastructure

commit 5d641a4
Author: Dan Smith <[email protected]>
Date:   Fri Dec 10 11:58:22 2021 -0500

    use new xml.lite APIs to simplify code

commit 01f811e
Author: Dan Smith <[email protected]>
Date:   Fri Dec 10 11:57:57 2021 -0500

    Squashed commit of the following:

    commit 86c1856
    Author: Dan Smith <[email protected]>
    Date:   Fri Dec 10 11:57:26 2021 -0500

        xml.lite tweaks from coda-oss

    commit f21f79d
    Author: Dan Smith <[email protected]>
    Date:   Thu Dec 9 17:36:09 2021 -0500

        test_xmlParser might be be running in coda-oss

    commit 3526cd0
    Author: Dan Smith <[email protected]>
    Date:   Thu Dec 9 16:09:17 2021 -0500

        "private" is part of name mangling

    commit 351a45f
    Author: Dan Smith <[email protected]>
    Date:   Thu Dec 9 15:54:13 2021 -0500

        more xml.lite updates from coda-oss

    commit 1c72ba2
    Author: Dan Smith <[email protected]>
    Date:   Thu Dec 9 15:15:34 2021 -0500

        trying to fine the right macro for SWIG

    commit 8d6e11d
    Author: Dan Smith <[email protected]>
    Date:   Thu Dec 9 14:51:34 2021 -0500

        need updates to Python bindings too

    commit e208dd4
    Author: Dan Smith <[email protected]>
    Date:   Thu Dec 9 14:41:12 2021 -0500

        xml.lite changes from coda-oss

    commit 169df75
    Author: Dan Smith <[email protected]>
    Date:   Wed Dec 8 14:32:37 2021 -0500

        latest from coda-oss

commit 25b83eb
Author: Dan Smith <[email protected]>
Date:   Fri Dec 10 09:06:38 2021 -0500

    Squashed commit of the following:

    commit 3f59a62
    Author: Dan Smith <[email protected]>
    Date:   Fri Dec 10 08:44:54 2021 -0500

        still can't get French XML working

    commit 1a826e5
    Author: Dan Smith <[email protected]>
    Date:   Thu Dec 9 17:37:11 2021 -0500

        Squashed commit of the following:

        commit f21f79d
        Author: Dan Smith <[email protected]>
        Date:   Thu Dec 9 17:36:09 2021 -0500

            test_xmlParser might be be running in coda-oss

        commit 3526cd0
        Author: Dan Smith <[email protected]>
        Date:   Thu Dec 9 16:09:17 2021 -0500

            "private" is part of name mangling

        commit 351a45f
        Author: Dan Smith <[email protected]>
        Date:   Thu Dec 9 15:54:13 2021 -0500

            more xml.lite updates from coda-oss

        commit 1c72ba2
        Author: Dan Smith <[email protected]>
        Date:   Thu Dec 9 15:15:34 2021 -0500

            trying to fine the right macro for SWIG

        commit 8d6e11d
        Author: Dan Smith <[email protected]>
        Date:   Thu Dec 9 14:51:34 2021 -0500

            need updates to Python bindings too

        commit e208dd4
        Author: Dan Smith <[email protected]>
        Date:   Thu Dec 9 14:41:12 2021 -0500

            xml.lite changes from coda-oss

        commit 169df75
        Author: Dan Smith <[email protected]>
        Date:   Wed Dec 8 14:32:37 2021 -0500

            latest from coda-oss

    commit f705be9
    Author: Dan Smith <[email protected]>
    Date:   Thu Dec 9 17:35:35 2021 -0500

        add overloads for other code

    commit d613234
    Author: Dan Smith <[email protected]>
    Date:   Thu Dec 9 16:57:55 2021 -0500

        option to disable XML validation for SICD too

    commit bc2c97e
    Author: Dan Smith <[email protected]>
    Date:   Thu Dec 9 16:09:44 2021 -0500

        Squashed commit of the following:

        commit 3526cd0
        Author: Dan Smith <[email protected]>
        Date:   Thu Dec 9 16:09:17 2021 -0500

            "private" is part of name mangling

        commit 351a45f
        Author: Dan Smith <[email protected]>
        Date:   Thu Dec 9 15:54:13 2021 -0500

            more xml.lite updates from coda-oss

        commit 1c72ba2
        Author: Dan Smith <[email protected]>
        Date:   Thu Dec 9 15:15:34 2021 -0500

            trying to fine the right macro for SWIG

        commit 8d6e11d
        Author: Dan Smith <[email protected]>
        Date:   Thu Dec 9 14:51:34 2021 -0500

            need updates to Python bindings too

        commit e208dd4
        Author: Dan Smith <[email protected]>
        Date:   Thu Dec 9 14:41:12 2021 -0500

            xml.lite changes from coda-oss

        commit 169df75
        Author: Dan Smith <[email protected]>
        Date:   Wed Dec 8 14:32:37 2021 -0500

            latest from coda-oss

    commit 4cd813b
    Author: Dan Smith <[email protected]>
    Date:   Thu Dec 9 15:16:01 2021 -0500

        Squashed commit of the following:

        commit 1c72ba2
        Author: Dan Smith <[email protected]>
        Date:   Thu Dec 9 15:15:34 2021 -0500

            trying to fine the right macro for SWIG

        commit 8d6e11d
        Author: Dan Smith <[email protected]>
        Date:   Thu Dec 9 14:51:34 2021 -0500

            need updates to Python bindings too

        commit e208dd4
        Author: Dan Smith <[email protected]>
        Date:   Thu Dec 9 14:41:12 2021 -0500

            xml.lite changes from coda-oss

        commit 169df75
        Author: Dan Smith <[email protected]>
        Date:   Wed Dec 8 14:32:37 2021 -0500

            latest from coda-oss

    commit 7b210f9
    Author: Dan Smith <[email protected]>
    Date:   Thu Dec 9 14:51:53 2021 -0500

        Squashed commit of the following:

        commit 8d6e11d
        Author: Dan Smith <[email protected]>
        Date:   Thu Dec 9 14:51:34 2021 -0500

            need updates to Python bindings too

        commit e208dd4
        Author: Dan Smith <[email protected]>
        Date:   Thu Dec 9 14:41:12 2021 -0500

            xml.lite changes from coda-oss

        commit 169df75
        Author: Dan Smith <[email protected]>
        Date:   Wed Dec 8 14:32:37 2021 -0500

            latest from coda-oss

    commit 4a4d621
    Author: Dan Smith <[email protected]>
    Date:   Thu Dec 9 14:47:21 2021 -0500

        trying to get the "french" SICD unit-test working

    commit d63e6ea
    Author: Dan Smith <[email protected]>
    Date:   Thu Dec 9 14:42:55 2021 -0500

        Squashed commit of the following:

        commit e208dd4
        Author: Dan Smith <[email protected]>
        Date:   Thu Dec 9 14:41:12 2021 -0500

            xml.lite changes from coda-oss

        commit 169df75
        Author: Dan Smith <[email protected]>
        Date:   Wed Dec 8 14:32:37 2021 -0500

            latest from coda-oss

    commit d1a7e6f
    Author: Dan Smith <[email protected]>
    Date:   Thu Dec 9 09:29:07 2021 -0500

        test_valid_six unittests now working

    commit d6fb257
    Author: Dan Smith <[email protected]>
    Date:   Wed Dec 8 17:45:26 2021 -0500

        trying to re-enable more unittests

    commit c787e98
    Author: Dan Smith <[email protected]>
    Date:   Wed Dec 8 16:54:28 2021 -0500

        unit-test breaks on Linux

    commit bd82787
    Author: Dan Smith <[email protected]>
    Date:   Wed Dec 8 15:25:31 2021 -0500

        Squashed commit of the following:

        commit c56ac71
        Author: Dan Smith <[email protected]>
        Date:   Wed Dec 8 15:23:00 2021 -0500

            merge in develop/SIDD-3.0

    commit e621466
    Author: Dan Smith <[email protected]>
    Date:   Wed Dec 8 15:08:04 2021 -0500

        merge in develop/8AMPI_PHSI

commit 67768ea
Author: Dan Smith <[email protected]>
Date:   Thu Dec 9 10:02:19 2021 -0500

    Squashed commit of the following:

    commit d1a7e6f
    Author: Dan Smith <[email protected]>
    Date:   Thu Dec 9 09:29:07 2021 -0500

        test_valid_six unittests now working

    commit d6fb257
    Author: Dan Smith <[email protected]>
    Date:   Wed Dec 8 17:45:26 2021 -0500

        trying to re-enable more unittests

    commit c787e98
    Author: Dan Smith <[email protected]>
    Date:   Wed Dec 8 16:54:28 2021 -0500

        unit-test breaks on Linux

    commit bd82787
    Author: Dan Smith <[email protected]>
    Date:   Wed Dec 8 15:25:31 2021 -0500

        Squashed commit of the following:

        commit c56ac71
        Author: Dan Smith <[email protected]>
        Date:   Wed Dec 8 15:23:00 2021 -0500

            merge in develop/SIDD-3.0

    commit e621466
    Author: Dan Smith <[email protected]>
    Date:   Wed Dec 8 15:08:04 2021 -0500

        merge in develop/8AMPI_PHSI

commit b582c05
Author: Dan Smith <[email protected]>
Date:   Wed Dec 8 15:26:24 2021 -0500

    Squashed commit of the following:

    commit bd82787
    Author: Dan Smith <[email protected]>
    Date:   Wed Dec 8 15:25:31 2021 -0500

        Squashed commit of the following:

        commit c56ac71
        Author: Dan Smith <[email protected]>
        Date:   Wed Dec 8 15:23:00 2021 -0500

            merge in develop/SIDD-3.0

    commit e621466
    Author: Dan Smith <[email protected]>
    Date:   Wed Dec 8 15:08:04 2021 -0500

        merge in develop/8AMPI_PHSI

commit c56ac71
Author: Dan Smith <[email protected]>
Date:   Wed Dec 8 15:23:00 2021 -0500

    merge in develop/SIDD-3.0

* xml.lite tweaks from coda-oss

* simple URI validation

* create a "path" after splitting on ":"

* sanity-check a URI

* latest from coda-oss

* latest from coda-oss

* adjust for coda-oss changes

* latest from coda-oss

* explicitly =delete move

* str::strip from coda-oss

* use strongly-type TRE fields

* latest from coda-oss

* latest from nitro
  • Loading branch information
J. Daniel Smith authored Jan 5, 2022
1 parent e2e3def commit 28afa65
Show file tree
Hide file tree
Showing 14 changed files with 140 additions and 72 deletions.
24 changes: 16 additions & 8 deletions externals/coda-oss/modules/c++/logging/include/logging/Filter.h
Original file line number Diff line number Diff line change
Expand Up @@ -24,8 +24,9 @@
// Filter.h
///////////////////////////////////////////////////////////

#ifndef __LOGGING_FILTER_H__
#define __LOGGING_FILTER_H__
#ifndef CODA_OSS_logging_Filter_h_INCLUDED_
#define CODA_OSS_logging_Filter_h_INCLUDED_
#pragma once

#include <string>
#include "logging/LogRecord.h"
Expand All @@ -38,18 +39,25 @@ namespace logging
*
* \brief Filter instances are used to perform arbitrary filtering of LogRecords.
*/
class Filter
struct Filter
{
public:
Filter(std::string name = "") : mName(name){}
virtual ~Filter(){}
virtual ~Filter() = default;

bool filter(const LogRecord* record) const;
std::string getName() const { return mName; }
virtual bool filter(const LogRecord* record) const;
virtual bool filter(const LogRecord& record) const
{
return filter(&record);
}

std::string getName() const
{
return mName;
}

protected:
std::string mName;
};

}
#endif
#endif // CODA_OSS_logging_Filter_h_INCLUDED_
24 changes: 15 additions & 9 deletions externals/coda-oss/modules/c++/logging/include/logging/Filterer.h
Original file line number Diff line number Diff line change
Expand Up @@ -24,8 +24,9 @@
// Filterer.h
///////////////////////////////////////////////////////////

#ifndef __LOGGING_FILTERER_H__
#define __LOGGING_FILTERER_H__
#ifndef CODA_OSS_logging_Filterer_h_INCLUDED_
#define CODA_OSS_logging_Filterer_h_INCLUDED_
#pragma once

#include <string>
#include <map>
Expand All @@ -41,27 +42,32 @@ namespace logging
* A base class for loggers and handlers which allows them to share
* common code.
*/
class Filterer
struct Filterer
{
public:
Filterer(){}
virtual ~Filterer(){}
Filterer() = default;
virtual ~Filterer() = default;

/*!
* Adds a Filter to the managed map of Filters. We do NOT take control of
* the pointer
*/
void addFilter(Filter* filter);
void addFilter(Filter&);

bool filter(const LogRecord* record) const;
virtual bool filter(const LogRecord* record) const;
virtual bool filter(const LogRecord& record) const
{
return filter(&record);
}

//! Removes the specified Filter
void removeFilter(Filter* filter);
void removeFilter(Filter&);

protected:
std::map<std::string, Filter*> filters;

};

}
#endif
#endif // CODA_OSS_logging_Filterer_h_INCLUDED_

20 changes: 13 additions & 7 deletions externals/coda-oss/modules/c++/logging/include/logging/Handler.h
Original file line number Diff line number Diff line change
Expand Up @@ -24,8 +24,8 @@
// Handler.h
///////////////////////////////////////////////////////////

#ifndef __LOGGING_HANDLER_H__
#define __LOGGING_HANDLER_H__
#ifndef CODA_OSS_logging_Handler_h_INCLUDED_
#define CODA_OSS_logging_Handler_h_INCLUDED_

#include <string>
#include "logging/LogRecord.h"
Expand All @@ -51,16 +51,15 @@ struct Handler : public Filterer
* Construct a Handler at the specified LogLevel (LogLevel::LOG_NOTSET is default)
*/
Handler(LogLevel level = LogLevel::LOG_NOTSET);
virtual ~Handler()
{
}
virtual ~Handler() = default;
Handler& operator=(const Handler&) = delete;

/*!
* Sets the Formatter to use when formatting LogRecords
* Not Threads Safe!
*/
virtual void setFormatter(Formatter* formatter);
virtual void setFormatter(std::unique_ptr<Formatter>&&);

//! Sets the minimum LogLevel required to emit LogRecords
void setLevel(LogLevel level);
Expand All @@ -74,6 +73,10 @@ struct Handler : public Filterer
* and emitted.
*/
virtual bool handle(const LogRecord* record);
virtual bool handle(const LogRecord& record)
{
return handle(&record);
}

virtual void close();

Expand All @@ -85,7 +88,10 @@ struct Handler : public Filterer
// for writing directly to stream,
// used for the bulk of the logging for speed
virtual void emitRecord(const LogRecord* record) = 0;

virtual void emitRecord(const LogRecord& record)
{
return emitRecord(&record);
}

LogLevel mLevel = LogLevel::LOG_NOTSET;
sys::Mutex mHandlerLock;
Expand All @@ -94,4 +100,4 @@ struct Handler : public Filterer
};

}
#endif
#endif // CODA_OSS_logging_Handler_h_INCLUDED_
13 changes: 9 additions & 4 deletions externals/coda-oss/modules/c++/logging/include/logging/Logger.h
Original file line number Diff line number Diff line change
Expand Up @@ -24,12 +24,13 @@
// Logger.h
///////////////////////////////////////////////////////////

#ifndef __LOGGING_LOGGER_H__
#define __LOGGING_LOGGER_H__
#ifndef CODA_OSS_logging_Logger_h_INCLUDED_
#define CODA_OSS_logging_Logger_h_INCLUDED_

#include <string>
#include <vector>
#include <memory>

#include "logging/Filterer.h"
#include "logging/LogRecord.h"
#include "logging/Handler.h"
Expand Down Expand Up @@ -104,6 +105,7 @@ struct Logger : public Filterer
* This Logger does not own the passed-in Handler.
*/
void addHandler(Handler* handler, bool own = false);
void addHandler(std::unique_ptr<Handler>&&); // own = true

/*!
* Removes the specified Handler from the list of Handlers.
Expand Down Expand Up @@ -139,14 +141,17 @@ struct Logger : public Filterer

protected:
void handle(const LogRecord* record);
void handle(const LogRecord& record)
{
handle(&record);
}

typedef std::pair<Handler*, bool> Handler_T;
typedef std::vector<Handler_T> Handlers_T;

std::string mName;
Handlers_T mHandlers;

};
typedef std::shared_ptr<Logger> LoggerPtr;
}
#endif
#endif // CODA_OSS_logging_Logger_h_INCLUDED_
Original file line number Diff line number Diff line change
Expand Up @@ -57,6 +57,7 @@ struct StreamHandler : public Handler
//! adds the need to write epilogue before deleting formatter
// and then writing the prologue with the new formatter
virtual void setFormatter(Formatter* formatter);
virtual void setFormatter(std::unique_ptr<Formatter>&&);

virtual void close();

Expand Down
15 changes: 10 additions & 5 deletions externals/coda-oss/modules/c++/logging/source/Filterer.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -34,13 +34,16 @@ void logging::Filterer::addFilter(logging::Filter* filter)
filters[filter->getName()] = filter;
}
}
void logging::Filterer::addFilter(logging::Filter& filter)
{
addFilter(&filter);
}

bool logging::Filterer::filter(const logging::LogRecord* record) const
{
for (std::map<std::string, logging::Filter*>::const_iterator p = filters.begin();
p != filters.end(); ++p)
for (const auto& p : filters)
{
if (!p->second->filter(record))
if (!p.second->filter(record))
return false;
}
return true;
Expand All @@ -49,6 +52,8 @@ bool logging::Filterer::filter(const logging::LogRecord* record) const
void logging::Filterer::removeFilter(logging::Filter* filter)
{
filters.erase(filter->getName());

}

void logging::Filterer::removeFilter(logging::Filter& filter)
{
removeFilter(&filter);
}
9 changes: 6 additions & 3 deletions externals/coda-oss/modules/c++/logging/source/Handler.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -39,11 +39,10 @@ Handler::Handler(LogLevel level)
void Handler::close()
{
// delete if necessary
if (mFormatter != &mDefaultFormatter &&
mFormatter != NULL)
if (mFormatter != &mDefaultFormatter) // "delete NULL" is ok
{
delete mFormatter;
mFormatter = NULL;
mFormatter = nullptr;
}
}

Expand Down Expand Up @@ -83,4 +82,8 @@ void Handler::setFormatter(Formatter* formatter)
mFormatter = formatter;
}
}
void Handler::setFormatter(std::unique_ptr<Formatter>&& formatter)
{
setFormatter(formatter.release());
}
}
27 changes: 14 additions & 13 deletions externals/coda-oss/modules/c++/logging/source/Logger.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -34,20 +34,18 @@ logging::Logger::~Logger()

void logging::Logger::log(logging::LogLevel level, const std::string& msg)
{
logging::LogRecord *rec = new logging::LogRecord(mName, msg, level);
const logging::LogRecord rec(mName, msg, level);
handle(rec);
delete rec;
}

void logging::Logger::log(LogLevel level, const except::Context& ctxt)
{
logging::LogRecord *rec = new logging::LogRecord(mName, ctxt.getMessage(),
const logging::LogRecord rec(mName, ctxt.getMessage(),
level, ctxt.getFile(),
ctxt.getFunction(),
ctxt.getLine(),
ctxt.getTime());
handle(rec);
delete rec;
}

void logging::Logger::log(LogLevel level, const except::Throwable& t)
Expand Down Expand Up @@ -155,12 +153,12 @@ void logging::Logger::handle(const logging::LogRecord* record)
{
if (filter(record))
{
for (Handlers_T::iterator p = mHandlers.begin(); p != mHandlers.end(); ++p)
for (const auto& p : mHandlers)
{
//std::cout << (int)(*p)->getLevel() << std::endl;
//only handle if it is above/equal to threshold
if (p->first->getLevel() <= record->getLevel())
p->first->handle(record);
if (p.first->getLevel() <= record->getLevel())
p.first->handle(record);
}
}
}
Expand All @@ -181,6 +179,10 @@ void logging::Logger::addHandler(logging::Handler* handler, bool own)
if (!found)
mHandlers.push_back(Handler_T(handler, own));
}
void logging::Logger::addHandler(std::unique_ptr<logging::Handler>&& handler)
{
addHandler(handler.release(), true /*own*/);
}

void logging::Logger::removeHandler(logging::Handler* handler)
{
Expand All @@ -197,20 +199,19 @@ void logging::Logger::removeHandler(logging::Handler* handler)

void logging::Logger::setLevel(LogLevel level)
{
for (Handlers_T::iterator p = mHandlers.begin(); p != mHandlers.end(); ++p)
for (const auto& p : mHandlers)
{
//set the level
p->first->setLevel(level);
p.first->setLevel(level);
}
}

void logging::Logger::reset()
{
for (Handlers_T::iterator p = mHandlers.begin(); p
!= mHandlers.end(); ++p)
for (const auto& p : mHandlers)
{
if (p->second && p->first)
delete p->first;
if (p.second && p.first)
delete p.first;
}
mHandlers.clear();
}
11 changes: 11 additions & 0 deletions externals/coda-oss/modules/c++/logging/source/StreamHandler.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -71,6 +71,17 @@ void StreamHandler::setFormatter(Formatter* formatter)
// start log with formatter injection
write(mFormatter->getPrologue());
}
void StreamHandler::setFormatter(std::unique_ptr<Formatter>&& formatter)
{
// end log with formatter injection
write(mFormatter->getEpilogue());

// delete old and reset to new
Handler::setFormatter(std::move(formatter));

// start log with formatter injection
write(mFormatter->getPrologue());
}

void StreamHandler::close()
{
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -61,9 +61,9 @@ sys::Mutex RunNothing::counterLock;

TEST_CASE(testExceptionLogger)
{
std::unique_ptr<logging::Logger> log(new logging::Logger("test"));
logging::Logger log("test");

mem::SharedPtr<logging::ExceptionLogger> exLog(new logging::ExceptionLogger(log.get()));
logging::ExceptionLogger exLog(&log);

size_t counter(0);
uint16_t numThreads(2);
Expand All @@ -73,11 +73,11 @@ TEST_CASE(testExceptionLogger)
mt::GenerationThreadPool pool(numThreads);
pool.start();

runs.push_back(new RunNothing(counter, exLog.get()));
runs.push_back(new RunNothing(counter, &exLog));
pool.addAndWaitGroup(runs);
runs.clear();

runs.push_back(new RunNothing(counter, exLog.get()));
runs.push_back(new RunNothing(counter, &exLog));
pool.addAndWaitGroup(runs);
runs.clear();

Expand Down
Loading

0 comments on commit 28afa65

Please sign in to comment.