Age | Commit message (Collapse) | Author | Files | Lines |
|
only about 160 out of 1500 files that include localconsts.h need
to include performance.h
Saves 12% from compile times (with profiler ON) and 0.1% with profiler OFF.
****
mana/plus!189
|
|
.. instead of manaplus's header.
& update copyrights to 2025.
Open problem: should it be "The ManaVerse Client", "the ManaVerse client" or something in between in "This file is part of ..." line? All caps is current situation.
Squashed with:
* update copyright dates for 2025
* Revert "Switch to branch of mplint that checks for manaverse copyright headers"
* Change back to uppercase The
Why? This line was changed 3 times in history and all had an uppercase
The.
PS: ManaVerse has no endorsment from the parent project, ManaPlus.
* Change to "This file is part of the ManaVerse Client"
poppet says 'the' instead of 'The'
find . \( -name "*.h" -o -name "*.cpp" -o -name "*.cc" -o -name "*.inc"
\) -execdir sed -i -e 's/This file is part of The ManaPlus Client/This
file is part of the ManaVerse Client/' {} \+
* Change 'The ManaPlus Client' to 'The ManaVerse Client' in headers
find . \( -name "*.h" -o -name "*.cpp" -o -name "*.cc" -o -name "*.inc"
\) -execdir sed -i -e '2s/ManaPlus/ManaVerse/' {} \+
* Switch to branch of mplint that checks for manaverse copyright headers
.. instead of manaplus's
****
mana/plus!179
|
|
introduced in 8ad99cc3 / "Add profiler lines into some functions."
****
mana/plus!181
|
|
Rationale: By the standard, cstdint guarantees these basic types
in std::, but *may* put them into the global namespace.
By including stdint.h directly, we guarantee that these types are
in the global namespace. stdint.h is
also a C++ standard header guaranteed as part of the C compatibility
support, thus toolchains should support this.
The proper(tm) way of going about this would mean either prefixing
[u]int[0-9]*_t with std:: everywhere where they are used (a chore)
or having a proxy header that includes cstdint and around 50
using statements. The latter sounds to me like a good approach for
toolchains that do not provide stdint.h and do not pollute global
namespace. If such a toolchain exists in the wild.
See discussion at
https://stackoverflow.com/questions/13642827/cstdint-vs-stdint-h
In practice, my toolchain's (gcc 14 on GNU) cstdint includes stdint.h
then drags those types into std:: as well.
****
mana/plus!170
|
|
2024-08/01/#Debug.log:[14:25:43] Wrong actual or planned inbound packet
size!. Packet id: 205(0xcd), Planned size: 3, Actual size: 6
2 bytes packet id + 1 byte flag = 3
2 bytes packet id + 4 byte flag (wrong) = 6
Squashed with:
* Separate processKickAck handler for eathena
* Move ea's adminrecv and its sole handler processKickAck into TMWA
namespace
* Revert "Fix wrong read of clif_GM_kickack packet"
This would break TMWA packet processing, as @thorbjorn pointed out.
This reverts commit efe43bac4de90b3e427d985b77cb3dd3206bcf9d.
****
mana/plus!148
|
|
Not sure how to trigger this one, couldn't do it with a few simple
tests.
****
mana/plus!149
|
|
2025-03/22/#Debug.log-[19:38:37] Wrong actual or planned inbound packet size!. Packet id: 433(0x1b1), Planned size: 7, Actual size: 4
2025-03/22/#Debug.log:[19:38:52] You picked up 1 [@@760|Overlord's Helmet@@].
****
mana/plus!146
|
|
Pointless change, I know.
****
mana/plus!115
|
|
****
mana/plus!114
|
|
& shaves ~5KB from debug binary.
previd=""; while read -r line; do if [[ $line =~ .*"msg.read"[^\"]*"\""([^\"]*).* ]]; then id="${BASH_REMATCH[1]}"; if [[ "$id" == "$previd" ]]; then printf "%s\n" "$line"; fi; previd="$id"; fi; done < <(grep -r 'msg.read.*' src/net/)
****
mana/plus!91
|
|
This has the side effect that now #Debug history is also loaded.
****
mana/plus!84
|
|
|
|
|
|
See #73 for discussion
Squashed with:
* Fix mana pearl showing in arrows slot in Classic
* Fix off-by-one. Was causing Old Towel to render in torso slot in ML.
****
mana/plus!78
|
|
See merge request mana/plus!64
|
|
No server in use still sends that field.
****
Approved-by: Jesusalva Jesusalva <jesusalva@themanaworld.org>
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
Handle player stop packet always.
Check desync always if tile changed client side.
Check desync always if server move response packet received.
Removed unused configuration option.
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|