[crossfire] auto_apply_fix() issue

Preston Crow pc-crossfire06 at crowcastle.net
Sat Feb 5 12:57:55 CST 2022


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 at 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
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mailman.metalforge.org/pipermail/crossfire/attachments/20220205/ba1a3a27/attachment.htm>


More information about the crossfire mailing list