Brennan Ravenscroft

Nevermind, I see the intention is to be able to add to other items, which would require an existing mutable list to be accessible.

You could, however, get around the binary break by adding two methods - one `getItemDecorators` that returns an immutable view, and one `addItemDecorator` that allows for adding a decorator. Then...

Test Results - [x] Iron Golems are replaced with Giants - [x] At most one house in Plains villages are Diamond Cubes - [x] Diamond Cube always generates - [x]...

Good catch, we don't want to have longer names breaking the whole gui. Though we probably want to make that full name visible somehow... A solution adding a hovered tooltip...

docs have been updated for all classes in `net.minecraftforge.common.capabilities`

As an aside I'd like to ask @Sm0keySa1m0n if you know how much this would impact https://github.com/MinecraftForge/MinecraftForge/pull/8706 Would it make it (easier/harder/impossible) to achieve given the new design?

Further changes: * `ItemStack#capNBT` removed in favor of direct usage of `CapabilityProvider#lazyCapNbt` * `IComparableCapabilityProvider` removed and rolled into `IForgeItemStack` * `AttachCapabilitiesEvent.ItemStacks` updated to only accept `IItemStackCapabilityProvider` * `isEquivalentTo` and `copy`...

I don't like IComparable for that simply because it also adds copy() on top of compare(). I'm not sure if we'll ever see a viable usecase for it other than...

| Could it be used for player cloning or is that too far fetched? Possibly, but it would require separating copy() from compare() to allow a subinterface for players that...

Latest changeset * `IItemStackCapabilityProvider` split into `IComparableCapabilityProvider` and `ICopyableCapabilityProvider`. * `AttachCapabilitiesEvent.Players` added. Attaching to `Player` now requires subclassing `ICopyableCapabilityProvider`. * Caps that are attached to Players will now automatically be...