scummvm debugger

[eriktorbjorn]

Yes, but I think this IF structure lets argv[3] be used only if argc is DIFFERENT from 3! Or not?

Well, you know argc is at least 3, because that's the first thing the function checks. So, to make up an example:

"object 42" - argc is 2, so it fails.
"object 42 pickup" - argc is 3, so it succeeds, using _vm->_currentRoom
"object 42 pickup 69" - argc is 4, so it succeeds, using argv[3], i.e. "69"

So I still don't see any problem.

[jumpjack]

Yes, but I think this IF structure lets argv[3] be used only if argc is DIFFERENT from 3! Or not?

Well, you know argc is at least 3, because that's the first thing the function checks. So, to make up an example:

"object 42" - argc is 2, so it fails.
"object 42 pickup" - argc is 3, so it succeeds, using _vm->_currentRoom
"object 42 pickup 69" - argc is 4, so it succeeds, using argv[3], i.e. "69"

So I still don't see any problem.

ok, thanks, now I understand