Recent content by Haru

About Code Review and Why You'd Want Your Code to Be Reviewed Hello, fellow developers and code contributors! As you certainly know, years ago, Hercules adopted a workflow based on pull requests, that includes code review as one of the necessary steps before any new piece of code makes it into...

Rationale: This changeset updates the syntax of the configuration files, to allow for more flexibility, and get rid of our some times buggy custom parser. Contents: All the txt-based configuration files in the conf folder have been replaced with more modular files that use the libconfig...

Rationale: This changeset offers improvements to the HPMHooking, making it capable to detect, at compile time, an error in the type of a hook function, as well as allowing pre-hooks to be more powerful when it comes to pointer-type arguments. Contents: The HPMHooking macros addHookPre() and...

Rationale: This is according to our Supported Platforms policy. For an overview of supported OSes and compilers, please see the wiki page https://github.com/HerculesWS/Hercules/wiki/Supported-Platforms Contents: VS2015 is our primary target compiler on Windows, and this merge removes all the...

Rationale: This is a follow-up to the Mob DB conversion to libconfig format from some months ago, and brings it on par with the Item DB. Contents: - The JName field has been restored (and made optional), for the entries that need it. - Several numeric values in the mob DB have been replaced...

Rationale: This is an (unplanned) episode in the txt->libconfig conversion project. The reason for the conversion is that we suddenly needed a way to add attributes (specifically, "deprecated" to some constants, since simply removing them would mean that any scripts using them would silently...

Rationale: Historically, we've received several crash reports or null pointer reports that were of barely any use, because they were produced by servers built without debug information. Contents: The default build mode, when using the UNIX ./configure && make build scripts, is now set...

Rationale: This is part of a larger source cleanup project. Following the code style guidelines we decided to adopt (linux kernel guidelines), typedefs should be avoided except where necessary. At the same time, we noticed that we have far too many explicit casts through the code, that might...

Rationale: This is part of the NPC scripts standardization project. In the past, NPCs were defined with numeric View IDs, while now we've replaced most of them with more human-readable (and as such easier to maintain) constants - the same constants that AEGIS scripts use, making the numeric IDs...

Mob DB file structure overhaul   Hello~!   Just like the Item DB, the Mob DB has a less than optimal (read: terrible) format, that doesn't play well with Git or any other Version Control System.   For example, this is a snippet from the current file: 1012,RODA_FROG,Roda Frog,Roda...

Hello everyone~! Following Ind's resignation from his administrator position last month, we've been working on transferring our services to a new host. Some services have already been transferred without any downtime (most notably the stat-server), and now it's finally time to move the rest...

Item DB file structure overhaul   Hello~! Uguu~?! We noticed that the Item Database file format, unchanged for years, is less than optimal (read: terrible) The file is really hard to read (is it the fifteenth or the sixteenth zero? No wait, that line has an extra comma!!) Whenever you merge an...