Nevermind, the original code up there also works fine. I just copied it wrong for Roodylib.

if not (i is worn and i is clothing and obj = player)

In case anyone see this topic while forum spelunking, we have since decided the line should be:
[code]if not (i is worn and i is clothing and obj = player)[/code]

The only thing I can see it breaking is if somebody was flagged something as "worn" even though it wasn't wearable, and was counting on it to not be calculated in the player's holding value. But (a) that's probably unlikely, since it has to be "clothing" unless it's forced to "worn" in code (grammar def wouldn't allow it), and (b) it's really only an issue if you're managing inventory, which is unpopular anyway. :)

Again, I'll have to look through the logic of this, but if it stands up, it should probably become a fix.

Thanks.

--Kent

if i is not worn or i is not clothing or obj ~= player

"mobile" is an alias attribute for "worn". If you are carrying something which is mobile (i.e, you could drag it with a rope, but maybe you can carry it too), then its size isn't going to be added to player.holding in the CalculateHolding routine.

It looks like it would be easy enough to fix by adding a check to see if the "worn" item is also "clothing":

[code]if i is not worn or i is not clothing or obj ~= player[/code]