Stop the monster list from being edited while we're iterating
Remove obsolete debugging check

That check won't be doing anything, since i is never modified in the loop. i.e., its sole effect is to make sure that the compiler's for (...) implementation is sane.

Also, it needs i >= z.size(), not i > z.size(), anyway.
Show note symbols on the minimap. Closes #508
Stop counting bionics/gunmods as weapons

This stops some inventory UI bugs where category labels would get mixed up.

This doesn't fix any systemic issues; for that, factoring out common logic in player::sort_inv() and find_firsts() would probably be best.
Merge remote-tracking branch 'dda/pr/511' into dda-master
Merge remote-tracking branch 'dda/pr/514' into dda-master
Merge remote-tracking branch 'dda/pr/515' into dda-master