I'm looking in use-obj.c and it seems to me that Trumpet of Warning is the only one which is impossible to id by use? I'm not sure if this was intentional or not, and I didn't examine too far so maybe I'm missing something.
It ignores the return value of fire_beam (all the other cases use it to determine whether the thing should be identified), and then it calls make_aggressive which returns nothing (so there doesn't seem to be any way from this fcn to even tell that it had an effect.)
I guess make_aggressive could be changed to return a value if any monsters switched behaviour, and *ident = TRUE on that?
This is obviously a minor thing, but if it's a bug I can just fix it lol
It ignores the return value of fire_beam (all the other cases use it to determine whether the thing should be identified), and then it calls make_aggressive which returns nothing (so there doesn't seem to be any way from this fcn to even tell that it had an effect.)
I guess make_aggressive could be changed to return a value if any monsters switched behaviour, and *ident = TRUE on that?
This is obviously a minor thing, but if it's a bug I can just fix it lol
Comment