Yes, I'm aware of that, but I'm paranoid about how code might change in the future.

On 2022-02-05 13:42, Nicolas Weeger wrote:
Hello.


The patch looks fine to me, thanks :)


One small remark:

The test

if ( !tmp->inv ) return;

at start of auto_apply_fix_inventory isn't required, since you always check the
inv before calling the function.


Best regards


Nicolas

_______________________________________________
crossfire mailing list
crossfire@metalforge.org
http://mailman.metalforge.org/mailman/listinfo/crossfire
IRC: http://crossfire.real-time.com/irc/index.html
Discord: http://crossfire.real-time.com/discord/index.html
Project Site: https://sourceforge.net/projects/crossfire/
Wiki: http://wiki.cross-fire.org/
Website: http://crossfire.real-time.com

_______________________________________________
crossfire mailing list
crossfire@metalforge.org
http://mailman.metalforge.org/mailman/listinfo/crossfire
IRC: http://crossfire.real-time.com/irc/index.html
Discord: http://crossfire.real-time.com/discord/index.html
Project Site: https://sourceforge.net/projects/crossfire/
Wiki: http://wiki.cross-fire.org/
Website: http://crossfire.real-time.com

Reply via email to