Age | Commit message (Collapse) | Author | Files | Lines |
|
Signed-off-by: Haru <haru@dotalux.com>
|
|
Signed-off-by: Haru <haru@dotalux.com>
|
|
- The functions were returning `(bool)-1` instead of `false` in case of
error, or `(bool)0` instead of `true` in case of success.
Signed-off-by: Haru <haru@dotalux.com>
|
|
INFINITE_DURATION, INVALID_TIMER, SC_NONE, INDEX_NOT_FOUND, depending on context.
Signed-off-by: Haru <haru@dotalux.com>
|
|
- The variable should be signed, since it uses the value -1 to indicate
infinite duration (and it's stored as signed in the database).
- Added #define for the special value -1 (INFINITE_DURATION).
- This fixes an issue causing status changes to fail being saved to
database (thanks to Michi for reporting it).
- Related to commit 8dc75721.
Signed-off-by: Haru <haru@dotalux.com>
|
|
- The npc-side code no longer depends on the client data layout.
Signed-off-by: Haru <haru@dotalux.com>
|
|
- The npc-side code no longer depends on the client data layout.
Signed-off-by: Haru <haru@dotalux.com>
|
|
- The npc-side code no longer depends on the client data layout.
Signed-off-by: Haru <haru@dotalux.com>
|
|
struct itemlist
- The skill-side code no longer depends on the client packet data layout.
- Some data validation has been added.
Signed-off-by: Haru <haru@dotalux.com>
|
|
- The npc-side code no longer depends on the client packet data layout.
Signed-off-by: Haru <haru@dotalux.com>
|
|
- To be used in various non-clif functions that right now take client
data buffers directly.
Signed-off-by: Haru <haru@dotalux.com>
|
|
Signed-off-by: Haru <haru@dotalux.com>
|
|
Signed-off-by: Haru <haru@dotalux.com>
|
|
Signed-off-by: Haru <haru@dotalux.com>
|
|
- WBUFP now returns a pointer to void, which automatically casts to any
non-const or const pointer.
Signed-off-by: Haru <haru@dotalux.com>
|
|
- This is necessary for compatibility with a const RP2PTR/RFIFO2PTR
Signed-off-by: Haru <haru@dotalux.com>
|
|
- While this is arguable, those explicit typecasts are potentially
dangerous/misleading (for example, a const specifier might get
accidentally dropped without even generating a compiler warning, or a
variable type might change during code changes, and any related
warning would get silenced by the explicit typecast).
- As a reminder Hercules is written in C, and not in C++ (and there's
no such thing as "compiling in C++ mode" - they're two different
languages.) As such, it is legal to let the compiler automatically
promote void* from/to any non-const pointer type, as well as const
void* from/to any const pointer type.
Signed-off-by: Haru <haru@dotalux.com>
|
|
Signed-off-by: Haru <haru@dotalux.com>
|
|
Now it no longer needs to memcpy data to a temporary buffer.
Signed-off-by: Haru <haru@dotalux.com>
|
|
- This is necessary for compatibility with a const RFIFOP.
Signed-off-by: Haru <haru@dotalux.com>
|
|
A read FIFO/buffer should be, well, read-only.
Signed-off-by: Haru <haru@dotalux.com>
|
|
Signed-off-by: Haru <haru@dotalux.com>
|
|
- Changed literal 1L values used in unit64 bit-shift operations to 1ULL.
- Follow-up to f606519c3b611573ade246ec82db26437bf03668 (reported by
zackdreaver)
Signed-off-by: Haru <haru@dotalux.com>
|
|
Reported in http://herc.ws/board/topic/11994-map-crashed-on-friendly-poring-quest-novice-quest/
|
|
Signed-off-by: Haru <haru@dotalux.com>
|
|
|
|
|
|
Also add compiler flag for check for missing noreturn attributes.
|
|
Add all missing -Wformat flags into configure.
|
|
Add -Wcast-qual into configure comment.
|
|
Added some forgotten <stdio.h> and "common/conf.h" includes to .c files,
that were previously masked by the above.
Signed-off-by: Haru <haru@dotalux.com>
|
|
Signed-off-by: Haru <haru@dotalux.com>
|
|
- The return value is now consistent with the libconfig standard
(CONFIG_TRUE/CONFIG_FALSE).
- Removed some redundant error messages.
Signed-off-by: Haru <haru@dotalux.com>
|
|
Related to #1152
Signed-off-by: Haru <haru@dotalux.com>
|
|
Signed-off-by: Haru <haru@dotalux.com>
|
|
PR_MAGNIFICAT Skill Fix.
|
|
Revamped Aegis style songs behavior. Fixes #835
|
|
Mob DB support for the db2sql plugin
|
|
WFIFO/RFIFO.
Change packet database enums to defines in mmo.h
|
|
Enable pcre by default in configure and show error if pcre missing.
|
|
Fixes crash with @send
|
|
more meaningful names for fields of enum guild_permission
|
|
Signed-off-by: Haru <haru@dotalux.com>
|
|
This fixes several rounding errors happening in various places (i.e.
the base exp for HORONG being calculated as 819 instead of 820 when the
server rates are set to 1x)
Signed-off-by: Haru <haru@dotalux.com>
|
|
- Separated load and validation logic.
- The load/validation functions now return the loaded mob ID
Signed-off-by: Haru <haru@dotalux.com>
|
|
- To be used by the constdb2doc plugin
Signed-off-by: Haru <haru@dotalux.com>
|
|
Now packet id will be passed to Hercules Plugin Manager instead of let it figure it on itself
|
|
|
|
|
|
Fixes monsters not taking rude attack when OFFICIAL_WALKPATH is used.
|