Question MSDP Inventory

Do you also get the same warnings ?

Must be a better way to do it, maybe Thomas or someone else has an idea ?

Yeah I do, mine is:

comm.c:2997:34: warning: zero-length gnu_printf format string [-Wformat-zero-length]
It still compiles and works.

But I did a bit of googling and found this explanation:
"It's not harmful. It's just letting you know there's something questionable about your code, that you might have made a mistake. Although sprintf (str, ""); is legal, it's a rather round-about way of making str a zero-length string, and could indicate a logical error in your code. It's not forbidden; just brought to your attention." - Phil Perry on Stack Overflow website.

So more googling revealed that another way to do it is this: Which compiles without error - and is used in other parts of TBAmud code, so I think this is the better option.

So
buf[0] = '\0';

Instead of the line:
snprintf(buf, sizeof(buf), "");
?

There are only two remotely similar placed in my tbacode that says: *buf = '\0';

Do you know Whiskey ?

Yeah so replace snprintf(buf, sizeof(buf), "");
with
buf[0] = '\0';

That works and it compiles without warning. I've updated the original post with our findings

So that was inventory, what about what is worn in eq ?