Age | Commit message (Collapse) | Author | Files | Lines |
|
Further idea: a lot of these models are wrappers around a string options array. Maybe convert the classes into a single template class taking an array parameter?
****
mana/plus!111
Approved-by: Led Mitz <smoothshifter@tuta.io>
|
|
description/tooltip.
****
mana/plus!105
|
|
****
mana/plus!104
|
|
Memory may not be freed properly, also code is IMO ugly. See !109 for the other way of implementing this (unrelated). Was done in socialWindow as the join confirmation is already there.
****
mana/plus!110
|
|
Curiously, this increases -g -O2 exe size by 96 bytes, but disassembly
dump around this code is the same (redundant check was already optimised out?)
****
mana/plus!106
|
|
use @toevent, then switch character (and stay on character screen).
Thread 1 "manaplus" received signal SIGSEGV, Segmentation fault.
0x.. in Game::getCurrentMap (this=<optimized out>) at ./game.h:108
108 { return mCurrentMap; }
(gdb) bt
0x.. in Game::getCurrentMap (this=<optimized out>) at ./game.h:108
Ea::PlayerRecv::processMapMask (msg=...) at net/ea/playerrecv.cpp:242
****
mana/plus!108
|
|
Adds a light blue bar for MP below player's HP bar. It shows only if M.Atk is above 0.
I didn't add a checkbox to the settings to disable this, let me know if I should.
Squashed with:
* Sneak in another TODO
* Sneak in a TODO
****
mana/plus!68
Co-authored-by: Fedja Beader <fedja@protonmail.ch>
|
|
Move to target & move to target type intentionally kept in basic's targetting section.
****
mana/plus!103
|
|
work for ML
Squashed with:
* Sync with last comment (Evol->Evol2)
* Another rename, to reflect TMW->Manasource rebrand (maybe?)
****
mana/plus!79
|
|
Split from mana/plus!79
|
|
Shaves some bytes from exe, too
****
mana/plus!102
|
|
TODO: are all these casts actually required? SMH
****
mana/plus!101
|
|
****
mana/plus!96
|
|
Squashed with:
* Appease linters + more reformat
* Further improve skill error handling and messages.
24751878 "Improve skill error messages." added trailing dots only on the
second branch. This adds missing dots on the first one as well as
compacts the logic.
****
mana/plus!93
|
|
& 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
|
|
* Run gcc8 job again in this MR
* Squash the %c strftime gcc warning
****
mana/plus!100
|
|
Server does not communicate sent time.. or it did, until 2017 ??
See src/map/clif.c #if PACKETVER >= 20170419
The RODEX_EXPIRY constant seems not to be communicated either,
preventing us from deducing send time from expiry time.
****
mana/plus!90
|
|
if any, are worth it.
In fact, this might even save some time. Calculating distance may be
faster than iterating through memory twice.
****
mana/plus!95
|
|
****
mana/plus!94
|
|
How to reproduce: spam emote then quickly switch char
Thread 1 "manaplus" received signal SIGSEGV, Segmentation fault.
SkillDialog::getSkill (this=0x0, id=id@entry=1) at gui/windows/skilldialog.cpp:805
0 SkillDialog::getSkill (this=0x0, id=id@entry=1) at gui/windows/skilldialog.cpp:805
1 EAthena::SkillRecv::processSkillFailed (msg=...) at net/eathena/skillrecv.cpp:302
...
****
mana/plus!92
|
|
g
****
mana/plus!77
|
|
****
mana/plus!88
|
|
|
|
|
|
raise default (5->100).
|
|
|
|
Addendum to cbab9d6d "Fix for login button"
See also mana/plus!67 and mana/plus#64
****
mana/plus!86
|
|
This has the side effect that now #Debug history is also loaded.
****
mana/plus!84
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
****
mana/plus!83
|
|
What happened:
1) herc evol sends 0x100 as packet ID offset
2) 0x100 is added to 0x0062 of SMSG_CHAR_PASSWORD_RESPONSE
3) SMSG_CHAR_PASSWORD_RESPONSE is then inserted into 0x162
slot of packet map, shadowing SMSG_GUILD_SKILL_INFO
4) on guild lvl-up, server sends this in SMSG_GUILD_SKILL_INFO
5) ManaVerse exits in confusion.
Note 1: due to popupmenu paste not working in change password dialog, tested password change only with invalid old password, which worked (opened error dialog instead of client exiting).
Note 2: Some packet handlers are bound using &Ea::LoginRecv prefix, this one had &LoginRecv and this was kept this way (uses vtable lookup?).
~~TODO: SMSG_CHAR_PASSWORD_RESPONSE seems to be something about password changes, but I could not find this packet either in TMWA or herc. Still, I need to test if these still work on all active servers~~ Tested.
Squashed with:
* Fix exit on password change, too
* Fix guild lvl-up packet parsing causing client exit in ML
****
mana/plus!81
|
|
Problem happens only on Windows that uses \ for path separators and only when primary download location does not contain requested file with requested checksum.
To repro, it should be enough to change one checksum in resources.xml
Report from Manatauro:
![pathjoin](/uploads/4b969168ba7babc21108c1db1a680f66/pathjoin.jpg)
****
mana/plus!82
|
|
|
|
Should this function gain emplace()-like semantics?
|
|
|
|
|
|
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
|