fiddlerwoaroof/crawl - Change PL6OZOP5RJ5BMGQWKYTF4R2SQEULSAJSB5IYUIGEM7NRMGUBXIPAC

Ran into a bug where searching backwards through inventory for a free

slot which won't leave gaps (in find_free_slot()) would fail if the inventory was empty; fixed bug by making find_free_slot() retry with forwards search if backwards search failed. I can't reproduce the bug, but the fix doesn't seem to cause any problems when there isn't a bug.

git-svn-id: https://crawl-ref.svn.sourceforge.net/svnroot/crawl-ref/trunk@2368 c06c8d41-db1a-0410-9941-cceddc491573

Created by zelgadis on October 7, 2007

PL6OZOP5RJ5BMGQWKYTF4R2SQEULSAJSB5IYUIGEM7NRMGUBXIPAC

Dependencies

In channels

main

Change contents

Replacement in crawl-ref/source/items.cc at line 1379 [3.2584728]

B:BD[2.353304] → [2.353304:353519]

    if (searchforward)
    {
        // Return first free slot
        for (slot = 0; slot < ENDOFPACK; ++slot) {
            if (!is_valid_item(you.inv[slot]))
                return slot;
        }
    }
    else

[2.353304]

[2.353519]

    if (!searchforward)

Insertion in crawl-ref/source/items.cc at line 1399 [3.2584728]

[2.354141]

    }
    // Either searchforward is true, or search backwards failed and
    // we re-try searching the oposite direction.
    // Return first free slot
    for (slot = 0; slot < ENDOFPACK; ++slot) {
        if (!is_valid_item(you.inv[slot]))
            return slot;

Insertion in crawl-ref/source/items.cc at line 1409 [3.2584728]
[2.354147]
[2.354147]