Build fail on Linux

[Talmeron]

Description: Build fail on linux using both ccc and clang

Current behaviour: [ 16%] Building CXX object src/plugins/CMakeFiles/plugins.dir/playerbot/PlayerbotFactory.cpp.o In file included from /home/exaurdon/SingleCore_TC/src/plugins/playerbot/PlayerbotFactory.cpp:3: In file included from /home/exaurdon/SingleCore_TC/src/plugins/playerbot/PlayerbotFactory.h:3: In file included from /home/exaurdon/SingleCore_TC/src/plugins/playerbot/strategy/actions/InventoryAction.h:4: /home/exaurdon/SingleCore_TC/src/plugins/playerbot/strategy/actions/../ItemVisitors.h:17:59: warning: implicit conversion of NULL constant to 'size_type' (aka 'unsigned long') [-Wnull-conversion] FindItemVisitor() : IterateItemsVisitor(), result(NULL) {}

                                                          0
/home/exaurdon/SingleCore_TC/src/plugins/playerbot/PlayerbotFactory.cpp:1317:25: warning: 
      address of array 'tSpell->learnedSpell' will always evaluate to 'true'
      [-Wpointer-bool-conversion]
            if (tSpell->learnedSpell)
            ~~  ~~~~~~~~^~~~~~~~~~~~
/home/exaurdon/SingleCore_TC/src/plugins/playerbot/PlayerbotFactory.cpp:1336:2: error: 
      expected unqualified-id
        if (bot->getLevel() > 20 && bot->GetTeamId()==TeamId::TEAM_ALLIANCE)
        ^
/home/exaurdon/SingleCore_TC/src/plugins/playerbot/PlayerbotFactory.cpp:1340:2: error: 
      expected unqualified-id
        if (bot->getLevel() > 20 && bot->GetTeamId() == TeamId::TEAM_HORDE)
        ^
/home/exaurdon/SingleCore_TC/src/plugins/playerbot/PlayerbotFactory.cpp:1345:2: error: 
      expected unqualified-id
        if (bot->getLevel() > 40 && bot->GetTeamId() == TeamId::TEAM_ALLIANCE)
        ^
/home/exaurdon/SingleCore_TC/src/plugins/playerbot/PlayerbotFactory.cpp:1350:2: error: 
      expected unqualified-id
        if (bot->getLevel() > 40 && bot->GetTeamId() == TeamId::TEAM_HORDE)
        ^
/home/exaurdon/SingleCore_TC/src/plugins/playerbot/PlayerbotFactory.cpp:1354:1: error: 
      extraneous closing brace ('}')
}
^
/home/exaurdon/SingleCore_TC/src/plugins/playerbot/PlayerbotFactory.cpp:1815:46: warning: 
      '&&' within '||' [-Wlogical-op-parentheses]
  ...(proto->Class != ITEM_CLASS_ARMOR && proto->Class != ITEM_CLASS_WEAPON || ...
      ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ ~~
/home/exaurdon/SingleCore_TC/src/plugins/playerbot/PlayerbotFactory.cpp:1815:46: note: 
      place parentheses around the '&&' expression to silence this warning
  ...(proto->Class != ITEM_CLASS_ARMOR && proto->Class != ITEM_CLASS_WEAPON || ...
                                       ^
      (                                                                    )
3 warnings and 5 errors generated.
src/plugins/CMakeFiles/plugins.dir/build.make:518: recipe for target 'src/plugins/CMakeFiles/plugins.dir/playerbot/PlayerbotFactory.cpp.o' failed
make[2]: *** [src/plugins/CMakeFiles/plugins.dir/playerbot/PlayerbotFactory.cpp.o] Error 1
CMakeFiles/Makefile2:1938: recipe for target 'src/plugins/CMakeFiles/plugins.dir/all' failed
make[1]: *** [src/plugins/CMakeFiles/plugins.dir/all] Error 2
Makefile:127: recipe for target 'all' failed
make: *** [all] Error 2

**Expected behaviour**: Not do that

**Steps to reproduce the problem**:
Make

**Branch(es)**: 335/6x  

**TC hash/commit**:  

**TDB version**:  

**Operating system**:  Linux Peppermint

[//]: # (This template is for problem reports, for other type of reports edit it accordingly)
[//]: # (If this is a crash report, include the crashlog with https://gist.github.com/)

[conan513]

what is the branch?

[Talmeron]

Lastest pull as of Yesterday.

[Talmeron]

npcbots, sorry.

[conan513]

last two commit is unfinished, try it now