Question BUG empty container writes garbage to obj keywords

If I drank everything out of a barrel (or any drink container) it would then push the last thing I typed into they keywords (as an imm and as a player) or garbage characters (the dreaded half diamond question mark))!

 I'm positive I was the cause, but couldn't track it. so I grabbed a fresh copy of tba and.. the same thing happened.

I saw a thread in 2020 and it looks like the 2025 code is updated, but the problem persists. I attached what I did in the updated TBA as "drinking problem" and my bug report from the crash in my mud.

Anyone replicate/verify?

Reproducable. I never liked this stock behavior. When you empty a liquid container it removes a keyword. So name_from_drinkcon is likely buggy.

Try removing this from do_drink:
/* Remove liquid keyword */
name_from_drinkcon(obj);

Probably a good idea for us to remove name_from_drinkcon completely.

Anyone see a reason to keep this?

The reason is that if you drink from a bottle of water, you can "drink water". Once the container is empty, it is just a bottle, and "drink water" makes no sense. Once you fill it with wine, "drink water" is misleading, and you want to be able to  "drink wine".

The problem seems to be that, instead of just altering _this instance_ of the container, it frees the prototype, too. I think it's an easy fix.

Edit: Indeed, it was:  github.com/tbamud/tbamud/pull/149