Opened 13 years ago

Closed 13 years ago

Last modified 11 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 by peres, 13 years ago

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 by SF/megath, 13 years ago

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

comment:3 by peres, 13 years ago

Owner: set to peres
Status: newclosed

comment:4 by peres, 13 years ago

Resolution: fixed

comment:5 by digitall, 11 months ago

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