[crossfire] Alchemy formulae file broken

Preston Crow pc-crossfire06 at crowcastle.net
Thu Feb 10 11:25:30 CST 2022


I pushed a fix to compute the total_chance at the end.

On 2022-02-10 02:09, Preston Crow wrote:
> I was doing some testing, and I found several basic alchemy formulas 
> never showed up in books (mercury, philosophical oil, and 
> philosophical salt).  After a lot of digging, I found that this 
> accounted for a chance of 30, and there are 30 chance-1 formulas near 
> the end of the file that aren't getting added to the total correctly.  
> This is because in recipe.c, it adds the chance to the total_chance 
> when it reads the ingredients, but on those 30 formulas, the chance is 
> listed in the file after the ingredients.
>
> While we could say the bug is in the formula file for putting the 
> entries in the wrong order, it would seem like the right fix is to 
> change recipe.c to not care about the order.  I think trying to set 
> the total_chance field as we parse the file is wrong, and we should 
> scan the lists at the end to set the value.
>


More information about the crossfire mailing list