Warn if consume_drug item has unused vitamins

[RenechCDDA]

Summary

Bugfixes "Fix some cases where food with consume_drug would not give vitamins and vice versa "

Purpose of change

Take an aspirin. Notice that you do not receive any vit_bloodthinner.

Put some calcium tablets into the faction camp's larder. Notice that they do not confer any vitamins.

The consume_drug_iuse actor can contain a vitamins field to optionally give vitamins. The items with this tend to be but don't have to be (I think??) comestibles. If the item has calories/vitamins defined in comestible they won't be added to the consumer's body.

Describe the solution

The least worst solution: Warn on load and force people to duplicate this stuff in json

Describe alternatives you've considered

• Shuffle the comestible's vitamins field into the iuse actor somehow. AWFUL. Do not want.

• Pray for E to eat food and a to activate it, no mixes #76920

• Extract consume_drug iuse out into an islot? 😩
  --Force it into comestible, somehow??

Testing

Not done testing, but also needs some testing.

In particular this is a fragile solution, it relies on the finalization order of itype's use functions and comestible data. I am really not happy with this.

Currently(with only one commit) should expect to fail basic build. No JSON work has been done.

Additional context

[RenechCDDA]

Related item where this issue presents:

Cataclysm-DDA/data/json/items/comestibles/med.json

Lines 1897 to 1901 in fdd2fba

	"vitamins": [ [ "calcium", 24, 48 ], [ "iron", 24, 48 ], [ "vitC", 24, 48 ] ]
	},
	"//": "Yes, the vitamins are intentionally duplicated here. The consume/use/iteminfo UI needs them in use_action, but to actually pass on the vitamins they need to be in the comestible data, as below.",
	"//2": "TODO: Fix this...",
	"vitamins": [ [ "calcium", 36 ], [ "iron", 36 ], [ "vitC", 36 ] ]

[ShnitzelX2]

Provided it works, a warning is better than silently failing.

I think the basic problem here is that consume_drug_iuse has fields that should be loaded through islot_comestible instead of separately. The "vitamins" fields from those two classes don't even use the same reader!

#76920 is more of a UI issue and would be nice to have even if some drugs are edible.

[RenechCDDA]

I simplified the C++ a lot. The nutrients function actually goes unused now, but will be needed for some other cleanups so it's staying in.

Now to do all the json crap.