ouch/inspircd
2
0
Fork 0
mirror of https://github.com/inspircd/inspircd.git synced 2025-03-10 19:19:02 -04:00
Code Issues Packages Projects Releases Wiki Activity

Merge pull request #1147 from SaberUK/insp20+gcc6

Browse Source 
[2.0] Fix GCC 6 warning about null checking this.
This commit is contained in:
Attila Molnar 2016-02-29 12:40:21 +01:00
commit ccd4c11ea1
1 changed files with 11 additions and 5 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

16
src/configparser.cpp
View File

@ -388,19 +388,20 @@ bool ParseStack::ParseExec(const std::string& name, int flags, const std::string
return ok;
}
bool ConfigTag::readString(const std::string& key, std::string& value, bool allow_lf)
{
#ifdef __clang__
# pragma clang diagnostic push
# pragma clang diagnostic ignored "-Wunknown-pragmas"
# pragma clang diagnostic ignored "-Wundefined-bool-conversion"
#elif defined __GNUC__
# pragma GCC diagnostic push
# pragma GCC diagnostic ignored "-Wpragmas"
# pragma GCC diagnostic ignored "-Wnonnull-compare"
#endif
bool ConfigTag::readString(const std::string& key, std::string& value, bool allow_lf)
{
// TODO: this is undefined behaviour but changing the API is too risky for 2.0.
if (!this)
return false;
#ifdef __clang__
# pragma clang diagnostic pop
#endif
for(std::vector<KeyVal>::iterator j = items.begin(); j != items.end(); ++j)
{
if(j->first != key)
@ -418,6 +419,11 @@ bool ConfigTag::readString(const std::string& key, std::string& value, bool allo
}
return false;
}
#ifdef __clang__
# pragma clang diagnostic pop
#elif defined __GNUC__
# pragma GCC diagnostic pop
#endif
std::string ConfigTag::getString(const std::string& key, const std::string& def)
{