Jump not jumping

The jump statement is pretty straightforward, but this is the first time I've used it, so perhaps there's some arcane knowledge I'm missing.

    changed (integer intChange) {
        if (intChange & CHANGED_ALLOWED_DROP) {
            integer x;
            integer intLoadCard = FALSE;

            string  strItemName;
            integer intItemType;
            key     keyItemCreator;

            integer intInvCount = llGetInventoryNumber(INVENTORY_ALL);

            llSay(0, (string)intInvCount);

            for (x=0; x<intInvCount; x++) {
                strItemName    = llGetInventoryName(INVENTORY_ALL, x);
                keyItemCreator = llGetInventoryCreator(strItemName);
                intItemType    = llGetInventoryType(strItemName);

                llSay(0, (string)x + " [" + strItemName + "]");

                if (keyItemCreator == keyCreator) {
                    llSay(0, "My Creator!");
                    jump continue;
                }

                if (keyItemCreator == keyTenant
                &&  intItemType == INVENTORY_NOTECARD
                &&  strItemName == "Guest List") {
                    intLoadCard = TRUE;
                    jump continue;
                }

                llSay(0, "Kill it!!!!");

                llRemoveInventory(strItemName);

                @continue;
            }
        }
    }

The "keyCreator" is elsewhere set to llGetCreator(). On inventory items belonging to the creator, I get both the "my creator" and "kill it" messages -- i.e. there is no jump.

Pardon the thread overloading, but also with this code, when the loop comes to the script itself, llGetInventoryName() returns a null string. I've tested it in other places, and it returns the script name with no problem. Any issues here I might not be aware of?

I figured out this part -- deleting inventory items changes the numbering of the remaining ones; I need to make a list (yay!) of items to be deleted, and delete them after the loop (with, of course, another loop).

The trick when deleting inventory is to loop BACKWARDS through the list.

for (x=(intInvCount-1);x>=0;x--)
{
}

It seems by testing, that only one jump is allowed per label. See here: http://www.lslwiki.net/lslwiki/wakka.php?wakka=jump&show_comments=1

You can fix this by having @continue1; @continue2; one right after the other, and jump continue1; jump continue2; instead of one jump continue; used twice. Or you can structure your code better so that jumps are not needed (since, you know, using jumps is considered bad habit in programming).

How bizarre!

That's what I ended up doing -- restructuring the code, but I don't know that I'd say it's better; I think the original was easier to read. And the old saw about jump == evil just ain't always so. No one says anything about such things as break and continue (what I was trying to emulate), and that's essentially what they are.

The whole jump/break/continue handling in LSL is a complete crock IMNSHO


You might want to use lslint on your code, it will catch junk like this and other gotcha's like the compiler exceeding it's own stack!