add .clang-format
Change-Id: I2c36b7886a25d0b235693b0776019d29608e1d52
Signed-off-by: Patrick Venture <venture@google.com>
diff --git a/.clang-format b/.clang-format
new file mode 100644
index 0000000..ea71ad6
--- /dev/null
+++ b/.clang-format
@@ -0,0 +1,99 @@
+---
+Language: Cpp
+# BasedOnStyle: LLVM
+AccessModifierOffset: -2
+AlignAfterOpenBracket: Align
+AlignConsecutiveAssignments: false
+AlignConsecutiveDeclarations: false
+AlignEscapedNewlinesLeft: false
+AlignOperands: true
+AlignTrailingComments: true
+AllowAllParametersOfDeclarationOnNextLine: true
+AllowShortBlocksOnASingleLine: false
+AllowShortCaseLabelsOnASingleLine: false
+AllowShortFunctionsOnASingleLine: None
+AllowShortIfStatementsOnASingleLine: false
+AllowShortLoopsOnASingleLine: false
+AlwaysBreakAfterDefinitionReturnType: None
+AlwaysBreakAfterReturnType: None
+AlwaysBreakBeforeMultilineStrings: false
+AlwaysBreakTemplateDeclarations: true
+BinPackArguments: true
+BinPackParameters: true
+BraceWrapping:
+ AfterClass: true
+ AfterControlStatement: true
+ AfterEnum: true
+ AfterFunction: true
+ AfterNamespace: true
+ AfterObjCDeclaration: true
+ AfterStruct: true
+ AfterUnion: true
+ BeforeCatch: true
+ BeforeElse: true
+ IndentBraces: false
+BreakBeforeBinaryOperators: None
+BreakBeforeBraces: Custom
+BreakBeforeTernaryOperators: true
+BreakConstructorInitializers: AfterColon
+ColumnLimit: 80
+CommentPragmas: '^ IWYU pragma:'
+ConstructorInitializerAllOnOneLineOrOnePerLine: false
+ConstructorInitializerIndentWidth: 4
+ContinuationIndentWidth: 4
+Cpp11BracedListStyle: true
+DerivePointerAlignment: false
+PointerAlignment: Left
+DisableFormat: false
+ExperimentalAutoDetectBinPacking: false
+FixNamespaceComments: true
+ForEachMacros: [ foreach, Q_FOREACH, BOOST_FOREACH ]
+IncludeBlocks: Regroup
+IncludeCategories:
+ - Regex: '^[<"](gtest|gmock)'
+ Priority: 5
+ - Regex: '^"config.h"'
+ Priority: -1
+ - Regex: '^".*\.hpp"'
+ Priority: 1
+ - Regex: '^<.*\.h>'
+ Priority: 2
+ - Regex: '^<.*'
+ Priority: 3
+ - Regex: '.*'
+ Priority: 4
+IndentCaseLabels: true
+IndentWidth: 4
+IndentWrappedFunctionNames: true
+KeepEmptyLinesAtTheStartOfBlocks: true
+MacroBlockBegin: ''
+MacroBlockEnd: ''
+MaxEmptyLinesToKeep: 1
+NamespaceIndentation: None
+ObjCBlockIndentWidth: 2
+ObjCSpaceAfterProperty: false
+ObjCSpaceBeforeProtocolList: true
+PenaltyBreakBeforeFirstCallParameter: 19
+PenaltyBreakComment: 300
+PenaltyBreakFirstLessLess: 120
+PenaltyBreakString: 1000
+PenaltyExcessCharacter: 1000000
+PenaltyReturnTypeOnItsOwnLine: 60
+ReflowComments: true
+SortIncludes: true
+SortUsingDeclarations: true
+SpaceAfterCStyleCast: false
+SpaceBeforeAssignmentOperators: true
+SpaceBeforeParens: ControlStatements
+SpaceInEmptyParentheses: false
+SpacesBeforeTrailingComments: 1
+SpacesInAngles: false
+SpacesInContainerLiterals: true
+SpacesInCStyleCastParentheses: false
+SpacesInParentheses: false
+SpacesInSquareBrackets: false
+Standard: Cpp11
+TabWidth: 4
+UseTab: Never
+...
+
diff --git a/callouts/callout_test.cpp b/callouts/callout_test.cpp
index 2eff6a0..729bbf5 100644
--- a/callouts/callout_test.cpp
+++ b/callouts/callout_test.cpp
@@ -1,14 +1,15 @@
-#include <iostream>
-#include <sdbusplus/exception.hpp>
-#include <phosphor-logging/elog.hpp>
-#include <phosphor-logging/elog-errors.hpp>
#include "elog_meta.hpp"
+#include <iostream>
+#include <phosphor-logging/elog-errors.hpp>
+#include <phosphor-logging/elog.hpp>
+#include <sdbusplus/exception.hpp>
+
using namespace phosphor::logging;
int main(int argc, char** argv)
{
- if(2 != argc)
+ if (2 != argc)
{
std::cerr << "usage: callout-test <sysfs path>" << std::endl;
return -1;
@@ -17,10 +18,9 @@
using namespace example::xyz::openbmc_project::Example::Elog;
try
{
- elog<TestCallout>(
- TestCallout::DEV_ADDR(0xDEADEAD),
- TestCallout::CALLOUT_ERRNO_TEST(0),
- TestCallout::CALLOUT_DEVICE_PATH_TEST(argv[1]));
+ elog<TestCallout>(TestCallout::DEV_ADDR(0xDEADEAD),
+ TestCallout::CALLOUT_ERRNO_TEST(0),
+ TestCallout::CALLOUT_DEVICE_PATH_TEST(argv[1]));
}
catch (TestCallout& e)
{
@@ -29,5 +29,3 @@
return 0;
}
-
-
diff --git a/elog.cpp b/elog.cpp
index 363553b..5cefe81 100644
--- a/elog.cpp
+++ b/elog.cpp
@@ -1,4 +1,5 @@
#include "config.h"
+
#include <phosphor-logging/elog.hpp>
#include <stdexcept>
@@ -17,16 +18,14 @@
constexpr auto MAPPER_PATH = "/xyz/openbmc_project/object_mapper";
constexpr auto MAPPER_INTERFACE = "xyz.openbmc_project.ObjectMapper";
- constexpr auto IFACE_INTERNAL("xyz.openbmc_project.Logging.Internal.Manager");
+ constexpr auto IFACE_INTERNAL(
+ "xyz.openbmc_project.Logging.Internal.Manager");
// Transaction id is located at the end of the string separated by a period.
auto b = sdbusplus::bus::new_default();
- auto mapper = b.new_method_call(
- MAPPER_BUSNAME,
- MAPPER_PATH,
- MAPPER_INTERFACE,
- "GetObject");
+ auto mapper = b.new_method_call(MAPPER_BUSNAME, MAPPER_PATH,
+ MAPPER_INTERFACE, "GetObject");
mapper.append(OBJ_INTERNAL, std::vector<std::string>({IFACE_INTERNAL}));
auto mapperResponseMsg = b.call(mapper);
@@ -43,12 +42,9 @@
}
const auto& host = mapperResponse.cbegin()->first;
- auto m = b.new_method_call(
- host.c_str(),
- OBJ_INTERNAL,
- IFACE_INTERNAL,
- funcName);
- return m;
+ auto m =
+ b.new_method_call(host.c_str(), OBJ_INTERNAL, IFACE_INTERNAL, funcName);
+ return m;
}
void commit(const char* name)
@@ -78,4 +74,3 @@
} // namespace logging
} // namespace phosphor
-
diff --git a/elog_entry.cpp b/elog_entry.cpp
index 74fcc4b..04aa917 100644
--- a/elog_entry.cpp
+++ b/elog_entry.cpp
@@ -1,6 +1,7 @@
#include "elog_entry.hpp"
-#include "log_manager.hpp"
+
#include "elog_serialize.hpp"
+#include "log_manager.hpp"
namespace phosphor
{
@@ -16,15 +17,14 @@
bool Entry::resolved(bool value)
{
- auto current = sdbusplus::xyz::openbmc_project::
- Logging::server::Entry::resolved();
+ auto current =
+ sdbusplus::xyz::openbmc_project::Logging::server::Entry::resolved();
if (value != current)
{
- value ?
- associations({}) :
- associations(assocs);
- current = sdbusplus::xyz::openbmc_project::
- Logging::server::Entry::resolved(value);
+ value ? associations({}) : associations(assocs);
+ current =
+ sdbusplus::xyz::openbmc_project::Logging::server::Entry::resolved(
+ value);
serialize(*this);
}
@@ -32,4 +32,4 @@
}
} // namespace logging
-} // namepsace phosphor
+} // namespace phosphor
diff --git a/elog_entry.hpp b/elog_entry.hpp
index 5ee8f5f..abcfe50 100644
--- a/elog_entry.hpp
+++ b/elog_entry.hpp
@@ -1,11 +1,12 @@
#pragma once
-#include <sdbusplus/bus.hpp>
-#include <sdbusplus/server/object.hpp>
+#include "org/openbmc/Associations/server.hpp"
#include "xyz/openbmc_project/Logging/Entry/server.hpp"
#include "xyz/openbmc_project/Object/Delete/server.hpp"
#include "xyz/openbmc_project/Software/Version/server.hpp"
-#include "org/openbmc/Associations/server.hpp"
+
+#include <sdbusplus/bus.hpp>
+#include <sdbusplus/server/object.hpp>
namespace phosphor
{
@@ -19,7 +20,7 @@
sdbusplus::xyz::openbmc_project::Software::server::Version>;
using AssociationList =
- std::vector<std::tuple<std::string, std::string, std::string>>;
+ std::vector<std::tuple<std::string, std::string, std::string>>;
namespace internal
{
@@ -34,104 +35,96 @@
*/
class Entry : public EntryIfaces
{
- public:
- Entry() = delete;
- Entry(const Entry&) = delete;
- Entry& operator=(const Entry&) = delete;
- Entry(Entry&&) = delete;
- Entry& operator=(Entry&&) = delete;
- virtual ~Entry() = default;
+ public:
+ Entry() = delete;
+ Entry(const Entry&) = delete;
+ Entry& operator=(const Entry&) = delete;
+ Entry(Entry&&) = delete;
+ Entry& operator=(Entry&&) = delete;
+ virtual ~Entry() = default;
- /** @brief Constructor to put object onto bus at a dbus path.
- * Defer signal registration (pass true for deferSignal to the
- * base class) until after the properties are set.
- * @param[in] bus - Bus to attach to.
- * @param[in] path - Path to attach at.
- * @param[in] idErr - The error entry id.
- * @param[in] timestampErr - The commit timestamp.
- * @param[in] severityErr - The severity of the error.
- * @param[in] msgErr - The message of the error.
- * @param[in] additionalDataErr - The error metadata.
- * @param[in] objects - The list of associations.
- * @param[in] fwVersion - The BMC code version.
- * @param[in] parent - The error's parent.
- */
- Entry(sdbusplus::bus::bus& bus,
- const std::string& path,
- uint32_t idErr,
- uint64_t timestampErr,
- Level severityErr,
- std::string&& msgErr,
- std::vector<std::string>&& additionalDataErr,
- AssociationList&& objects,
- const std::string& fwVersion,
- internal::Manager& parent) :
- EntryIfaces(bus, path.c_str(), true),
- parent(parent)
- {
- id(idErr);
- severity(severityErr);
- timestamp(timestampErr);
- message(std::move(msgErr));
- additionalData(std::move(additionalDataErr));
- associations(std::move(objects));
- // Store a copy of associations in case we need to recreate
- assocs = associations();
- sdbusplus::xyz::openbmc_project::
- Logging::server::Entry::resolved(false);
+ /** @brief Constructor to put object onto bus at a dbus path.
+ * Defer signal registration (pass true for deferSignal to the
+ * base class) until after the properties are set.
+ * @param[in] bus - Bus to attach to.
+ * @param[in] path - Path to attach at.
+ * @param[in] idErr - The error entry id.
+ * @param[in] timestampErr - The commit timestamp.
+ * @param[in] severityErr - The severity of the error.
+ * @param[in] msgErr - The message of the error.
+ * @param[in] additionalDataErr - The error metadata.
+ * @param[in] objects - The list of associations.
+ * @param[in] fwVersion - The BMC code version.
+ * @param[in] parent - The error's parent.
+ */
+ Entry(sdbusplus::bus::bus& bus, const std::string& path, uint32_t idErr,
+ uint64_t timestampErr, Level severityErr, std::string&& msgErr,
+ std::vector<std::string>&& additionalDataErr,
+ AssociationList&& objects, const std::string& fwVersion,
+ internal::Manager& parent) :
+ EntryIfaces(bus, path.c_str(), true),
+ parent(parent)
+ {
+ id(idErr);
+ severity(severityErr);
+ timestamp(timestampErr);
+ message(std::move(msgErr));
+ additionalData(std::move(additionalDataErr));
+ associations(std::move(objects));
+ // Store a copy of associations in case we need to recreate
+ assocs = associations();
+ sdbusplus::xyz::openbmc_project::Logging::server::Entry::resolved(
+ false);
- version(fwVersion);
- purpose(VersionPurpose::BMC);
+ version(fwVersion);
+ purpose(VersionPurpose::BMC);
- // Emit deferred signal.
- this->emit_object_added();
- };
+ // Emit deferred signal.
+ this->emit_object_added();
+ };
- /** @brief Constructor that puts an "empty" error object on the bus,
- * with only the id property populated. Rest of the properties
- * to be set by the caller. Caller should emit the added signal.
- * @param[in] bus - Bus to attach to.
- * @param[in] path - Path to attach at.
- * @param[in] id - The error entry id.
- * @param[in] parent - The error's parent.
- */
- Entry(sdbusplus::bus::bus& bus,
- const std::string& path,
- uint32_t entryId,
- internal::Manager& parent) :
- EntryIfaces(bus, path.c_str(), true),
- parent(parent)
- {
- id(entryId);
- };
+ /** @brief Constructor that puts an "empty" error object on the bus,
+ * with only the id property populated. Rest of the properties
+ * to be set by the caller. Caller should emit the added signal.
+ * @param[in] bus - Bus to attach to.
+ * @param[in] path - Path to attach at.
+ * @param[in] id - The error entry id.
+ * @param[in] parent - The error's parent.
+ */
+ Entry(sdbusplus::bus::bus& bus, const std::string& path, uint32_t entryId,
+ internal::Manager& parent) :
+ EntryIfaces(bus, path.c_str(), true),
+ parent(parent)
+ {
+ id(entryId);
+ };
- /** @brief Set resolution status of the error.
- * @param[in] value - boolean indicating resolution
- * status (true = resolved)
- * @returns value of 'Resolved' property
- */
- bool resolved(bool value) override;
+ /** @brief Set resolution status of the error.
+ * @param[in] value - boolean indicating resolution
+ * status (true = resolved)
+ * @returns value of 'Resolved' property
+ */
+ bool resolved(bool value) override;
- using sdbusplus::xyz::openbmc_project::
- Logging::server::Entry::resolved;
+ using sdbusplus::xyz::openbmc_project::Logging::server::Entry::resolved;
- /** @brief Delete this d-bus object.
- */
- void delete_() override;
+ /** @brief Delete this d-bus object.
+ */
+ void delete_() override;
- /** @brief Severity level to check in cap.
- * @details Errors with severity lesser than this will be
- * considered as low priority and maximum ERROR_INFO_CAP
- * number errors of this category will be captured.
- */
- static constexpr auto sevLowerLimit = Entry::Level::Informational;
+ /** @brief Severity level to check in cap.
+ * @details Errors with severity lesser than this will be
+ * considered as low priority and maximum ERROR_INFO_CAP
+ * number errors of this category will be captured.
+ */
+ static constexpr auto sevLowerLimit = Entry::Level::Informational;
- private:
- /** @brief This entry's associations */
- AssociationList assocs = {};
+ private:
+ /** @brief This entry's associations */
+ AssociationList assocs = {};
- /** @brief This entry's parent */
- internal::Manager& parent;
+ /** @brief This entry's parent */
+ internal::Manager& parent;
};
} // namespace logging
diff --git a/elog_meta.cpp b/elog_meta.cpp
index 514296e..98425d4 100644
--- a/elog_meta.cpp
+++ b/elog_meta.cpp
@@ -1,4 +1,5 @@
#include "config.h"
+
#include "elog_meta.hpp"
namespace phosphor
@@ -13,41 +14,34 @@
#if defined PROCESS_META
template <>
-void build<xyz::openbmc_project::Common::
- Callout::Device::CALLOUT_DEVICE_PATH>(
- const std::string& match,
- const std::vector<std::string>& data,
+void build<xyz::openbmc_project::Common::Callout::Device::CALLOUT_DEVICE_PATH>(
+ const std::string& match, const std::vector<std::string>& data,
AssociationList& list)
{
std::map<std::string, std::string> metadata;
parse(data, metadata);
auto iter = metadata.find(match);
- if(metadata.end() != iter)
+ if (metadata.end() != iter)
{
- auto comp = [](const auto& first, const auto& second)
- {
+ auto comp = [](const auto& first, const auto& second) {
return (strcmp(std::get<0>(first), second) < 0);
};
- auto callout = std::lower_bound(callouts.begin(),
- callouts.end(),
- (iter->second).c_str(),
- comp);
- if((callouts.end() != callout) &&
- !strcmp((iter->second).c_str(), std::get<0>(*callout)))
+ auto callout = std::lower_bound(callouts.begin(), callouts.end(),
+ (iter->second).c_str(), comp);
+ if ((callouts.end() != callout) &&
+ !strcmp((iter->second).c_str(), std::get<0>(*callout)))
{
- list.emplace_back(std::make_tuple(CALLOUT_FWD_ASSOCIATION,
- CALLOUT_REV_ASSOCIATION,
- std::string(INVENTORY_ROOT) +
- std::get<1>(*callout)));
+ list.emplace_back(std::make_tuple(
+ CALLOUT_FWD_ASSOCIATION, CALLOUT_REV_ASSOCIATION,
+ std::string(INVENTORY_ROOT) + std::get<1>(*callout)));
}
}
}
template <>
-void build<xyz::openbmc_project::Common::
- Callout::Inventory::CALLOUT_INVENTORY_PATH>(
- const std::string& match,
- const std::vector<std::string>& data,
+void build<
+ xyz::openbmc_project::Common::Callout::Inventory::CALLOUT_INVENTORY_PATH>(
+ const std::string& match, const std::vector<std::string>& data,
AssociationList& list)
{
std::map<std::string, std::string> metadata;
diff --git a/elog_meta.hpp b/elog_meta.hpp
index c0de428..0fe05f2 100644
--- a/elog_meta.hpp
+++ b/elog_meta.hpp
@@ -1,13 +1,14 @@
#pragma once
-#include <vector>
-#include <string>
-#include <tuple>
+#include "callouts-gen.hpp"
+#include "elog_entry.hpp"
+
#include <algorithm>
#include <cstring>
#include <phosphor-logging/elog-errors.hpp>
-#include "elog_entry.hpp"
-#include "callouts-gen.hpp"
+#include <string>
+#include <tuple>
+#include <vector>
namespace phosphor
{
@@ -21,8 +22,7 @@
namespace associations
{
-using Type = void(const std::string&,
- const std::vector<std::string>&,
+using Type = void(const std::string&, const std::vector<std::string>&,
AssociationList& list);
/** @brief Pull out metadata name and value from the string
@@ -34,10 +34,10 @@
std::map<std::string, std::string>& metadata)
{
constexpr auto separator = '=';
- for(const auto& entry: data)
+ for (const auto& entry : data)
{
auto pos = entry.find(separator);
- if(std::string::npos != pos)
+ if (std::string::npos != pos)
{
auto key = entry.substr(0, entry.find(separator));
auto value = entry.substr(entry.find(separator) + 1);
@@ -54,8 +54,7 @@
* @param [out] list - list of error association objects
*/
template <typename M>
-void build(const std::string& match,
- const std::vector<std::string>& data,
+void build(const std::string& match, const std::vector<std::string>& data,
AssociationList& list) = delete;
// Example template specialization - we don't want to do anything
@@ -69,33 +68,28 @@
}
template <>
-inline void build<example::xyz::openbmc_project::
- Example::Device::Callout::CALLOUT_DEVICE_PATH_TEST>(
- const std::string& match,
- const std::vector<std::string>& data,
- AssociationList& list)
+inline void
+ build<example::xyz::openbmc_project::Example::Device::Callout::
+ CALLOUT_DEVICE_PATH_TEST>(const std::string& match,
+ const std::vector<std::string>& data,
+ AssociationList& list)
{
constexpr auto ROOT = "/xyz/openbmc_project/inventory";
std::map<std::string, std::string> metadata;
parse(data, metadata);
auto iter = metadata.find(match);
- if(metadata.end() != iter)
+ if (metadata.end() != iter)
{
- auto comp = [](const auto& first, const auto& second)
- {
+ auto comp = [](const auto& first, const auto& second) {
return (strcmp(std::get<0>(first), second) < 0);
};
- auto callout = std::lower_bound(callouts.begin(),
- callouts.end(),
- (iter->second).c_str(),
- comp);
- if((callouts.end() != callout) &&
- !strcmp((iter->second).c_str(), std::get<0>(*callout)))
+ auto callout = std::lower_bound(callouts.begin(), callouts.end(),
+ (iter->second).c_str(), comp);
+ if ((callouts.end() != callout) &&
+ !strcmp((iter->second).c_str(), std::get<0>(*callout)))
{
- list.push_back(std::make_tuple("callout",
- "fault",
- std::string(ROOT) +
- std::get<1>(*callout)));
+ list.push_back(std::make_tuple(
+ "callout", "fault", std::string(ROOT) + std::get<1>(*callout)));
}
}
}
@@ -107,22 +101,18 @@
#if defined PROCESS_META
template <>
-void build<xyz::openbmc_project::Common::
- Callout::Device::CALLOUT_DEVICE_PATH>(
- const std::string& match,
- const std::vector<std::string>& data,
+void build<xyz::openbmc_project::Common::Callout::Device::CALLOUT_DEVICE_PATH>(
+ const std::string& match, const std::vector<std::string>& data,
AssociationList& list);
template <>
-void build<xyz::openbmc_project::Common::
- Callout::Inventory::CALLOUT_INVENTORY_PATH>(
- const std::string& match,
- const std::vector<std::string>& data,
+void build<
+ xyz::openbmc_project::Common::Callout::Inventory::CALLOUT_INVENTORY_PATH>(
+ const std::string& match, const std::vector<std::string>& data,
AssociationList& list);
#endif // PROCESS_META
-
} // namespace associations
} // namespace metadata
} // namespace logging
diff --git a/elog_serialize.cpp b/elog_serialize.cpp
index d807a93..72534a4 100644
--- a/elog_serialize.cpp
+++ b/elog_serialize.cpp
@@ -1,12 +1,13 @@
-#include <cereal/types/string.hpp>
-#include <cereal/types/vector.hpp>
-#include <cereal/types/tuple.hpp>
-#include <cereal/archives/binary.hpp>
-#include <fstream>
+#include "config.h"
#include "elog_serialize.hpp"
+
+#include <cereal/archives/binary.hpp>
+#include <cereal/types/string.hpp>
+#include <cereal/types/tuple.hpp>
+#include <cereal/types/vector.hpp>
+#include <fstream>
#include <phosphor-logging/log.hpp>
-#include "config.h"
// Register class version
// From cereal documentation;
@@ -25,12 +26,11 @@
* @param[in] version - Class version that enables handling
* a serialized data across code levels
*/
-template<class Archive>
+template <class Archive>
void save(Archive& a, const Entry& e, const std::uint32_t version)
{
- a(e.id(), e.severity(), e.timestamp(),
- e.message(), e.additionalData(), e.associations(), e.resolved(),
- e.version());
+ a(e.id(), e.severity(), e.timestamp(), e.message(), e.additionalData(),
+ e.associations(), e.resolved(), e.version());
}
/** @brief Function required by Cereal to perform deserialization.
@@ -40,11 +40,10 @@
* @param[in] version - Class version that enables handling
* a serialized data across code levels
*/
-template<class Archive>
+template <class Archive>
void load(Archive& a, Entry& e, const std::uint32_t version)
{
- using namespace
- sdbusplus::xyz::openbmc_project::Logging::server;
+ using namespace sdbusplus::xyz::openbmc_project::Logging::server;
uint32_t id{};
Entry::Level severity{};
@@ -57,13 +56,13 @@
if (version < std::stoul(FIRST_CEREAL_CLASS_VERSION_WITH_FWLEVEL))
{
- a(id, severity, timestamp, message,
- additionalData, associations, resolved);
+ a(id, severity, timestamp, message, additionalData, associations,
+ resolved);
}
else
{
- a(id, severity, timestamp, message,
- additionalData, associations, resolved, fwVersion);
+ a(id, severity, timestamp, message, additionalData, associations,
+ resolved, fwVersion);
}
e.id(id);
@@ -71,12 +70,12 @@
e.timestamp(timestamp);
e.message(message);
e.additionalData(additionalData);
- e.sdbusplus::xyz::openbmc_project::
- Logging::server::Entry::resolved(resolved);
+ e.sdbusplus::xyz::openbmc_project::Logging::server::Entry::resolved(
+ resolved);
e.associations(associations);
e.version(fwVersion);
- e.purpose(sdbusplus::xyz::openbmc_project::Software::
- server::Version::VersionPurpose::BMC);
+ e.purpose(sdbusplus::xyz::openbmc_project::Software::server::Version::
+ VersionPurpose::BMC);
}
fs::path serialize(const Entry& e, const fs::path& dir)
@@ -101,13 +100,13 @@
}
return false;
}
- catch(cereal::Exception& e)
+ catch (cereal::Exception& e)
{
log<level::ERR>(e.what());
fs::remove(path);
return false;
}
- catch(const std::length_error& e)
+ catch (const std::length_error& e)
{
// Running into: USCiLab/cereal#192
// This may be indicating some other issue in the
diff --git a/elog_serialize.hpp b/elog_serialize.hpp
index f7f3bd9..eaa47d4 100644
--- a/elog_serialize.hpp
+++ b/elog_serialize.hpp
@@ -1,10 +1,12 @@
#pragma once
+#include "config.h"
+
+#include "elog_entry.hpp"
+
+#include <experimental/filesystem>
#include <string>
#include <vector>
-#include <experimental/filesystem>
-#include "elog_entry.hpp"
-#include "config.h"
namespace phosphor
{
diff --git a/log_manager.cpp b/log_manager.cpp
index ca48210..46ada75 100644
--- a/log_manager.cpp
+++ b/log_manager.cpp
@@ -1,28 +1,33 @@
-#include <fstream>
-#include <future>
-#include <iostream>
-#include <chrono>
-#include <cstdio>
+#include "config.h"
+
+#include "log_manager.hpp"
+
+#include "elog_entry.hpp"
+#include "elog_meta.hpp"
+#include "elog_serialize.hpp"
+
#include <poll.h>
-#include <set>
-#include <string>
-#include <vector>
-#include <sdbusplus/vtable.hpp>
#include <sys/inotify.h>
#include <systemd/sd-bus.h>
#include <systemd/sd-journal.h>
#include <unistd.h>
-#include "config.h"
-#include "elog_entry.hpp"
+
+#include <chrono>
+#include <cstdio>
+#include <fstream>
+#include <future>
+#include <iostream>
#include <phosphor-logging/log.hpp>
-#include "log_manager.hpp"
-#include "elog_meta.hpp"
-#include "elog_serialize.hpp"
+#include <sdbusplus/vtable.hpp>
+#include <set>
+#include <string>
+#include <vector>
using namespace phosphor::logging;
using namespace std::chrono;
extern const std::map<metadata::Metadata,
- std::function<metadata::associations::Type>> meta;
+ std::function<metadata::associations::Type>>
+ meta;
namespace phosphor
{
@@ -92,12 +97,13 @@
// Flush all the pending log messages into the journal
journalSync();
- sd_journal *j = nullptr;
+ sd_journal* j = nullptr;
int rc = sd_journal_open(&j, SD_JOURNAL_LOCAL_ONLY);
if (rc < 0)
{
- logging::log<logging::level::ERR>("Failed to open journal",
- logging::entry("DESCRIPTION=%s", strerror(-rc)));
+ logging::log<logging::level::ERR>(
+ "Failed to open journal",
+ logging::entry("DESCRIPTION=%s", strerror(-rc)));
return;
}
@@ -109,7 +115,7 @@
metalist.insert(metamap->second.begin(), metamap->second.end());
}
- //Add _PID field information in AdditionalData.
+ // Add _PID field information in AdditionalData.
metalist.insert("_PID");
std::vector<std::string> additionalData;
@@ -118,12 +124,12 @@
// The result from the sd_journal_get_data() is of the form VARIABLE=value.
SD_JOURNAL_FOREACH_BACKWARDS(j)
{
- const char *data = nullptr;
+ const char* data = nullptr;
size_t length = 0;
// Look for the transaction id metadata variable
- rc = sd_journal_get_data(j, transactionIdVar, (const void **)&data,
- &length);
+ rc = sd_journal_get_data(j, transactionIdVar, (const void**)&data,
+ &length);
if (rc < 0)
{
// This journal entry does not have the TRANSACTION_ID
@@ -140,8 +146,7 @@
// 'data + transactionIdVarOffset' will be in the form of '1234'.
// 'length - transactionIdVarOffset' will be the length of '1234'.
if ((length <= (transactionIdVarOffset)) ||
- (transactionIdStr.compare(0,
- transactionIdStr.size(),
+ (transactionIdStr.compare(0, transactionIdStr.size(),
data + transactionIdVarOffset,
length - transactionIdVarOffset) != 0))
{
@@ -153,8 +158,8 @@
// Search for all metadata variables in the current journal entry.
for (auto i = metalist.cbegin(); i != metalist.cend();)
{
- rc = sd_journal_get_data(j, (*i).c_str(),
- (const void **)&data, &length);
+ rc = sd_journal_get_data(j, (*i).c_str(), (const void**)&data,
+ &length);
if (rc < 0)
{
// Metadata variable not found, check next metadata variable.
@@ -177,8 +182,9 @@
// Not all the metadata variables were found in the journal.
for (auto& metaVarStr : metalist)
{
- logging::log<logging::level::INFO>("Failed to find metadata",
- logging::entry("META_FIELD=%s", metaVarStr.c_str()));
+ logging::log<logging::level::INFO>(
+ "Failed to find metadata",
+ logging::entry("META_FIELD=%s", metaVarStr.c_str()));
}
}
@@ -195,24 +201,18 @@
realErrors.push_back(entryId);
}
auto ms = std::chrono::duration_cast<std::chrono::milliseconds>(
- std::chrono::system_clock::now().time_since_epoch()).count();
- auto objPath = std::string(OBJ_ENTRY) + '/' +
- std::to_string(entryId);
+ std::chrono::system_clock::now().time_since_epoch())
+ .count();
+ auto objPath = std::string(OBJ_ENTRY) + '/' + std::to_string(entryId);
- AssociationList objects {};
+ AssociationList objects{};
processMetadata(errMsg, additionalData, objects);
- auto e = std::make_unique<Entry>(
- busLog,
- objPath,
- entryId,
- ms, // Milliseconds since 1970
- errLvl,
- std::move(errMsg),
- std::move(additionalData),
- std::move(objects),
- fwVersion,
- *this);
+ auto e = std::make_unique<Entry>(busLog, objPath, entryId,
+ ms, // Milliseconds since 1970
+ errLvl, std::move(errMsg),
+ std::move(additionalData),
+ std::move(objects), fwVersion, *this);
serialize(*e);
entries.insert(std::make_pair(entryId, std::move(e)));
}
@@ -223,14 +223,14 @@
{
// additionalData is a list of "metadata=value"
constexpr auto separator = '=';
- for(const auto& entry: additionalData)
+ for (const auto& entry : additionalData)
{
auto found = entry.find(separator);
- if(std::string::npos != found)
+ if (std::string::npos != found)
{
auto metadata = entry.substr(0, found);
auto iter = meta.find(metadata);
- if(meta.end() != iter)
+ if (meta.end() != iter)
{
(iter->second)(metadata, additionalData, objects);
}
@@ -241,15 +241,14 @@
void Manager::erase(uint32_t entryId)
{
auto entry = entries.find(entryId);
- if(entries.end() != entry)
+ if (entries.end() != entry)
{
// Delete the persistent representation of this error.
fs::path errorPath(ERRLOG_PERSIST_PATH);
errorPath /= std::to_string(entryId);
fs::remove(errorPath);
- auto removeId = [](std::list<uint32_t>& ids , uint32_t id)
- {
+ auto removeId = [](std::list<uint32_t>& ids, uint32_t id) {
auto it = std::find(ids.begin(), ids.end(), id);
if (it != ids.end())
{
@@ -269,14 +268,13 @@
else
{
logging::log<level::ERR>("Invalid entry ID to delete",
- logging::entry("ID=%d", entryId));
+ logging::entry("ID=%d", entryId));
}
}
void Manager::restore()
{
- auto sanity = [](const auto& id, const auto& restoredId)
- {
+ auto sanity = [](const auto& id, const auto& restoredId) {
return id == restoredId;
};
std::vector<uint32_t> errorIds;
@@ -287,18 +285,15 @@
return;
}
- for(auto& file: fs::directory_iterator(dir))
+ for (auto& file : fs::directory_iterator(dir))
{
auto id = file.path().filename().c_str();
auto idNum = std::stol(id);
auto e = std::make_unique<Entry>(
- busLog,
- std::string(OBJ_ENTRY) + '/' + id,
- idNum,
- *this);
+ busLog, std::string(OBJ_ENTRY) + '/' + id, idNum, *this);
if (deserialize(file.path(), *e))
{
- //validate the restored error entry id
+ // validate the restored error entry id
if (sanity(static_cast<uint32_t>(idNum), e->id()))
{
e->emit_object_added();
@@ -490,4 +485,4 @@
} // namespace internal
} // namespace logging
-} // namepsace phosphor
+} // namespace phosphor
diff --git a/log_manager.hpp b/log_manager.hpp
index 3da450d..864de1a 100644
--- a/log_manager.hpp
+++ b/log_manager.hpp
@@ -1,22 +1,23 @@
#pragma once
-#include <list>
-#include <sdbusplus/bus.hpp>
-#include <phosphor-logging/log.hpp>
#include "elog_entry.hpp"
-#include "xyz/openbmc_project/Logging/Internal/Manager/server.hpp"
#include "xyz/openbmc_project/Collection/DeleteAll/server.hpp"
+#include "xyz/openbmc_project/Logging/Internal/Manager/server.hpp"
+
+#include <list>
+#include <phosphor-logging/log.hpp>
+#include <sdbusplus/bus.hpp>
namespace phosphor
{
namespace logging
{
-extern const std::map<std::string,std::vector<std::string>> g_errMetaMap;
-extern const std::map<std::string,level> g_errLevelMap;
+extern const std::map<std::string, std::vector<std::string>> g_errMetaMap;
+extern const std::map<std::string, level> g_errLevelMap;
-using DeleteAllIface = sdbusplus::server::object::object <
- sdbusplus::xyz::openbmc_project::Collection::server::DeleteAll >;
+using DeleteAllIface = sdbusplus::server::object::object<
+ sdbusplus::xyz::openbmc_project::Collection::server::DeleteAll>;
namespace details
{
@@ -39,142 +40,140 @@
*/
class Manager : public details::ServerObject<details::ManagerIface>
{
- public:
- Manager() = delete;
- Manager(const Manager&) = delete;
- Manager& operator=(const Manager&) = delete;
- Manager(Manager&&) = delete;
- Manager& operator=(Manager&&) = delete;
- virtual ~Manager() = default;
+ public:
+ Manager() = delete;
+ Manager(const Manager&) = delete;
+ Manager& operator=(const Manager&) = delete;
+ Manager(Manager&&) = delete;
+ Manager& operator=(Manager&&) = delete;
+ virtual ~Manager() = default;
- /** @brief Constructor to put object onto bus at a dbus path.
- * @param[in] bus - Bus to attach to.
- * @param[in] path - Path to attach at.
- */
- Manager(sdbusplus::bus::bus& bus, const char* objPath) :
- details::ServerObject<details::ManagerIface>(bus, objPath),
- busLog(bus),
- entryId(0),
- fwVersion(readFWVersion()) {};
+ /** @brief Constructor to put object onto bus at a dbus path.
+ * @param[in] bus - Bus to attach to.
+ * @param[in] path - Path to attach at.
+ */
+ Manager(sdbusplus::bus::bus& bus, const char* objPath) :
+ details::ServerObject<details::ManagerIface>(bus, objPath), busLog(bus),
+ entryId(0), fwVersion(readFWVersion()){};
- /*
- * @fn commit()
- * @brief sd_bus Commit method implementation callback.
- * @details Create an error/event log based on transaction id and
- * error message.
- * @param[in] transactionId - Unique identifier of the journal entries
- * to be committed.
- * @param[in] errMsg - The error exception message associated with the
- * error log to be committed.
- */
- void commit(uint64_t transactionId, std::string errMsg) override;
+ /*
+ * @fn commit()
+ * @brief sd_bus Commit method implementation callback.
+ * @details Create an error/event log based on transaction id and
+ * error message.
+ * @param[in] transactionId - Unique identifier of the journal entries
+ * to be committed.
+ * @param[in] errMsg - The error exception message associated with the
+ * error log to be committed.
+ */
+ void commit(uint64_t transactionId, std::string errMsg) override;
- /*
- * @fn commit()
- * @brief sd_bus CommitWithLvl method implementation callback.
- * @details Create an error/event log based on transaction id and
- * error message.
- * @param[in] transactionId - Unique identifier of the journal entries
- * to be committed.
- * @param[in] errMsg - The error exception message associated with the
- * error log to be committed.
- * @param[in] errLvl - level of the error
- */
- void commitWithLvl(uint64_t transactionId, std::string errMsg,
- uint32_t errLvl) override;
+ /*
+ * @fn commit()
+ * @brief sd_bus CommitWithLvl method implementation callback.
+ * @details Create an error/event log based on transaction id and
+ * error message.
+ * @param[in] transactionId - Unique identifier of the journal entries
+ * to be committed.
+ * @param[in] errMsg - The error exception message associated with the
+ * error log to be committed.
+ * @param[in] errLvl - level of the error
+ */
+ void commitWithLvl(uint64_t transactionId, std::string errMsg,
+ uint32_t errLvl) override;
- /** @brief Erase specified entry d-bus object
- *
- * @param[in] entryId - unique identifier of the entry
- */
- void erase(uint32_t entryId);
+ /** @brief Erase specified entry d-bus object
+ *
+ * @param[in] entryId - unique identifier of the entry
+ */
+ void erase(uint32_t entryId);
- /** @brief Construct error d-bus objects from their persisted
- * representations.
- */
- void restore();
+ /** @brief Construct error d-bus objects from their persisted
+ * representations.
+ */
+ void restore();
- /** @brief Erase all error log entries
- *
- */
- void eraseAll()
+ /** @brief Erase all error log entries
+ *
+ */
+ void eraseAll()
+ {
+ auto iter = entries.begin();
+ while (iter != entries.end())
{
- auto iter = entries.begin();
- while (iter != entries.end())
- {
- auto entry = iter->first;
- ++iter;
- erase(entry);
- }
+ auto entry = iter->first;
+ ++iter;
+ erase(entry);
}
+ }
- /** @brief Returns the count of high severity errors
- *
- * @return int - count of real errors
- */
- int getRealErrSize();
+ /** @brief Returns the count of high severity errors
+ *
+ * @return int - count of real errors
+ */
+ int getRealErrSize();
- /** @brief Returns the count of Info errors
- *
- * @return int - count of info errors
- */
- int getInfoErrSize();
+ /** @brief Returns the count of Info errors
+ *
+ * @return int - count of info errors
+ */
+ int getInfoErrSize();
- private:
- /*
- * @fn _commit()
- * @brief commit() helper
- * @param[in] transactionId - Unique identifier of the journal entries
- * to be committed.
- * @param[in] errMsg - The error exception message associated with the
- * error log to be committed.
- * @param[in] errLvl - level of the error
- */
- void _commit(uint64_t transactionId, std::string&& errMsg,
- Entry::Level errLvl);
+ private:
+ /*
+ * @fn _commit()
+ * @brief commit() helper
+ * @param[in] transactionId - Unique identifier of the journal entries
+ * to be committed.
+ * @param[in] errMsg - The error exception message associated with the
+ * error log to be committed.
+ * @param[in] errLvl - level of the error
+ */
+ void _commit(uint64_t transactionId, std::string&& errMsg,
+ Entry::Level errLvl);
- /** @brief Call metadata handler(s), if any. Handlers may create
- * associations.
- * @param[in] errorName - name of the error
- * @param[in] additionalData - list of metadata (in key=value format)
- * @param[out] objects - list of error's association objects
- */
- void processMetadata(const std::string& errorName,
- const std::vector<std::string>& additionalData,
- AssociationList& objects) const;
+ /** @brief Call metadata handler(s), if any. Handlers may create
+ * associations.
+ * @param[in] errorName - name of the error
+ * @param[in] additionalData - list of metadata (in key=value format)
+ * @param[out] objects - list of error's association objects
+ */
+ void processMetadata(const std::string& errorName,
+ const std::vector<std::string>& additionalData,
+ AssociationList& objects) const;
- /** @brief Synchronize unwritten journal messages to disk.
- * @details This is the same implementation as the systemd command
- * "journalctl --sync".
- */
- void journalSync();
+ /** @brief Synchronize unwritten journal messages to disk.
+ * @details This is the same implementation as the systemd command
+ * "journalctl --sync".
+ */
+ void journalSync();
- /** @brief Reads the BMC code level
- *
- * @return std::string - the version string
- */
- static std::string readFWVersion();
+ /** @brief Reads the BMC code level
+ *
+ * @return std::string - the version string
+ */
+ static std::string readFWVersion();
- /** @brief Persistent sdbusplus DBus bus connection. */
- sdbusplus::bus::bus& busLog;
+ /** @brief Persistent sdbusplus DBus bus connection. */
+ sdbusplus::bus::bus& busLog;
- /** @brief Persistent map of Entry dbus objects and their ID */
- std::map<uint32_t, std::unique_ptr<Entry>> entries;
+ /** @brief Persistent map of Entry dbus objects and their ID */
+ std::map<uint32_t, std::unique_ptr<Entry>> entries;
- /** @brief List of error ids for high severity errors */
- std::list<uint32_t> realErrors;
+ /** @brief List of error ids for high severity errors */
+ std::list<uint32_t> realErrors;
- /** @brief List of error ids for Info(and below) severity */
- std::list<uint32_t> infoErrors;
+ /** @brief List of error ids for Info(and below) severity */
+ std::list<uint32_t> infoErrors;
- /** @brief Id of last error log entry */
- uint32_t entryId;
+ /** @brief Id of last error log entry */
+ uint32_t entryId;
- /** @brief The BMC firmware version */
- const std::string fwVersion;
+ /** @brief The BMC firmware version */
+ const std::string fwVersion;
};
-} //namespace internal
+} // namespace internal
/** @class Manager
* @brief Implementation for delete all error log entries.
@@ -183,36 +182,36 @@
*/
class Manager : public DeleteAllIface
{
- public:
- Manager() = delete;
- Manager(const Manager&) = delete;
- Manager& operator=(const Manager&) = delete;
- Manager(Manager&&) = delete;
- Manager& operator=(Manager&&) = delete;
- virtual ~Manager() = default;
+ public:
+ Manager() = delete;
+ Manager(const Manager&) = delete;
+ Manager& operator=(const Manager&) = delete;
+ Manager(Manager&&) = delete;
+ Manager& operator=(Manager&&) = delete;
+ virtual ~Manager() = default;
- /** @brief Constructor to put object onto bus at a dbus path.
- * Defer signal registration (pass true for deferSignal to the
- * base class) until after the properties are set.
- * @param[in] bus - Bus to attach to.
- * @param[in] path - Path to attach at.
- * @param[in] manager - Reference to internal manager object.
- */
- Manager(sdbusplus::bus::bus& bus,
- const std::string& path,
- internal::Manager& manager) :
- DeleteAllIface(bus, path.c_str(), true),
- manager(manager) {};
+ /** @brief Constructor to put object onto bus at a dbus path.
+ * Defer signal registration (pass true for deferSignal to the
+ * base class) until after the properties are set.
+ * @param[in] bus - Bus to attach to.
+ * @param[in] path - Path to attach at.
+ * @param[in] manager - Reference to internal manager object.
+ */
+ Manager(sdbusplus::bus::bus& bus, const std::string& path,
+ internal::Manager& manager) :
+ DeleteAllIface(bus, path.c_str(), true),
+ manager(manager){};
- /** @brief Delete all d-bus objects.
- */
- void deleteAll()
- {
- manager.eraseAll();
- }
- private:
- /** @brief This is a reference to manager object */
- internal::Manager& manager;
+ /** @brief Delete all d-bus objects.
+ */
+ void deleteAll()
+ {
+ manager.eraseAll();
+ }
+
+ private:
+ /** @brief This is a reference to manager object */
+ internal::Manager& manager;
};
} // namespace logging
diff --git a/log_manager_main.cpp b/log_manager_main.cpp
index f06a79d..48a7dee 100644
--- a/log_manager_main.cpp
+++ b/log_manager_main.cpp
@@ -1,10 +1,12 @@
-#include <sdbusplus/bus.hpp>
-#include <sdbusplus/server/manager.hpp>
-#include <experimental/filesystem>
#include "config.h"
+
#include "log_manager.hpp"
-int main(int argc, char *argv[])
+#include <experimental/filesystem>
+#include <sdbusplus/bus.hpp>
+#include <sdbusplus/server/manager.hpp>
+
+int main(int argc, char* argv[])
{
auto bus = sdbusplus::bus::new_default();
@@ -23,7 +25,7 @@
bus.request_name(BUSNAME_LOGGING);
- while(true)
+ while (true)
{
bus.process_discard();
bus.wait();
diff --git a/logging_test.cpp b/logging_test.cpp
index 0ad10b7..3a144f8 100644
--- a/logging_test.cpp
+++ b/logging_test.cpp
@@ -1,18 +1,19 @@
// A basic unit test that runs on a BMC (qemu or hardware)
#include <getopt.h>
-#include <iostream>
#include <systemd/sd-journal.h>
-#include <sstream>
-#include <sdbusplus/exception.hpp>
+
+#include <iostream>
+#include <phosphor-logging/elog-errors.hpp>
#include <phosphor-logging/elog.hpp>
#include <phosphor-logging/log.hpp>
-#include <phosphor-logging/elog-errors.hpp>
+#include <sdbusplus/exception.hpp>
+#include <sstream>
using namespace phosphor;
using namespace logging;
-const char *usage = "Usage: logging-test [OPTION] \n\n\
+const char* usage = "Usage: logging-test [OPTION] \n\n\
Options: \n\
[NONE] Default test case. \n\
-h, --help Display this usage text. \n\
@@ -21,29 +22,31 @@
AutoTestSimple, AutoTestCreateAndCommit\n";
// validate the journal metadata equals the input value
-int validate_journal(const char *i_entry, const char *i_value)
+int validate_journal(const char* i_entry, const char* i_value)
{
- sd_journal *journal;
- const void *data;
+ sd_journal* journal;
+ const void* data;
size_t l;
int rc;
bool validated = false;
rc = sd_journal_open(&journal, SD_JOURNAL_LOCAL_ONLY);
- if (rc < 0) {
- std::cerr << "Failed to open journal: " << strerror(-rc) << "\n";
- return 1;
+ if (rc < 0)
+ {
+ std::cerr << "Failed to open journal: " << strerror(-rc) << "\n";
+ return 1;
}
rc = sd_journal_query_unique(journal, i_entry);
- if (rc < 0) {
- std::cerr << "Failed to query journal: " << strerror(-rc) << "\n";
- return 1;
+ if (rc < 0)
+ {
+ std::cerr << "Failed to query journal: " << strerror(-rc) << "\n";
+ return 1;
}
SD_JOURNAL_FOREACH_UNIQUE(journal, data, l)
{
std::string journ_entry((const char*)data);
std::cout << journ_entry << "\n";
- if(journ_entry.find(i_value) != std::string::npos)
+ if (journ_entry.find(i_value) != std::string::npos)
{
std::cout << "We found it!\n";
validated = true;
@@ -54,10 +57,11 @@
sd_journal_close(journal);
rc = (validated) ? 0 : 1;
- if(rc)
+ if (rc)
{
std::cerr << "Failed to find " << i_entry << " with value " << i_value
- <<" in journal!" << "\n";
+ << " in journal!"
+ << "\n";
}
return rc;
@@ -69,35 +73,36 @@
log<level::DEBUG>("Basic phosphor logging test");
// TEST 2 - Log with metadata field
- const char *file_name = "phosphor_logging_test.txt";
+ const char* file_name = "phosphor_logging_test.txt";
int number = 0xFEFE;
- log<level::DEBUG>("phosphor logging test with attribute",
- entry("FILE_NAME_WITH_NUM_TEST=%s_%x", file_name, number));
+ log<level::DEBUG>(
+ "phosphor logging test with attribute",
+ entry("FILE_NAME_WITH_NUM_TEST=%s_%x", file_name, number));
// Now read back and verify our data made it into the journal
int rc = validate_journal("FILE_NAME_WITH_NUM_TEST",
"phosphor_logging_test.txt_fefe");
- if(rc)
- return(rc);
+ if (rc)
+ return (rc);
// TEST 3 - Create error log with 2 meta data fields (rvalue and lvalue)
number = 0x1234;
- const char *test_string = "/tmp/test_string/";
+ const char* test_string = "/tmp/test_string/";
try
{
elog<example::xyz::openbmc_project::Example::Elog::TestErrorOne>(
- example::xyz::openbmc_project::Example::Elog::
- TestErrorOne::ERRNUM(number),
- example::xyz::openbmc_project::Example::Elog::
- TestErrorOne::FILE_PATH(test_string),
- example::xyz::openbmc_project::Example::Elog::
- TestErrorOne::FILE_NAME("elog_test_3.txt"),
- example::xyz::openbmc_project::Example::Elog::
- TestErrorTwo::DEV_ADDR(0xDEADDEAD),
- example::xyz::openbmc_project::Example::Elog::
- TestErrorTwo::DEV_ID(100),
- example::xyz::openbmc_project::Example::Elog::
- TestErrorTwo::DEV_NAME("test case 3"));
+ example::xyz::openbmc_project::Example::Elog::TestErrorOne::ERRNUM(
+ number),
+ example::xyz::openbmc_project::Example::Elog::TestErrorOne::
+ FILE_PATH(test_string),
+ example::xyz::openbmc_project::Example::Elog::TestErrorOne::
+ FILE_NAME("elog_test_3.txt"),
+ example::xyz::openbmc_project::Example::Elog::TestErrorTwo::
+ DEV_ADDR(0xDEADDEAD),
+ example::xyz::openbmc_project::Example::Elog::TestErrorTwo::DEV_ID(
+ 100),
+ example::xyz::openbmc_project::Example::Elog::TestErrorTwo::
+ DEV_NAME("test case 3"));
}
catch (example::xyz::openbmc_project::Example::Elog::TestErrorOne& e)
{
@@ -112,44 +117,39 @@
stream << std::hex << number;
rc = validate_journal(TestErrorOne::ERRNUM::str_short,
std::string(stream.str()).c_str());
- if(rc)
- return(rc);
+ if (rc)
+ return (rc);
- rc = validate_journal(TestErrorOne::FILE_PATH::str_short,
- test_string);
- if(rc)
- return(rc);
+ rc = validate_journal(TestErrorOne::FILE_PATH::str_short, test_string);
+ if (rc)
+ return (rc);
- rc = validate_journal(TestErrorOne::FILE_NAME::str_short,
- "elog_test_3.txt");
- if(rc)
- return(rc);
+ rc =
+ validate_journal(TestErrorOne::FILE_NAME::str_short, "elog_test_3.txt");
+ if (rc)
+ return (rc);
- rc = validate_journal(TestErrorTwo::DEV_ADDR::str_short,
- "0xDEADDEAD");
- if(rc)
- return(rc);
+ rc = validate_journal(TestErrorTwo::DEV_ADDR::str_short, "0xDEADDEAD");
+ if (rc)
+ return (rc);
- rc = validate_journal(TestErrorTwo::DEV_ID::str_short,
- "100");
- if(rc)
- return(rc);
+ rc = validate_journal(TestErrorTwo::DEV_ID::str_short, "100");
+ if (rc)
+ return (rc);
- rc = validate_journal(TestErrorTwo::DEV_NAME::str_short,
- "test case 3");
- if(rc)
- return(rc);
+ rc = validate_journal(TestErrorTwo::DEV_NAME::str_short, "test case 3");
+ if (rc)
+ return (rc);
// TEST 4 - Create error log with previous entry use
number = 0x9876;
try
{
- elog<TestErrorOne>(TestErrorOne::ERRNUM(number),
- prev_entry<TestErrorOne::FILE_PATH>(),
- TestErrorOne::FILE_NAME("elog_test_4.txt"),
- TestErrorTwo::DEV_ADDR(0xDEADDEAD),
- TestErrorTwo::DEV_ID(100),
- TestErrorTwo::DEV_NAME("test case 4"));
+ elog<TestErrorOne>(
+ TestErrorOne::ERRNUM(number), prev_entry<TestErrorOne::FILE_PATH>(),
+ TestErrorOne::FILE_NAME("elog_test_4.txt"),
+ TestErrorTwo::DEV_ADDR(0xDEADDEAD), TestErrorTwo::DEV_ID(100),
+ TestErrorTwo::DEV_NAME("test case 4"));
}
catch (sdbusplus::exception_t& e)
{
@@ -161,62 +161,56 @@
stream << std::hex << number;
rc = validate_journal(TestErrorOne::ERRNUM::str_short,
std::string(stream.str()).c_str());
- if(rc)
- return(rc);
+ if (rc)
+ return (rc);
// This should just be equal to what we put in test 3
- rc = validate_journal(TestErrorOne::FILE_PATH::str_short,
- test_string);
- if(rc)
- return(rc);
+ rc = validate_journal(TestErrorOne::FILE_PATH::str_short, test_string);
+ if (rc)
+ return (rc);
- rc = validate_journal(TestErrorOne::FILE_NAME::str_short,
- "elog_test_4.txt");
- if(rc)
- return(rc);
+ rc =
+ validate_journal(TestErrorOne::FILE_NAME::str_short, "elog_test_4.txt");
+ if (rc)
+ return (rc);
- rc = validate_journal(TestErrorTwo::DEV_ADDR::str_short,
- "0xDEADDEAD");
- if(rc)
- return(rc);
+ rc = validate_journal(TestErrorTwo::DEV_ADDR::str_short, "0xDEADDEAD");
+ if (rc)
+ return (rc);
- rc = validate_journal(TestErrorTwo::DEV_ID::str_short,
- "100");
- if(rc)
- return(rc);
+ rc = validate_journal(TestErrorTwo::DEV_ID::str_short, "100");
+ if (rc)
+ return (rc);
- rc = validate_journal(TestErrorTwo::DEV_NAME::str_short,
- "test case 4");
- if(rc)
- return(rc);
+ rc = validate_journal(TestErrorTwo::DEV_NAME::str_short, "test case 4");
+ if (rc)
+ return (rc);
// Compile fail tests
// Simple test to prove we fail to compile due to missing param
- //elog<TestErrorOne>(TestErrorOne::ERRNUM(1),
+ // elog<TestErrorOne>(TestErrorOne::ERRNUM(1),
// TestErrorOne::FILE_PATH("test"));
// Simple test to prove we fail to compile due to invalid param
- //elog<TestErrorOne>(TestErrorOne::ERRNUM(1),
+ // elog<TestErrorOne>(TestErrorOne::ERRNUM(1),
// TestErrorOne::FILE_PATH("test"),
// TestErrorOne::FILE_NAME(1));
return 0;
}
-void commitError(const char *text)
+void commitError(const char* text)
{
if (strcmp(text, "AutoTestSimple") == 0)
{
try
{
- elog<example::xyz::openbmc_project::Example::Elog::
- AutoTestSimple>(
- example::xyz::openbmc_project::Example::Elog::
- AutoTestSimple::STRING("FOO"));
+ elog<example::xyz::openbmc_project::Example::Elog::AutoTestSimple>(
+ example::xyz::openbmc_project::Example::Elog::AutoTestSimple::
+ STRING("FOO"));
}
- catch (example::xyz::openbmc_project::Example::Elog::
- AutoTestSimple& e)
+ catch (example::xyz::openbmc_project::Example::Elog::AutoTestSimple& e)
{
std::cout << "elog exception caught: " << e.what() << std::endl;
commit(e.name());
@@ -224,31 +218,29 @@
}
else if (strcmp(text, "AutoTestCreateAndCommit") == 0)
{
- report<example::xyz::openbmc_project::Example::Elog::
- AutoTestSimple>(
- example::xyz::openbmc_project::Example::Elog::
- AutoTestSimple::STRING("FOO"));
+ report<example::xyz::openbmc_project::Example::Elog::AutoTestSimple>(
+ example::xyz::openbmc_project::Example::Elog::AutoTestSimple::
+ STRING("FOO"));
}
return;
}
-int main(int argc, char *argv[])
+int main(int argc, char* argv[])
{
char arg;
if (argc == 1)
return elog_test();
- static struct option long_options[] =
- {
- {"help", no_argument, 0, 'h'},
- {"commit", required_argument, 0, 'c'},
- {0, 0, 0, 0}
- };
+ static struct option long_options[] = {
+ {"help", no_argument, 0, 'h'},
+ {"commit", required_argument, 0, 'c'},
+ {0, 0, 0, 0}};
int option_index = 0;
- while((arg=getopt_long(argc,argv,"hc:",long_options,&option_index)) != -1)
+ while ((arg = getopt_long(argc, argv, "hc:", long_options,
+ &option_index)) != -1)
{
switch (arg)
{
@@ -259,10 +251,8 @@
case '?':
std::cerr << usage;
return 1;
-
}
}
return 0;
-
}
diff --git a/org.openbmc.Associations.cpp b/org.openbmc.Associations.cpp
index d4580c9..25d79fd 100644
--- a/org.openbmc.Associations.cpp
+++ b/org.openbmc.Associations.cpp
@@ -1,7 +1,7 @@
#include <algorithm>
-#include <sdbusplus/server.hpp>
-#include <sdbusplus/exception.hpp>
#include <org/openbmc/Associations/server.hpp>
+#include <sdbusplus/exception.hpp>
+#include <sdbusplus/server.hpp>
namespace sdbusplus
{
@@ -12,15 +12,15 @@
namespace server
{
-Associations::Associations(bus::bus& bus, const char* path)
- : _org_openbmc_Associations_interface(
- bus, path, _interface, _vtable, this)
+Associations::Associations(bus::bus& bus, const char* path) :
+ _org_openbmc_Associations_interface(bus, path, _interface, _vtable, this)
{
}
-Associations::Associations(bus::bus& bus, const char* path,
- const std::map<std::string, PropertiesVariant>& vals)
- : Associations(bus, path)
+Associations::Associations(
+ bus::bus& bus, const char* path,
+ const std::map<std::string, PropertiesVariant>& vals) :
+ Associations(bus, path)
{
for (const auto& v : vals)
{
@@ -28,18 +28,17 @@
}
}
-
-
-auto Associations::associations() const ->
- std::vector<std::tuple<std::string, std::string, std::string>>
+auto Associations::associations() const
+ -> std::vector<std::tuple<std::string, std::string, std::string>>
{
return _associations;
}
-int Associations::_callback_get_associations(
- sd_bus* bus, const char* path, const char* interface,
- const char* property, sd_bus_message* reply, void* context,
- sd_bus_error* error)
+int Associations::_callback_get_associations(sd_bus* bus, const char* path,
+ const char* interface,
+ const char* property,
+ sd_bus_message* reply,
+ void* context, sd_bus_error* error)
{
using sdbusplus::server::binding::details::convertForMessage;
@@ -50,15 +49,15 @@
{
auto tbus = m.get_bus();
sdbusplus::server::transaction::Transaction t(tbus, m);
- sdbusplus::server::transaction::set_id
- (std::hash<sdbusplus::server::transaction::Transaction>{}(t));
+ sdbusplus::server::transaction::set_id(
+ std::hash<sdbusplus::server::transaction::Transaction>{}(t));
}
#endif
auto o = static_cast<Associations*>(context);
m.append(convertForMessage(o->associations()));
}
- catch(sdbusplus::internal_exception_t& e)
+ catch (sdbusplus::internal_exception_t& e)
{
sd_bus_error_set_const(error, e.name(), e.description());
return -EINVAL;
@@ -67,8 +66,9 @@
return true;
}
-auto Associations::associations(std::vector<std::tuple<std::string, std::string, std::string>> value) ->
- std::vector<std::tuple<std::string, std::string, std::string>>
+auto Associations::associations(
+ std::vector<std::tuple<std::string, std::string, std::string>> value)
+ -> std::vector<std::tuple<std::string, std::string, std::string>>
{
if (_associations != value)
{
@@ -79,10 +79,11 @@
return _associations;
}
-int Associations::_callback_set_associations(
- sd_bus* bus, const char* path, const char* interface,
- const char* property, sd_bus_message* value, void* context,
- sd_bus_error* error)
+int Associations::_callback_set_associations(sd_bus* bus, const char* path,
+ const char* interface,
+ const char* property,
+ sd_bus_message* value,
+ void* context, sd_bus_error* error)
{
try
{
@@ -91,8 +92,8 @@
{
auto tbus = m.get_bus();
sdbusplus::server::transaction::Transaction t(tbus, m);
- sdbusplus::server::transaction::set_id
- (std::hash<sdbusplus::server::transaction::Transaction>{}(t));
+ sdbusplus::server::transaction::set_id(
+ std::hash<sdbusplus::server::transaction::Transaction>{}(t));
}
#endif
@@ -102,7 +103,7 @@
m.read(v);
o->associations(v);
}
- catch(sdbusplus::internal_exception_t& e)
+ catch (sdbusplus::internal_exception_t& e)
{
sd_bus_error_set_const(error, e.name(), e.description());
return -EINVAL;
@@ -115,25 +116,27 @@
{
namespace Associations
{
-static const auto _property_associations =
- utility::tuple_to_array(message::types::type_id<
- std::vector<std::tuple<std::string, std::string, std::string>>>());
+static const auto _property_associations = utility::tuple_to_array(
+ message::types::type_id<
+ std::vector<std::tuple<std::string, std::string, std::string>>>());
}
-}
+} // namespace details
void Associations::setPropertyByName(const std::string& name,
const PropertiesVariant& val)
{
if (name == "associations")
{
- auto& v = message::variant_ns::get<std::vector<std::tuple<std::string, std::string, std::string>>>(val);
+ auto& v = message::variant_ns::get<
+ std::vector<std::tuple<std::string, std::string, std::string>>>(
+ val);
associations(v);
return;
}
}
-auto Associations::getPropertyByName(const std::string& name) ->
- PropertiesVariant
+auto Associations::getPropertyByName(const std::string& name)
+ -> PropertiesVariant
{
if (name == "associations")
{
@@ -143,20 +146,15 @@
return PropertiesVariant();
}
-
const vtable::vtable_t Associations::_vtable[] = {
vtable::start(),
vtable::property("associations",
- details::Associations::_property_associations
- .data(),
- _callback_get_associations,
- _callback_set_associations,
+ details::Associations::_property_associations.data(),
+ _callback_get_associations, _callback_set_associations,
vtable::property_::emits_change),
- vtable::end()
-};
+ vtable::end()};
} // namespace server
} // namespace openbmc
} // namespace org
} // namespace sdbusplus
-
diff --git a/org/openbmc/Associations/server.hpp b/org/openbmc/Associations/server.hpp
index 5fc71e0..fb3ff7e 100644
--- a/org/openbmc/Associations/server.hpp
+++ b/org/openbmc/Associations/server.hpp
@@ -1,7 +1,8 @@
#pragma once
-#include <tuple>
#include <systemd/sd-bus.h>
+
#include <sdbusplus/server.hpp>
+#include <tuple>
namespace sdbusplus
{
@@ -14,88 +15,83 @@
class Associations
{
- public:
- /* Define all of the basic class operations:
- * Not allowed:
- * - Default constructor to avoid nullptrs.
- * - Copy operations due to internal unique_ptr.
- * - Move operations due to 'this' being registered as the
- * 'context' with sdbus.
- * Allowed:
- * - Destructor.
- */
- Associations() = delete;
- Associations(const Associations&) = delete;
- Associations& operator=(const Associations&) = delete;
- Associations(Associations&&) = delete;
- Associations& operator=(Associations&&) = delete;
- virtual ~Associations() = default;
+ public:
+ /* Define all of the basic class operations:
+ * Not allowed:
+ * - Default constructor to avoid nullptrs.
+ * - Copy operations due to internal unique_ptr.
+ * - Move operations due to 'this' being registered as the
+ * 'context' with sdbus.
+ * Allowed:
+ * - Destructor.
+ */
+ Associations() = delete;
+ Associations(const Associations&) = delete;
+ Associations& operator=(const Associations&) = delete;
+ Associations(Associations&&) = delete;
+ Associations& operator=(Associations&&) = delete;
+ virtual ~Associations() = default;
- /** @brief Constructor to put object onto bus at a dbus path.
- * @param[in] bus - Bus to attach to.
- * @param[in] path - Path to attach at.
- */
- Associations(bus::bus& bus, const char* path);
+ /** @brief Constructor to put object onto bus at a dbus path.
+ * @param[in] bus - Bus to attach to.
+ * @param[in] path - Path to attach at.
+ */
+ Associations(bus::bus& bus, const char* path);
+ using PropertiesVariant = sdbusplus::message::variant<
+ std::vector<std::tuple<std::string, std::string, std::string>>>;
- using PropertiesVariant = sdbusplus::message::variant<
- std::vector<std::tuple<std::string, std::string, std::string>>>;
+ /** @brief Constructor to initialize the object from a map of
+ * properties.
+ *
+ * @param[in] bus - Bus to attach to.
+ * @param[in] path - Path to attach at.
+ * @param[in] vals - Map of property name to value for initialization.
+ */
+ Associations(bus::bus& bus, const char* path,
+ const std::map<std::string, PropertiesVariant>& vals);
- /** @brief Constructor to initialize the object from a map of
- * properties.
- *
- * @param[in] bus - Bus to attach to.
- * @param[in] path - Path to attach at.
- * @param[in] vals - Map of property name to value for initialization.
- */
- Associations(bus::bus& bus, const char* path,
- const std::map<std::string, PropertiesVariant>& vals);
+ /** Get value of associations */
+ virtual std::vector<std::tuple<std::string, std::string, std::string>>
+ associations() const;
+ /** Set value of associations */
+ virtual std::vector<std::tuple<std::string, std::string, std::string>>
+ associations(
+ std::vector<std::tuple<std::string, std::string, std::string>>
+ value);
+ /** @brief Sets a property by name.
+ * @param[in] name - A string representation of the property name.
+ * @param[in] val - A variant containing the value to set.
+ */
+ void setPropertyByName(const std::string& name,
+ const PropertiesVariant& val);
+ /** @brief Gets a property by name.
+ * @param[in] name - A string representation of the property name.
+ * @return - A variant containing the value of the property.
+ */
+ PropertiesVariant getPropertyByName(const std::string& name);
- /** Get value of associations */
- virtual std::vector<std::tuple<std::string, std::string, std::string>> associations() const;
- /** Set value of associations */
- virtual std::vector<std::tuple<std::string, std::string, std::string>> associations(std::vector<std::tuple<std::string, std::string, std::string>> value);
+ private:
+ /** @brief sd-bus callback for get-property 'associations' */
+ static int _callback_get_associations(sd_bus*, const char*, const char*,
+ const char*, sd_bus_message*, void*,
+ sd_bus_error*);
+ /** @brief sd-bus callback for set-property 'associations' */
+ static int _callback_set_associations(sd_bus*, const char*, const char*,
+ const char*, sd_bus_message*, void*,
+ sd_bus_error*);
- /** @brief Sets a property by name.
- * @param[in] name - A string representation of the property name.
- * @param[in] val - A variant containing the value to set.
- */
- void setPropertyByName(const std::string& name,
- const PropertiesVariant& val);
+ static constexpr auto _interface = "org.openbmc.Associations";
+ static const vtable::vtable_t _vtable[];
+ sdbusplus::server::interface::interface _org_openbmc_Associations_interface;
- /** @brief Gets a property by name.
- * @param[in] name - A string representation of the property name.
- * @return - A variant containing the value of the property.
- */
- PropertiesVariant getPropertyByName(const std::string& name);
-
-
- private:
-
- /** @brief sd-bus callback for get-property 'associations' */
- static int _callback_get_associations(
- sd_bus*, const char*, const char*, const char*,
- sd_bus_message*, void*, sd_bus_error*);
- /** @brief sd-bus callback for set-property 'associations' */
- static int _callback_set_associations(
- sd_bus*, const char*, const char*, const char*,
- sd_bus_message*, void*, sd_bus_error*);
-
-
- static constexpr auto _interface = "org.openbmc.Associations";
- static const vtable::vtable_t _vtable[];
- sdbusplus::server::interface::interface
- _org_openbmc_Associations_interface;
-
- std::vector<std::tuple<std::string, std::string, std::string>> _associations{};
-
+ std::vector<std::tuple<std::string, std::string, std::string>>
+ _associations{};
};
-
} // namespace server
} // namespace openbmc
} // namespace org
} // namespace sdbusplus
-
diff --git a/phosphor-logging/elog.hpp b/phosphor-logging/elog.hpp
index b76bd6c..5aace07 100644
--- a/phosphor-logging/elog.hpp
+++ b/phosphor-logging/elog.hpp
@@ -1,9 +1,10 @@
#pragma once
-#include <tuple>
-#include <utility>
+#include "xyz/openbmc_project/Logging/Entry/server.hpp"
+
#include <phosphor-logging/log.hpp>
#include <sdbusplus/exception.hpp>
-#include "xyz/openbmc_project/Logging/Entry/server.hpp"
+#include <tuple>
+#include <utility>
namespace phosphor
{
@@ -16,15 +17,13 @@
/**
* @brief Structure used by callers to indicate they want to use the last value
* put in the journal for input parameter.
-*/
+ */
template <typename T>
struct prev_entry
{
using type = T;
};
-
-
namespace details
{
/**
@@ -40,7 +39,10 @@
{
using type = T;
- auto get() { return value._entry; }
+ auto get()
+ {
+ return value._entry;
+ }
T value;
};
@@ -56,7 +58,10 @@
struct deduce_entry_type<prev_entry<T>>
{
using type = T;
- auto get() { return std::make_tuple(); }
+ auto get()
+ {
+ return std::make_tuple();
+ }
prev_entry<T> value;
};
@@ -64,8 +69,8 @@
/**
* @brief Typedef for above structure usage
*/
-template <typename T> using deduce_entry_type_t =
- typename deduce_entry_type<T>::type;
+template <typename T>
+using deduce_entry_type_t = typename deduce_entry_type<T>::type;
/**
* @brief Used to map an sdbusplus error to a phosphor-logging error type
@@ -84,8 +89,8 @@
/**
* @brief Typedef for above structure usage
*/
-template <typename T> using map_exception_type_t =
- typename map_exception_type<T>::type;
+template <typename T>
+using map_exception_type_t = typename map_exception_type<T>::type;
/** @fn commit()
* @brief Create an error log entry based on journal
@@ -116,10 +121,8 @@
void commit()
{
// Validate if the exception is derived from sdbusplus::exception.
- static_assert(
- std::is_base_of<sdbusplus::exception::exception, T>::value,
- "T must be a descendant of sdbusplus::exception::exception"
- );
+ static_assert(std::is_base_of<sdbusplus::exception::exception, T>::value,
+ "T must be a descendant of sdbusplus::exception::exception");
details::commit(T::errName);
}
@@ -132,40 +135,32 @@
void commit(Entry::Level level)
{
// Validate if the exception is derived from sdbusplus::exception.
- static_assert(
- std::is_base_of<sdbusplus::exception::exception, T>::value,
- "T must be a descendant of sdbusplus::exception::exception"
- );
+ static_assert(std::is_base_of<sdbusplus::exception::exception, T>::value,
+ "T must be a descendant of sdbusplus::exception::exception");
details::commit(T::errName, level);
}
-
/** @fn elog()
* @brief Create a journal log entry based on predefined
* error log information
* @tparam T - Error log type
* @param[in] i_args - Metadata fields to be added to the journal entry
*/
-template <typename T, typename ...Args>
+template <typename T, typename... Args>
void elog(Args... i_args)
{
// Validate if the exception is derived from sdbusplus::exception.
- static_assert(
- std::is_base_of<sdbusplus::exception::exception, T>::value,
- "T must be a descendant of sdbusplus::exception::exception"
- );
+ static_assert(std::is_base_of<sdbusplus::exception::exception, T>::value,
+ "T must be a descendant of sdbusplus::exception::exception");
// Validate the caller passed in the required parameters
- static_assert(std::is_same<typename details::
- map_exception_type_t<T>::metadata_types,
- std::tuple<
- details::deduce_entry_type_t<Args>...>>
- ::value,
- "You are not passing in required arguments for this error");
+ static_assert(
+ std::is_same<typename details::map_exception_type_t<T>::metadata_types,
+ std::tuple<details::deduce_entry_type_t<Args>...>>::value,
+ "You are not passing in required arguments for this error");
log<details::map_exception_type_t<T>::L>(
- T::errDesc,
- details::deduce_entry_type<Args>{i_args}.get()...);
+ T::errDesc, details::deduce_entry_type<Args>{i_args}.get()...);
// Now throw an exception for this error
throw T();
@@ -177,26 +172,21 @@
* @tparam T - exception
* @param[in] i_args - Metadata fields to be added to the journal entry
*/
-template <typename T, typename ...Args>
+template <typename T, typename... Args>
void report(Args... i_args)
{
- //validate if the exception is derived from sdbusplus::exception.
- static_assert(
- std::is_base_of<sdbusplus::exception::exception, T>::value,
- "T must be a descendant of sdbusplus::exception::exception"
- );
+ // validate if the exception is derived from sdbusplus::exception.
+ static_assert(std::is_base_of<sdbusplus::exception::exception, T>::value,
+ "T must be a descendant of sdbusplus::exception::exception");
// Validate the caller passed in the required parameters
- static_assert(std::is_same<typename details::
- map_exception_type_t<T>::metadata_types,
- std::tuple<
- details::deduce_entry_type_t<Args>...>>
- ::value,
- "You are not passing in required arguments for this error");
+ static_assert(
+ std::is_same<typename details::map_exception_type_t<T>::metadata_types,
+ std::tuple<details::deduce_entry_type_t<Args>...>>::value,
+ "You are not passing in required arguments for this error");
log<details::map_exception_type_t<T>::L>(
- T::errDesc,
- details::deduce_entry_type<Args>{i_args}.get()...);
+ T::errDesc, details::deduce_entry_type<Args>{i_args}.get()...);
commit<T>();
}
@@ -209,26 +199,21 @@
* @param[in] level - level of the error
* @param[in] i_args - Metadata fields to be added to the journal entry
*/
-template <typename T, typename ...Args>
+template <typename T, typename... Args>
void report(Entry::Level level, Args... i_args)
{
- //validate if the exception is derived from sdbusplus::exception.
- static_assert(
- std::is_base_of<sdbusplus::exception::exception, T>::value,
- "T must be a descendant of sdbusplus::exception::exception"
- );
+ // validate if the exception is derived from sdbusplus::exception.
+ static_assert(std::is_base_of<sdbusplus::exception::exception, T>::value,
+ "T must be a descendant of sdbusplus::exception::exception");
// Validate the caller passed in the required parameters
- static_assert(std::is_same<typename details::
- map_exception_type_t<T>::metadata_types,
- std::tuple<
- details::deduce_entry_type_t<Args>...>>
- ::value,
- "You are not passing in required arguments for this error");
+ static_assert(
+ std::is_same<typename details::map_exception_type_t<T>::metadata_types,
+ std::tuple<details::deduce_entry_type_t<Args>...>>::value,
+ "You are not passing in required arguments for this error");
log<details::map_exception_type_t<T>::L>(
- T::errDesc,
- details::deduce_entry_type<Args>{i_args}.get()...);
+ T::errDesc, details::deduce_entry_type<Args>{i_args}.get()...);
commit<T>(level);
}
@@ -236,4 +221,3 @@
} // namespace logging
} // namespace phosphor
-
diff --git a/phosphor-logging/log.hpp b/phosphor-logging/log.hpp
index a933591..e20605b 100644
--- a/phosphor-logging/log.hpp
+++ b/phosphor-logging/log.hpp
@@ -16,10 +16,11 @@
#pragma once
+#include <systemd/sd-journal.h>
+
+#include <sdbusplus/server/transaction.hpp>
#include <tuple>
#include <type_traits>
-#include <systemd/sd-journal.h>
-#include <sdbusplus/server/transaction.hpp>
namespace phosphor
{
@@ -32,14 +33,14 @@
*/
enum class level
{
- EMERG = LOG_EMERG,
- ALERT = LOG_ALERT,
- CRIT = LOG_CRIT,
- ERR = LOG_ERR,
+ EMERG = LOG_EMERG,
+ ALERT = LOG_ALERT,
+ CRIT = LOG_CRIT,
+ ERR = LOG_ERR,
WARNING = LOG_WARNING,
- NOTICE = LOG_NOTICE,
- INFO = LOG_INFO,
- DEBUG = LOG_DEBUG,
+ NOTICE = LOG_NOTICE,
+ INFO = LOG_INFO,
+ DEBUG = LOG_DEBUG,
};
/** @fn log()
@@ -55,7 +56,7 @@
* msg_str,
* entry("MY_METADATA=%s_%x, name, number));
*/
-template <level L, typename Msg, typename ...Entry>
+template <level L, typename Msg, typename... Entry>
void log(Msg msg, Entry... entry);
/** @fn entry()
@@ -68,7 +69,7 @@
* value is of any size and format
* @param[in] args - Remaining metadata strings
*/
-template <typename Arg, typename ...Args>
+template <typename Arg, typename... Args>
constexpr auto entry(Arg&& arg, Args&&... args);
namespace details
@@ -82,7 +83,7 @@
template <level L>
constexpr auto prio()
{
- constexpr const char *prio_str = "PRIORITY=%d";
+ constexpr const char* prio_str = "PRIORITY=%d";
constexpr const auto prio_tuple = std::make_tuple(prio_str, L);
return prio_tuple;
}
@@ -94,7 +95,7 @@
* @param[in] e - Tuple containing the data to be logged
* @param[unnamed] - std::integer_sequence of tuple's index values
*/
-template <typename T, size_t ...I>
+template <typename T, size_t... I>
void helper_log(T&& e, std::integer_sequence<size_t, I...>)
{
// https://www.freedesktop.org/software/systemd/man/sd_journal_print.html
@@ -116,63 +117,63 @@
} // namespace details
-template <level L, typename Msg, typename ...Entry>
+template <level L, typename Msg, typename... Entry>
void log(Msg msg, Entry... e)
{
static_assert((std::is_same<const char*, std::decay_t<Msg>>::value ||
std::is_same<char*, std::decay_t<Msg>>::value),
"First parameter must be a C-string.");
- constexpr const char *msg_str = "MESSAGE=%s";
+ constexpr const char* msg_str = "MESSAGE=%s";
const auto msg_tuple = std::make_tuple(msg_str, std::forward<Msg>(msg));
constexpr auto transactionStr = "TRANSACTION_ID=%lld";
auto transactionId = sdbusplus::server::transaction::get_id();
- auto log_tuple = std::tuple_cat(details::prio<L>(),
- msg_tuple,
+ auto log_tuple = std::tuple_cat(details::prio<L>(), msg_tuple,
entry(transactionStr, transactionId),
std::forward<Entry>(e)...);
details::log(log_tuple);
}
-template<class T>
+template <class T>
struct is_printf_argtype
: std::integral_constant<
- bool,
- (std::is_integral<typename std::remove_reference<T>::type>::value ||
- std::is_enum<typename std::remove_reference<T>::type>::value ||
- std::is_floating_point<typename
- std::remove_reference<T>::type>::value ||
- std::is_pointer<typename std::decay<T>::type>::value)>
-{};
+ bool,
+ (std::is_integral<typename std::remove_reference<T>::type>::value ||
+ std::is_enum<typename std::remove_reference<T>::type>::value ||
+ std::is_floating_point<
+ typename std::remove_reference<T>::type>::value ||
+ std::is_pointer<typename std::decay<T>::type>::value)>
+{
+};
-template<class T>
+template <class T>
struct is_char_ptr_argtype
: std::integral_constant<
- bool,
- (std::is_pointer<typename std::decay<T>::type>::value &&
- std::is_same<typename std::remove_cv<typename
- std::remove_pointer<typename
- std::decay<T>::type>::type>::type,
- char>::value)>
-{};
+ bool,
+ (std::is_pointer<typename std::decay<T>::type>::value &&
+ std::is_same<typename std::remove_cv<typename std::remove_pointer<
+ typename std::decay<T>::type>::type>::type,
+ char>::value)>
+{
+};
-template<bool...>
+template <bool...>
struct bool_pack;
-template<bool... bs>
+template <bool... bs>
using all_true = std::is_same<bool_pack<bs..., true>, bool_pack<true, bs...>>;
-template <typename Arg, typename ...Args>
+template <typename Arg, typename... Args>
constexpr auto entry(Arg&& arg, Args&&... args)
{
static_assert(is_char_ptr_argtype<Arg>::value,
"bad argument type: use char*");
static_assert(all_true<is_printf_argtype<Args>::value...>::value,
"bad argument type: use string.c_str() if needed");
- const auto entry_tuple = std::make_tuple(std::forward<Arg>(arg),
- std::forward<Args>(args)...);
+ const auto entry_tuple =
+ std::make_tuple(std::forward<Arg>(arg), std::forward<Args>(args)...);
return entry_tuple;
}
diff --git a/phosphor-rsyslog-config/main.cpp b/phosphor-rsyslog-config/main.cpp
index 3e4d928..ea0ca03 100644
--- a/phosphor-rsyslog-config/main.cpp
+++ b/phosphor-rsyslog-config/main.cpp
@@ -1,19 +1,19 @@
#include "config.h"
+
#include "server-conf.hpp"
+
#include <sdbusplus/bus.hpp>
-int main(int argc, char *argv[])
+int main(int argc, char* argv[])
{
auto bus = sdbusplus::bus::new_default();
- phosphor::rsyslog_config::Server
- serverConf(bus,
- BUSPATH_REMOTE_LOGGING_CONFIG,
- RSYSLOG_SERVER_CONFIG_FILE);
+ phosphor::rsyslog_config::Server serverConf(
+ bus, BUSPATH_REMOTE_LOGGING_CONFIG, RSYSLOG_SERVER_CONFIG_FILE);
bus.request_name(BUSNAME_SYSLOG_CONFIG);
- while(true)
+ while (true)
{
bus.process_discard();
bus.wait();
diff --git a/phosphor-rsyslog-config/server-conf.cpp b/phosphor-rsyslog-config/server-conf.cpp
index 4bf8c79..4bf167c 100644
--- a/phosphor-rsyslog-config/server-conf.cpp
+++ b/phosphor-rsyslog-config/server-conf.cpp
@@ -1,6 +1,8 @@
#include "server-conf.hpp"
+
#include "utils.hpp"
#include "xyz/openbmc_project/Common/error.hpp"
+
#include <fstream>
#include <phosphor-logging/elog.hpp>
#if __has_include("../../usr/include/phosphor-logging/elog-errors.hpp")
@@ -8,8 +10,8 @@
#else
#include <phosphor-logging/elog-errors.hpp>
#endif
-#include <netdb.h>
#include <arpa/inet.h>
+#include <netdb.h>
namespace phosphor
{
@@ -23,7 +25,7 @@
std::string Server::address(std::string value)
{
using Argument = xyz::openbmc_project::Common::InvalidArgument;
- std::string result {};
+ std::string result{};
try
{
@@ -61,7 +63,7 @@
uint16_t Server::port(uint16_t value)
{
- uint16_t result {};
+ uint16_t result{};
try
{
@@ -87,10 +89,8 @@
return result;
}
-void Server::writeConfig(
- const std::string& serverAddress,
- uint16_t serverPort,
- const char* filePath)
+void Server::writeConfig(const std::string& serverAddress, uint16_t serverPort,
+ const char* filePath)
{
std::fstream stream(filePath, std::fstream::out);
@@ -119,8 +119,7 @@
auto result = getaddrinfo(address.c_str(), nullptr, &hints, &res);
if (result)
{
- log<level::ERR>("bad address",
- entry("ADDRESS=%s", address.c_str()),
+ log<level::ERR>("bad address", entry("ADDRESS=%s", address.c_str()),
entry("ERRNO=%d", result));
return false;
}
diff --git a/phosphor-rsyslog-config/server-conf.hpp b/phosphor-rsyslog-config/server-conf.hpp
index 8660416..8e1b2cb 100644
--- a/phosphor-rsyslog-config/server-conf.hpp
+++ b/phosphor-rsyslog-config/server-conf.hpp
@@ -1,10 +1,11 @@
#pragma once
+#include "xyz/openbmc_project/Network/Client/server.hpp"
+
+#include <phosphor-logging/log.hpp>
#include <sdbusplus/bus.hpp>
#include <sdbusplus/server/object.hpp>
#include <string>
-#include <phosphor-logging/log.hpp>
-#include "xyz/openbmc_project/Network/Client/server.hpp"
namespace phosphor
{
@@ -24,96 +25,93 @@
*/
class Server : public Iface
{
- public:
- Server() = delete;
- Server(const Server&) = delete;
- Server& operator=(const Server&) = delete;
- Server(Server&&) = delete;
- Server& operator=(Server&&) = delete;
- virtual ~Server() = default;
+ public:
+ Server() = delete;
+ Server(const Server&) = delete;
+ Server& operator=(const Server&) = delete;
+ Server(Server&&) = delete;
+ Server& operator=(Server&&) = delete;
+ virtual ~Server() = default;
- /** @brief Constructor to put object onto bus at a dbus path.
- * @param[in] bus - Bus to attach to.
- * @param[in] path - Path to attach at.
- * @param[in] filePath - rsyslog remote logging config file
- */
- Server(sdbusplus::bus::bus& bus,
- const std::string& path,
- const char* filePath) :
- Iface(bus, path.c_str(), true),
- configFilePath(filePath),
- hostnameChange(
- bus,
- sdbusRule::propertiesChanged(
- "/org/freedesktop/hostname1", "org.freedesktop.hostname1"),
- std::bind(std::mem_fn(&Server::hostnameChanged),
- this, std::placeholders::_1))
+ /** @brief Constructor to put object onto bus at a dbus path.
+ * @param[in] bus - Bus to attach to.
+ * @param[in] path - Path to attach at.
+ * @param[in] filePath - rsyslog remote logging config file
+ */
+ Server(sdbusplus::bus::bus& bus, const std::string& path,
+ const char* filePath) :
+ Iface(bus, path.c_str(), true),
+ configFilePath(filePath),
+ hostnameChange(
+ bus,
+ sdbusRule::propertiesChanged("/org/freedesktop/hostname1",
+ "org.freedesktop.hostname1"),
+ std::bind(std::mem_fn(&Server::hostnameChanged), this,
+ std::placeholders::_1))
+ {
+ try
{
- try
- {
- restore(configFilePath.c_str());
- }
- catch(const std::exception& e)
- {
- log<level::ERR>(e.what());
- }
-
- emit_object_added();
+ restore(configFilePath.c_str());
+ }
+ catch (const std::exception& e)
+ {
+ log<level::ERR>(e.what());
}
- using NetworkClient::address;
- using NetworkClient::port;
+ emit_object_added();
+ }
- /** @brief Override that updates rsyslog config file as well
- * @param[in] value - remote server address
- * @returns value of changed address
- */
- virtual std::string address(std::string value) override;
+ using NetworkClient::address;
+ using NetworkClient::port;
- /** @brief Override that updates rsyslog config file as well
- * @param[in] value - remote server port
- * @returns value of changed port
- */
- virtual uint16_t port(uint16_t value) override;
+ /** @brief Override that updates rsyslog config file as well
+ * @param[in] value - remote server address
+ * @returns value of changed address
+ */
+ virtual std::string address(std::string value) override;
- /** @brief Restart rsyslog's systemd unit
- */
- virtual void restart();
+ /** @brief Override that updates rsyslog config file as well
+ * @param[in] value - remote server port
+ * @returns value of changed port
+ */
+ virtual uint16_t port(uint16_t value) override;
- private:
- /** @brief Update remote server address and port in
- * rsyslog config file.
- * @param[in] serverAddress - remote server address
- * @param[in] serverPort - remote server port
- * @param[in] filePath - rsyslog config file path
- */
- void writeConfig(
- const std::string& serverAddress,
- uint16_t serverPort,
- const char* filePath);
+ /** @brief Restart rsyslog's systemd unit
+ */
+ virtual void restart();
- /** @brief Checks if input IP address is valid (uses getaddrinfo)
- * @param[in] address - server address
- * @returns true if valid, false otherwise
- */
- bool addressValid(const std::string& address);
+ private:
+ /** @brief Update remote server address and port in
+ * rsyslog config file.
+ * @param[in] serverAddress - remote server address
+ * @param[in] serverPort - remote server port
+ * @param[in] filePath - rsyslog config file path
+ */
+ void writeConfig(const std::string& serverAddress, uint16_t serverPort,
+ const char* filePath);
- /** @brief Populate existing config into D-Bus properties
- * @param[in] filePath - rsyslog config file path
- */
- void restore(const char* filePath);
+ /** @brief Checks if input IP address is valid (uses getaddrinfo)
+ * @param[in] address - server address
+ * @returns true if valid, false otherwise
+ */
+ bool addressValid(const std::string& address);
- std::string configFilePath{};
+ /** @brief Populate existing config into D-Bus properties
+ * @param[in] filePath - rsyslog config file path
+ */
+ void restore(const char* filePath);
- /** @brief React to hostname change
- * @param[in] msg - sdbusplus message
- */
- void hostnameChanged(sdbusplus::message::message& msg)
- {
- restart();
- }
+ std::string configFilePath{};
- sdbusplus::bus::match_t hostnameChange;
+ /** @brief React to hostname change
+ * @param[in] msg - sdbusplus message
+ */
+ void hostnameChanged(sdbusplus::message::message& msg)
+ {
+ restart();
+ }
+
+ sdbusplus::bus::match_t hostnameChange;
};
} // namespace rsyslog_config
diff --git a/phosphor-rsyslog-config/utils.hpp b/phosphor-rsyslog-config/utils.hpp
index ea7984d..017a439 100644
--- a/phosphor-rsyslog-config/utils.hpp
+++ b/phosphor-rsyslog-config/utils.hpp
@@ -1,6 +1,7 @@
#pragma once
#include "config.h"
+
#include <sdbusplus/bus.hpp>
namespace phosphor
@@ -13,11 +14,8 @@
void restart()
{
auto bus = sdbusplus::bus::new_default();
- auto method = bus.new_method_call(
- SYSTEMD_BUSNAME,
- SYSTEMD_PATH,
- SYSTEMD_INTERFACE,
- "RestartUnit");
+ auto method = bus.new_method_call(SYSTEMD_BUSNAME, SYSTEMD_PATH,
+ SYSTEMD_INTERFACE, "RestartUnit");
method.append("rsyslog.service", "replace");
bus.call_noreply(method);
}
diff --git a/test/elog_errorwrap_test.cpp b/test/elog_errorwrap_test.cpp
index f341447..c94c638 100644
--- a/test/elog_errorwrap_test.cpp
+++ b/test/elog_errorwrap_test.cpp
@@ -16,7 +16,7 @@
// Max num of Info( and below Sev) errors can be created is qual to
// ERROR_INFO_CAP
- EXPECT_EQ(ERROR_INFO_CAP , manager.getInfoErrSize());
+ EXPECT_EQ(ERROR_INFO_CAP, manager.getInfoErrSize());
for (auto i = 0; i < ERROR_CAP + 20; i++)
{
@@ -24,9 +24,8 @@
}
// Max num of high severity errors can be created is qual to ERROR_CAP
EXPECT_EQ(ERROR_CAP, manager.getRealErrSize());
-
}
-}// namespace internal
-}// namespace logging
-}// namespace phosphor
+} // namespace internal
+} // namespace logging
+} // namespace phosphor
diff --git a/test/elog_errorwrap_test.hpp b/test/elog_errorwrap_test.hpp
index 7c673b6..537a7ff 100644
--- a/test/elog_errorwrap_test.hpp
+++ b/test/elog_errorwrap_test.hpp
@@ -1,14 +1,17 @@
+#include "config.h"
+
+#include "elog_serialize.hpp"
#include "log_manager.hpp"
#include "xyz/openbmc_project/Common/error.hpp"
-#include <gtest/gtest.h>
-#include <gmock/gmock.h>
-#include <sdbusplus/bus.hpp>
+
#include <experimental/filesystem>
+#include <phosphor-logging/elog-errors.hpp>
#include <phosphor-logging/elog.hpp>
#include <phosphor-logging/log.hpp>
-#include <phosphor-logging/elog-errors.hpp>
-#include "elog_serialize.hpp"
-#include "config.h"
+#include <sdbusplus/bus.hpp>
+
+#include <gmock/gmock.h>
+#include <gtest/gtest.h>
namespace phosphor
{
@@ -21,30 +24,24 @@
class journalInterface
{
- public:
- virtual void journalSync() = 0;
- virtual int sd_journal_open(sd_journal** j, int k) = 0;
- virtual int sd_journal_get_data(sd_journal* j,
- const char* transactionIdVar,
- const void** data,
- size_t length) = 0;
- virtual void sd_journal_close(sd_journal* j) = 0;
+ public:
+ virtual void journalSync() = 0;
+ virtual int sd_journal_open(sd_journal** j, int k) = 0;
+ virtual int sd_journal_get_data(sd_journal* j, const char* transactionIdVar,
+ const void** data, size_t length) = 0;
+ virtual void sd_journal_close(sd_journal* j) = 0;
};
class journalImpl : public journalInterface
{
- public:
- void journalSync();
- int sd_journal_open(sd_journal** j, int k);
- int sd_journal_get_data(sd_journal* j,
- const char* transactionIdVar,
- const void** data,
- size_t length);
- void sd_journal_close(sd_journal* j);
+ public:
+ void journalSync();
+ int sd_journal_open(sd_journal** j, int k);
+ int sd_journal_get_data(sd_journal* j, const char* transactionIdVar,
+ const void** data, size_t length);
+ void sd_journal_close(sd_journal* j);
};
-
-
int journalImpl::sd_journal_open(sd_journal** j, int k)
{
return 1;
@@ -57,8 +54,7 @@
int journalImpl::sd_journal_get_data(sd_journal* j,
const char* transactionIdVar,
- const void** data,
- size_t length)
+ const void** data, size_t length)
{
return 1;
}
@@ -68,42 +64,36 @@
return;
}
-
-class MockJournal : public Manager
+class MockJournal : public Manager
{
- public:
- MockJournal(sdbusplus::bus::bus& bus,
- const char* objPath): Manager(bus, objPath) {};
- MOCK_METHOD0(journalSync, void());
- MOCK_METHOD2(sd_journal_open, int(sd_journal**, int));
- MOCK_METHOD4(sd_journal_get_data, int(sd_journal*,
- const char*,
- const void**, size_t));
- MOCK_METHOD1(sd_journal_close, void(sd_journal*));
+ public:
+ MockJournal(sdbusplus::bus::bus& bus, const char* objPath) :
+ Manager(bus, objPath){};
+ MOCK_METHOD0(journalSync, void());
+ MOCK_METHOD2(sd_journal_open, int(sd_journal**, int));
+ MOCK_METHOD4(sd_journal_get_data,
+ int(sd_journal*, const char*, const void**, size_t));
+ MOCK_METHOD1(sd_journal_close, void(sd_journal*));
};
-
-
class TestLogManager : public testing::Test
{
- public:
- sdbusplus::bus::bus bus;
- MockJournal manager;
- TestLogManager()
- : bus(sdbusplus::bus::new_default()),
- manager(bus, "/xyz/openbmc_test/abc")
- {
- fs::create_directories(ERRLOG_PERSIST_PATH);
- }
+ public:
+ sdbusplus::bus::bus bus;
+ MockJournal manager;
+ TestLogManager() :
+ bus(sdbusplus::bus::new_default()),
+ manager(bus, "/xyz/openbmc_test/abc")
+ {
+ fs::create_directories(ERRLOG_PERSIST_PATH);
+ }
- ~TestLogManager()
- {
- fs::remove_all(ERRLOG_PERSIST_PATH);
- }
-
+ ~TestLogManager()
+ {
+ fs::remove_all(ERRLOG_PERSIST_PATH);
+ }
};
-
-}// nmaespace internal
-}// namespace logging
-}// namespace phosphor
+} // namespace internal
+} // namespace logging
+} // namespace phosphor
diff --git a/test/remote_logging_test_address.cpp b/test/remote_logging_test_address.cpp
index 1291a2e..5c8794e 100644
--- a/test/remote_logging_test_address.cpp
+++ b/test/remote_logging_test_address.cpp
@@ -21,6 +21,6 @@
EXPECT_THROW(config->address("not_an_IP"), InvalidArgument);
}
-}// namespace test
-}// namespace logging
-}// namespace phosphor
+} // namespace test
+} // namespace logging
+} // namespace phosphor
diff --git a/test/remote_logging_test_config.cpp b/test/remote_logging_test_config.cpp
index 8c3a98a..59feedd 100644
--- a/test/remote_logging_test_config.cpp
+++ b/test/remote_logging_test_config.cpp
@@ -1,4 +1,5 @@
#include "remote_logging_tests.hpp"
+
#include <fstream>
namespace phosphor
@@ -53,6 +54,6 @@
EXPECT_EQ(getConfig(configFilePath.c_str()), "#*.* @@remote-host:port");
}
-}// namespace test
-}// namespace logging
-}// namespace phosphor
+} // namespace test
+} // namespace logging
+} // namespace phosphor
diff --git a/test/remote_logging_test_port.cpp b/test/remote_logging_test_port.cpp
index a9d3bc3..cf1a42e 100644
--- a/test/remote_logging_test_port.cpp
+++ b/test/remote_logging_test_port.cpp
@@ -13,6 +13,6 @@
EXPECT_EQ(config->port(), 100);
}
-}// namespace test
-}// namespace logging
-}// namespace phosphor
+} // namespace test
+} // namespace logging
+} // namespace phosphor
diff --git a/test/remote_logging_tests.hpp b/test/remote_logging_tests.hpp
index 3606f5a..936e3bb 100644
--- a/test/remote_logging_tests.hpp
+++ b/test/remote_logging_tests.hpp
@@ -1,9 +1,12 @@
-#include <gtest/gtest.h>
+#include "config.h"
+
+#include "phosphor-rsyslog-config/server-conf.hpp"
+
#include <experimental/filesystem>
#include <sdbusplus/bus.hpp>
-#include "config.h"
-#include "phosphor-rsyslog-config/server-conf.hpp"
+
#include "gmock/gmock.h"
+#include <gtest/gtest.h>
namespace phosphor
{
@@ -20,45 +23,40 @@
class MockServer : public phosphor::rsyslog_config::Server
{
- public:
- MockServer(sdbusplus::bus::bus& bus,
- const std::string& path,
+ public:
+ MockServer(sdbusplus::bus::bus& bus, const std::string& path,
const char* filePath) :
- phosphor::rsyslog_config::Server(bus, path, filePath)
- {
- }
+ phosphor::rsyslog_config::Server(bus, path, filePath)
+ {
+ }
- MOCK_METHOD0(restart, void());
+ MOCK_METHOD0(restart, void());
};
class TestRemoteLogging : public testing::Test
{
- public:
- TestRemoteLogging()
- {
- configFilePath = std::string(dir.c_str()) + "/server.conf";
- config =
- new MockServer(bus,
- BUSPATH_REMOTE_LOGGING_CONFIG,
- configFilePath.c_str());
- }
+ public:
+ TestRemoteLogging()
+ {
+ configFilePath = std::string(dir.c_str()) + "/server.conf";
+ config = new MockServer(bus, BUSPATH_REMOTE_LOGGING_CONFIG,
+ configFilePath.c_str());
+ }
- ~TestRemoteLogging()
- {
- delete config;
- }
+ ~TestRemoteLogging()
+ {
+ delete config;
+ }
- static void TearDownTestCase()
- {
- fs::remove_all(dir);
- }
+ static void TearDownTestCase()
+ {
+ fs::remove_all(dir);
+ }
- MockServer* config;
- std::string configFilePath;
+ MockServer* config;
+ std::string configFilePath;
};
} // namespace test
} // namespace logging
} // namespace phosphor
-
-
diff --git a/test/serialization_test_path.cpp b/test/serialization_test_path.cpp
index 50e7d08..379b124 100644
--- a/test/serialization_test_path.cpp
+++ b/test/serialization_test_path.cpp
@@ -1,6 +1,6 @@
-#include "serialization_tests.hpp"
#include "elog_entry.hpp"
#include "elog_serialize.hpp"
+#include "serialization_tests.hpp"
namespace phosphor
{
@@ -13,10 +13,7 @@
{
auto id = 99;
auto e = std::make_unique<Entry>(
- bus,
- std::string(OBJ_ENTRY) + '/' + std::to_string(id),
- id,
- manager);
+ bus, std::string(OBJ_ENTRY) + '/' + std::to_string(id), id, manager);
auto path = serialize(*e, TestSerialization::dir);
EXPECT_EQ(path.c_str(), TestSerialization::dir / std::to_string(id));
}
@@ -24,5 +21,3 @@
} // namespace test
} // namespace logging
} // namespace phosphor
-
-
diff --git a/test/serialization_test_properties.cpp b/test/serialization_test_properties.cpp
index be9a7c7..8c92024 100644
--- a/test/serialization_test_properties.cpp
+++ b/test/serialization_test_properties.cpp
@@ -1,6 +1,6 @@
-#include "serialization_tests.hpp"
#include "elog_entry.hpp"
#include "elog_serialize.hpp"
+#include "serialization_tests.hpp"
namespace phosphor
{
@@ -18,25 +18,15 @@
std::string message{"test error"};
std::string fwLevel{"level42"};
auto input = std::make_unique<Entry>(
- bus,
- std::string(OBJ_ENTRY) + '/' + std::to_string(id),
- id,
- timestamp,
- Entry::Level::Informational,
- std::move(message),
- std::move(testData),
- std::move(assocations),
- fwLevel,
- manager);
+ bus, std::string(OBJ_ENTRY) + '/' + std::to_string(id), id, timestamp,
+ Entry::Level::Informational, std::move(message), std::move(testData),
+ std::move(assocations), fwLevel, manager);
auto path = serialize(*input, TestSerialization::dir);
auto idStr = path.filename().c_str();
id = std::stol(idStr);
auto output = std::make_unique<Entry>(
- bus,
- std::string(OBJ_ENTRY) + '/' + idStr,
- id,
- manager);
+ bus, std::string(OBJ_ENTRY) + '/' + idStr, id, manager);
deserialize(path, *output);
EXPECT_EQ(input->id(), output->id());
@@ -53,5 +43,3 @@
} // namespace test
} // namespace logging
} // namespace phosphor
-
-
diff --git a/test/serialization_tests.hpp b/test/serialization_tests.hpp
index a8054d3..6e41379 100644
--- a/test/serialization_tests.hpp
+++ b/test/serialization_tests.hpp
@@ -1,10 +1,14 @@
-#include <gtest/gtest.h>
-#include <experimental/filesystem>
-#include <stdlib.h>
-#include <sdbusplus/bus.hpp>
-#include "log_manager.hpp"
#include "config.h"
+#include "log_manager.hpp"
+
+#include <stdlib.h>
+
+#include <experimental/filesystem>
+#include <sdbusplus/bus.hpp>
+
+#include <gtest/gtest.h>
+
namespace phosphor
{
namespace logging
@@ -20,22 +24,19 @@
class TestSerialization : public testing::Test
{
- public:
- TestSerialization() :
- dir(fs::path(mkdtemp(tmplt)))
- {
- }
+ public:
+ TestSerialization() : dir(fs::path(mkdtemp(tmplt)))
+ {
+ }
- ~TestSerialization()
- {
- fs::remove_all(dir);
- }
+ ~TestSerialization()
+ {
+ fs::remove_all(dir);
+ }
- fs::path dir;
+ fs::path dir;
};
} // namespace test
} // namespace logging
} // namespace phosphor
-
-