Skip to content

Conversation

@Helias
Copy link
Member

@Helias Helias commented Aug 29, 2024

No description provided.

@Helias
Copy link
Member Author

Helias commented Aug 29, 2024

build is failing due to some cppcheck:

modules/mod-chat-transmitter/src/Requests/RequestQueryResult.h:30:9: error: There is an unknown macro here somewhere. Configuration is required. If NLOHMANN_JSON_EXPAND is a macro then please configure it. [unknownMacro]
        NLOHMANN_DEFINE_TYPE_INTRUSIVE(QueryResult, queryId, data, columns, success, error, affectedRows)
        ^
modules/mod-chat-transmitter/libs/nlohmann/json.hpp:23135:5: error: syntax error [syntaxError]
    JSON_HEDLEY_WARN_UNUSED_RESULT
    ^
modules/mod-chat-transmitter/libs/nlohmann/json.hpp:5466:5: error: syntax error [syntaxError]
    JSON_HEDLEY_RETURNS_NON_NULL
    ^
modules/mod-chat-transmitter/src/PlayerInfo.h:50:9: error: There is an unknown macro here somewhere. Configuration is required. If NLOHMANN_JSON_EXPAND is a macro then please configure it. [unknownMacro]
        NLOHMANN_DEFINE_TYPE_INTRUSIVE(PlayerInfo, name, guid, level, raceId, classId, gender, accountName, accountGuid, lastIpAddr)
        ^
modules/mod-chat-transmitter/src/Requests/RequestCommandResult.h:22:9: error: There is an unknown macro here somewhere. Configuration is required. If NLOHMANN_JSON_EXPAND is a macro then please configure it. [unknownMacro]
        NLOHMANN_DEFINE_TYPE_INTRUSIVE(CommandResult, commandId, output, success)
        ^
modules/mod-chat-transmitter/src/Requests/RequestElunaError.h:21:9: error: There is an unknown macro here somewhere. Configuration is required. If NLOHMANN_JSON_EXPAND is a macro then please configure it. [unknownMacro]
        NLOHMANN_DEFINE_TYPE_INTRUSIVE(ElunaError, guildId, trace)
        ^

@pangolp
Copy link
Contributor

pangolp commented Aug 30, 2024

I really don't know how to fix this, I would be interested in seeing the result, to see if it can be replicated in other modules, and solve these problems in the future, but at this moment, I don't know how to fix it.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants