 |
Committed with revision #9938 SysCVar and player |
 |
Some issues with code: 1) You should save/restore multiloot members (both of them). 2) You should initialize multiloot members in constructor (both of them). 3) You changed order: previously item was first frobbed, then added to inventory, now it is vice versa. This can break a frob script which checks inventory and e.g. looks if the item is already present. 4) Do not assign 0 or 1 to boolean variable (multiloot), assign false/true instead. Regarding p.3, I think you need to check for "inv_name" spawnarg in order to learn if item is going to inventory, then call frob script, then actually add it. |
|
|
1.) That was actually intentional, but now that I think about it not thought through (the boolean should be zero when loading a savegame). 2.) Oops. Newb mistake, sorry. 3.) I haven't thought of that possibility, the changed order was intentional, but you are right. 4.) I thought I have used true and false everywhere, but may have missed some. I am still used to this and afaik it is legitime, even though no good style. Will change it nevertheless. Thanks for helping me cleaning up my mess, although your not my mom. I appreciate it. :) |
|
|
Committed with revision #9941 |
|
|
I committed two small fixes in rev 9944. |
- Anonymous
