Merge pull request #30 from Baalberith6/master

We can't use unsigned int for equip index since we put -1 in sd->equip_index[i] as sign of an empty place.
Merge pull request #31 from Baalberith6/master

pc_get_child can return NULL > segfault
DB Import2

Add db/import-tmpl/ and add all file that could currently be
overwritten, (please add note if a db is missing)

db/import folder is created by "make import" just like conf/import and
msg_conff/import, (didn't found the visual equivalent)
Db import3

Remove map_index.txt as this ain't overwrittable
add missings newlines at end of file
* Fixed itemgroup_db that being reseted when read import file. http://rathena.org/board/tracker/issue-8419-cant-not-open-any-itemgroup/
* Fixed typo for maxsp bonus as follow up of 96443cd. Thank Baalberith

Signed-off-by: Cydh Ramdh <house.bad@gmail.com>
Global_Function updates.
* Added the following functions:
  - "F_GetPlural" (returns plural of noun) / "F_InsertPlural" (returns plural if arg != 1)  [credits: EnglishClub for formation rules]
  - "F_InsertArticle" (returns 'a' or 'an' preceding word)
  - "F_InsertComma" (formats number with commas)
  - "F_GetNumSuffix" (formats number with '-st', '-nd', '-rd', or '-th' suffix)
* Added and removed new functions where necessary. (incomplete)
* Added new types to "F_GetArmorType" function, and weapons will now pass to "F_GetWeaponType".
* Reformatted comments.

Other changes:
* Removed some garbage text from scripts.
* Slight edits to custom warper script.

Signed-off-by: Euphy <euphy.raliel@rathena.org>
Minor bug fixes.
* Added itemscript for WoE Weapon Supply Box (12675). (bugreport:8421)
http://rathena.org/board/tracker/issue-8421-woe-weapon-supply-box12675-missing-item-script/
* Fixed two block comment errors. (bugreport:8409)
http://rathena.org/board/tracker/issue-8409-script-error-on-item-db-re-line-6123/
* Follow-ups to 97687ca.

Signed-off-by: Euphy <euphy.raliel@rathena.org>
* Fixed bug: http://rathena.org/board/tracker/issue-8345-specialeffect-target-self-doesnt-work
* Fixed bug: http://rathena.org/board/tracker/issue-8169-hiding
* Fixed bug: http://rathena.org/board/tracker/issue-8366-egg-type-monster-issue
* Fixed bug: http://rathena.org/board/tracker/issue-8406-item-bonus-not-working
* Added fix with mob_hp update when mob get healed (Hercules merge)
* Fixed bugreport http://rathena.org/board/tracker/issue-8408-script-line-931-data-too-long-for-column-atkmatk-at-row-1/

Signed-off-by: Cydh Ramdh <house.bad@gmail.com>
DB Import Visual Studio updates.
Also dropped folder db/import, which should be user-created.
[Follow-up to ff29c23, credits @Lemongrass3110.]

Signed-off-by: Euphy <euphy.raliel@rathena.org>
Follow-up to Global_Function updates (97687ca).
* Added format string to "F_InsertPlural" for added flexibility.
* Added more checks to "F_GetPlural" for better handling of multi-word arguments (e.g. items).
* Updated some more scripts with new functions.

Some misc. cleanup of constants in scripts.

Signed-off-by: Euphy <euphy.raliel@rathena.org>
* Fixed item_combo bug, related bugreport http://rathena.org/board/tracker/issue-8321-maximum-hp-bug/
Thank lighta and everyone who told how to reproduce

Signed-off-by: Cydh Ramdh <house.bad@gmail.com>
Follow-up to Global_Functino updates (834f3ba).
"F_GetPlural" now works for most item/mob cases, but still does not detect inputs that are already pluralized.
- For formats "... of|in|on ..." (ex. Piece of Cake), preceding word will be pluralized (ex. Pieces).
- Added some additional checks to prevent common errors.

Signed-off-by: Euphy <euphy.raliel@rathena.org>
* pc_combocheck fix. *combo_idx should be has -1 as the init value, not 0. Thx Daegaladh

Signed-off-by: Cydh Ramdh <house.bad@gmail.com>
Bug Fixes
* Fixed an operator check for WE_MALE and WE_FEMALE. Thanks Baalberith.
Optimizations
* Fixed a possible null pointer dereference in Spirit Ball check, Change Cart check, WE_BABY, MH_PYROCLASTIC, and MH_LIGHT_OF_REGENE.
* Fixed a reassigned value being reset before being checked.
* Fixed a homunculus expression always being false because of prior check.
* Fixed sprintf specifier output trying to display int value when return value is unsigned int.
* Fixed a redundant check for when monster's HP is less than 0 when HP is declared as an unsigned int.
* Fixed a redundant check for when uAccLen or uTokenLen are less then 0 when both are declared as unsigned int.
* Various other clean ups.
* Fixed bug: http://rathena.org/board/tracker/issue-8427-use-reload-mobdb-crash-the-map-server/
Merge remote-tracking branch 'upstream/master'

Conflicts:
	src/map/battle.c
	src/map/skill.c