feature_has should ideally only run once per feature (to avoid redundant log entries). Maybe bucket could set a variable if it's enabled?
Remove the (* and ).
This should probably be static.
Does FoodItem not have its own constructor?
Use FoodItem_useTimeDepleted_non_virtual rather than de-referencing the address yourself.
Do we really care about MCPE compatibility that much? IMO, the code might be nicer if the milk bucket was just a separate item. What are your thoughts?
Why did you add this to the v2.5.3 changelog?
Should this be a pointer?
Does this have any impact on how long it takes to join a server? Also, should this be included in misc so it also applies to LAN games?
This should be 2. You can determine alignment by adding ERR("Alignment: %i", _Alignof(short)); to a file.