diff options
author | Willem Jan Palenstijn | 2016-07-02 21:31:29 +0200 |
---|---|---|
committer | Willem Jan Palenstijn | 2016-07-02 21:31:29 +0200 |
commit | 4b72a42da93da5560ecde38010d328196ff727fd (patch) | |
tree | 9de8c15c55a06c25ea8f007ace7c18141328e11f /engines/sci/engine | |
parent | d643cb651f5661a80c7d7ea493ff397cf0bef3fd (diff) | |
download | scummvm-rg350-4b72a42da93da5560ecde38010d328196ff727fd.tar.gz scummvm-rg350-4b72a42da93da5560ecde38010d328196ff727fd.tar.bz2 scummvm-rg350-4b72a42da93da5560ecde38010d328196ff727fd.zip |
SCI: Remove presumably long-outdated FIXME
Diffstat (limited to 'engines/sci/engine')
-rw-r--r-- | engines/sci/engine/kernel.cpp | 14 |
1 files changed, 3 insertions, 11 deletions
diff --git a/engines/sci/engine/kernel.cpp b/engines/sci/engine/kernel.cpp index 4370e6de04..2fc338b618 100644 --- a/engines/sci/engine/kernel.cpp +++ b/engines/sci/engine/kernel.cpp @@ -84,23 +84,15 @@ uint Kernel::getKernelNamesSize() const { } const Common::String &Kernel::getKernelName(uint number) const { - // FIXME: The following check is a temporary workaround for an issue - // leading to crashes when using the debugger's backtrace command. - if (number >= _kernelNames.size()) - return _invalid; + assert(number < _kernelFuncs.size()); return _kernelNames[number]; } Common::String Kernel::getKernelName(uint number, uint subFunction) const { - // FIXME: The following check is a temporary workaround for an issue - // leading to crashes when using the debugger's backtrace command. - if (number >= _kernelFuncs.size()) - return _invalid; + assert(number < _kernelFuncs.size()); const KernelFunction &kernelCall = _kernelFuncs[number]; - if (subFunction >= kernelCall.subFunctionCount) - return _invalid; - + assert(subFunction < kernelCall.subFunctionCount); return kernelCall.subFunctions[subFunction].name; } |