C++ [tfs 1.2] canWalkthrough NOPVPZONE

_M4G0_ · Jan 20, 2018

I did this modification

C++:

const Tile* playerTile = player->getTile();
    if (!playerTile || !playerTile->hasFlag(TILESTATE_PROTECTIONZONE) && !playerTile->hasFlag(TILESTATE_NOPVPZONE)) {
        return false;
    }

on windows compile normally and work, on linux this error

Code:

In member function ‘bool Player::canWalkthrough(const Creature*) const’:
/home/servidor/tfs/src/player.cpp:713:68: error: suggest parentheses around ‘&&’ within ‘||’ [-Werror=parentheses]
  if (!playerTile || !playerTile->hasFlag(TILESTATE_PROTECTIONZONE) && !playerTile->hasFlag(TILESTATE_NOPVPZONE)) {

if i insert

C++:

playerTile->hasFlag(TILESTATE_PROTECTIONZONE) || !playerTile->hasFlag(TILESTATE_NOPVPZONE)) {

compile on linux but dont work

TFS 1.2 10.99

Nekiro · Jan 20, 2018

Google what parentheses mean.

if (something || (something2 && something3))

_M4G0_ · Jan 20, 2018

Nekiro said:
Google what parentheses mean.

if (something || (something2 && something3))

thanks... but i tried

C++:

((!playerTile || (!playerTile->hasFlag(TILESTATE_PROTECTIONZONE)) || (!playerTile->hasFlag(TILESTATE_NOPVPZONE))))

Nekiro · Jan 20, 2018

_M4G0_ said:
thanks... but i tried

C++:

((!playerTile || (!playerTile->hasFlag(TILESTATE_PROTECTIONZONE)) || (!playerTile->hasFlag(TILESTATE_NOPVPZONE))))

It's not the same.

It should be:

Code:

if (!playerTile || (!playerTile->hasFlag(TILESTATE_PROTECTIONZONE) && !playerTile->hasFlag(TILESTATE_NOPVPZONE))) {

_M4G0_ · Jan 20, 2018

Nekiro said:
It's not the same.

It should be:

Code:

if (!playerTile || (!playerTile->hasFlag(TILESTATE_PROTECTIONZONE) && !playerTile->hasFlag(TILESTATE_NOPVPZONE))) {

this was before your help.

C++ [tfs 1.2] canWalkthrough NOPVPZONE

_M4G0_

Well-Known Member

Nekiro

Nekiro

Legendary OT User

_M4G0_

Well-Known Member

Nekiro

Legendary OT User

_M4G0_

Well-Known Member

Similar threads