Opened 13 years ago

Closed 12 years ago

Last modified 9 months ago

#8601 closed patch (fixed)

small patch to paralaction dropItem()

Reported by: SF/megath Owned by: peres
Priority: normal Component: Engine: Parallaction
Keywords: Cc:


fixes crash at first screen :)

Index: engines/parallaction/inventory.cpp

--- engines/parallaction/inventory.cpp (revision 25120)
+++ engines/parallaction/inventory.cpp (working copy)
@@ -159,7 +159,8 @@

if (_di == 0) continue;

- memcpy(&_inventory[_si], &_inventory[_si+1], sizeof(InventoryItem));
+ //memcpy(&_inventory[_si], &_inventory[_si+1], sizeof(InventoryItem));
+ memset(&_inventory[_si], 0, sizeof(InventoryItem));


Ticket imported from: #1639159. Ticket imported from: patches/706.

Change History (5)

comment:1 Changed 13 years ago by peres

Hi megath

please let me know the architecture you're trying Parallaction on and your OS.

Your patch crashes the engine as soon as you try to drop/combine items from the inventory, but I see your point. I should have fixed your issue on revision 25140, so please update.

comment:2 Changed 13 years ago by SF/megath

i386-linux, thank you, it doesn't crash anymore. When could we expect save/load feature ? :)

comment:3 Changed 12 years ago by peres

Owner: set to peres
Status: newclosed

comment:4 Changed 12 years ago by peres

Resolution: fixed

comment:5 Changed 9 months ago by digitall

Component: Engine: Parallaction
Note: See TracTickets for help on using tickets.