Commit message (Collapse) | Author | Age | |
---|---|---|---|
* | channel_the_elements() on sand should use *Fire* skill level | Bardur Arantsson | 2015-02-23 |
| | | | | | | | | It seems very weird that the sand-based variant of channel_the_elements uses a *spell* which is only related via the "Fire" skill to determine whether to use hellfire or regular fire. This change ensures that it is consistent with the rest of channel_the_elements and ensures that we're in line with the documentation of Geomancy. | ||
* | Reduce duplication slightly in channel_the_elements() | Bardur Arantsson | 2015-02-23 |
| | |||
* | Fix bug where Horns of Ylmir would use Shake's spell level | Bardur Arantsson | 2015-02-23 |
| | |||
* | Fix bug related to God spellbooks | Bardur Arantsson | 2015-02-23 |
| | |||
* | Revert "Limit scope of {set,unset}_stick_mode for readability" | Bardur Arantsson | 2015-02-23 |
| | | | | | | | | | Turns out I'd missed at least one case where the stick would be activated without "stick_mode" being set, this would have the effect of inadvertently reducing/increasing the effective spell level depending on Magic-Device vs. $SPELL_SCHOOL skill level. This reverts commit 427c432e6d2805c88e4639c61b32975ee01735f8. | ||
* | Move school_spells array to spells5.cc | Bardur Arantsson | 2015-02-23 |
| | |||
* | Consolide the two spell_type headers | Bardur Arantsson | 2015-02-23 |
| | |||
* | Fix header inclusion order problem in spells3.cc | Bardur Arantsson | 2015-02-23 |
| | | | | | We failed to include spells3.hpp first, which could cause spurious trouble for clients of spells3.hpp. | ||
* | Move spell forward declarations from externs.h to spells3.hpp | Bardur Arantsson | 2015-02-23 |
| | |||
* | Remove unused/unnecessary bits of types_fwd.h | Bardur Arantsson | 2015-02-23 |
| | |||
* | Replace all empty "spell_info" functions with a single function | Bardur Arantsson | 2015-02-23 |
| | |||
* | Reformat spell_type ctor | Bardur Arantsson | 2015-02-23 |
| | |||
* | Inline get_target() into the one extant caller | Bardur Arantsson | 2015-02-23 |
| | |||
* | Change spell_chance_device parameter to use spell_type pointer | Bardur Arantsson | 2015-02-23 |
| | | | | There's no need to go through the extra indirection of the spell list. | ||
* | Inline static overload of spell_chance_device | Bardur Arantsson | 2015-02-23 |
| | |||
* | Split spell_chance into device and book variants | Bardur Arantsson | 2015-02-23 |
| | | | | | The code calling spell_chance() always actually knows which kind of spell its looking at, so we might as well exploit that. | ||
* | Make get_level_device_1 take a spell_type instead of an index | Bardur Arantsson | 2015-02-23 |
| | |||
* | Add tests for get_level_device | Bardur Arantsson | 2015-02-23 |
| | | | | | This is just a very basic set of sanity checking tests based on empirical testing, but hopefully it'll all start to add up. | ||
* | Make get_level_s static | Bardur Arantsson | 2015-02-23 |
| | | | | There was exactly one usage outside of spells3.cc, so inline that. | ||
* | Make device skill a parameter of get_level_device() | Bardur Arantsson | 2015-02-23 |
| | | | | This uncouples get_level_device() from s_info for easier testing. | ||
* | Limit scope of {set,unset}_stick_mode for readability | Bardur Arantsson | 2015-02-23 |
| | |||
* | Change get_level_school() to use direct spell_type pointer instead of index | Bardur Arantsson | 2015-02-23 |
| | |||
* | Use get_level_school explicitly in is_ok_spell | Bardur Arantsson | 2015-02-23 |
| | | | | | We are guaranteed that is_ok_spell is only ever called on "school spells", so this should be behavior-preserving. | ||
* | Use get_level_{school,device}_1 explicitly where we can | Bardur Arantsson | 2015-02-23 |
| | | | | | This gets us a little further away from relying on the get_level_use_stick global variable. | ||
* | Extract the two branches of get_level() into separate functions | Bardur Arantsson | 2015-02-23 |
| | |||
* | Remove unnecessary do_cmd_cast() extern decl | Bardur Arantsson | 2015-02-23 |
| | |||
* | Add BanditCpp 1.1.4 test harness | Bardur Arantsson | 2015-02-23 |
| | |||
* | Promote parameter of get_level_device() to spell_type * and make it static | Bardur Arantsson | 2014-12-23 |
| | |||
* | Inline set_target into convey_recall() in spells3.cc | Bardur Arantsson | 2014-12-23 |
| | |||
* | Remove dead extern declaration | Bardur Arantsson | 2014-12-23 |
| | |||
* | Move find_position() to spells3.cc | Bardur Arantsson | 2014-12-23 |
| | | | | Said file is the only place it's currently used. | ||
* | Remove "tolua" from .gitignore | Bardur Arantsson | 2014-12-23 |
| | |||
* | Make a few functions in skills.cc static | Bardur Arantsson | 2014-12-23 |
| | |||
* | Move private_check_user_directory() to modules.cc | Bardur Arantsson | 2014-12-23 |
| | | | | We need to move it out of main.c to support our test harness. | ||
* | Clean up compiler warnings | Bardur Arantsson | 2014-12-23 |
| | |||
* | Fix Theme description of Ulmo | Bardur Arantsson | 2014-12-23 |
| | |||
* | Fix signature of object_flavor | Bardur Arantsson | 2014-12-23 |
| | |||
* | Remove dead get_table_name() and its data file | Bardur Arantsson | 2014-12-23 |
| | |||
* | Remove unused struct birther fields | Bardur Arantsson | 2014-12-23 |
| | |||
* | Remove struct magic_type | Bardur Arantsson | 2014-12-23 |
| | |||
* | Make "pref_tmp_value" local | Bardur Arantsson | 2014-12-23 |
| | |||
* | Remove unused angband_sound_name | Bardur Arantsson | 2014-12-23 |
| | |||
* | Remove unused MAX_HOOKS | Bardur Arantsson | 2014-12-23 |
| | |||
* | Enable "maxmize stats" permanently | Bardur Arantsson | 2014-12-23 |
| | |||
* | Remove old-style hooks code | Bardur Arantsson | 2014-12-23 |
| | |||
* | Remove unused HOOK_BIRTH | Bardur Arantsson | 2014-12-23 |
| | |||
* | Remove unused HOOK_INIT_GAME | Bardur Arantsson | 2014-12-23 |
| | |||
* | Remove unused HOOK_DEBUG_COMMAND | Bardur Arantsson | 2014-12-23 |
| | |||
* | Remove unused HOOK_BUILDING_ACTION | Bardur Arantsson | 2014-12-23 |
| | |||
* | Remove unused HOOK_STORE_STOCK | Bardur Arantsson | 2014-12-23 |
| |