Nothing is as goot it can't get better. :-)

I think it would be better and safer if SpellManager::instance was defined
as

SpellManager&
SpellManager::instance()
{
    SpellManager manager;
    return manager;
}

It make it clear it is a singleton pattern (returning a reference) and
theres no longer any need for its callers to check the returnvalue.

I didn't change it myself since I believe this is work in progress, and I
was too lazy to look up where it's used from. :-)

/Mike


Reply via email to