diff options
| author | Thorbjørn Lindeijer <thorbjorn@lindeijer.nl> | 2011-06-02 00:32:53 +0200 |
|---|---|---|
| committer | Thorbjørn Lindeijer <thorbjorn@lindeijer.nl> | 2011-06-02 16:41:47 +0200 |
| commit | 4e27937924766948d7ff9200f04a37fe4d59018c (patch) | |
| tree | c1f5c255192c85a82c77c2f8f916984e2b863c44 /src/net/tmwa/beinghandler.cpp | |
| parent | c2eab288ecc7d7c5e26d02ccecf285cbc0c218ed (diff) | |
| download | mana-client-4e27937924766948d7ff9200f04a37fe4d59018c.tar.gz mana-client-4e27937924766948d7ff9200f04a37fe4d59018c.tar.bz2 mana-client-4e27937924766948d7ff9200f04a37fe4d59018c.tar.xz mana-client-4e27937924766948d7ff9200f04a37fe4d59018c.zip | |
Arbitrary code cleanups
Just some stuff that piles up while "looking" at the code, which
eventually gets annoying to ignore while staging real changes.
* Replaced a few NULL occurrences with 0
* Rely on default parameter for std::vector::resize.
* Replaced a few "" with std::string()
* Prefer .empty() to == ""
* Removed a few comparisons with NULL
* Don't check pointers before deleting them
* Removed a bunch of redundant semicolons
* Made some global variables static (local to their compilation unit)
* Prefer prefix ++/-- operators to postfix versions when possible
* Corrected location of a comment
Diffstat (limited to 'src/net/tmwa/beinghandler.cpp')
| -rw-r--r-- | src/net/tmwa/beinghandler.cpp | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/src/net/tmwa/beinghandler.cpp b/src/net/tmwa/beinghandler.cpp index b7cf7bd9..7adb6ed7 100644 --- a/src/net/tmwa/beinghandler.cpp +++ b/src/net/tmwa/beinghandler.cpp @@ -78,7 +78,7 @@ BeingHandler::BeingHandler(bool enableSync): handledMessages = _messages; } -Being *createBeing(int id, short job) +static Being *createBeing(int id, short job) { ActorSprite::Type type = ActorSprite::UNKNOWN; if (job <= 25 || (job >= 4001 && job <= 4049)) |