History

log cleanup per CR; add more specific hints (instead of relying on __FUNCTION__)

humbletim committed 9 years ago

Commit 075574b1 parent c5598381 file · +11 −9Browse files

Showing the changes to libraries/script-engine/src/ScriptEngine.cpp (1 of 1 file). Show all

Unified · Split

libraries/script-engine/src/ScriptEngine.cpp+11−9
@@ -564,7 +564,6 @@ void ScriptEngine::resetModuleCache(bool deleteScriptCache) { · 563 unmodified lines
564564 if (it.flags() & QScriptValue::SkipInEnumeration) {
565565 continue;
566566 }
567− //scriptCache->deleteScript(it.name());
568567 qCDebug(scriptengine) << "resetModuleCache(true) -- staging " << it.name() << " for cache reset at next require";
569568 cacheMeta.setProperty(it.name(), true);
570569 }
@@ -594,8 +593,8 @@ void ScriptEngine::init() { · 23 unmodified lines
594593 entityScriptingInterface->init();
595594 connect(entityScriptingInterface.data(), &EntityScriptingInterface::deletingEntity, this, [this](const EntityItemID& entityID) {
596595 if (_entityScripts.contains(entityID)) {
597− if (isEntityScriptRunning(entityID)) {
598− qCWarning(scriptengine) << "deletingEntity while entity script is still running!" << entityID;
596+ if (isEntityScriptRunning(entityID) && !isStopping()) {
597+ qCWarning(scriptengine) << "deletingEntity while entity script is still running" << entityID;
599598 }
600599 _entityScripts.remove(entityID);
601600 emit entityScriptDetailsUpdated();
@@ -954,7 +953,7 @@ QScriptValue ScriptEngine::evaluate(const QString& sourceCode, const QString& fi · 352 unmodified lines
954953 syntaxError.setProperty("detail", "evaluate");
955954 }
956955 raiseException(syntaxError);
957− maybeEmitUncaughtException(__FUNCTION__);
956+ maybeEmitUncaughtException("lint");
958957 return syntaxError;
959958 }
960959 QScriptProgram program { sourceCode, fileName, lineNumber };
@@ -962,14 +961,14 @@ QScriptValue ScriptEngine::evaluate(const QString& sourceCode, const QString& fi · 1 unmodified line
962961 // can this happen?
963962 auto err = makeError("could not create QScriptProgram for " + fileName);
964963 raiseException(err);
965− maybeEmitUncaughtException(__FUNCTION__);
964+ maybeEmitUncaughtException("compile");
966965 return err;
967966 }
968967
969968 QScriptValue result;
970969 {
971970 result = BaseScriptEngine::evaluate(program);
972− maybeEmitUncaughtException(__FUNCTION__);
971+ maybeEmitUncaughtException("evaluate");
973972 }
974973 return result;
975974 }
@@ -1644,11 +1643,11 @@ QScriptValue ScriptEngine::require(const QString& moduleId) { · 668 unmodified lines
16441643
16451644 auto exports = module.property("exports");
16461645 if (!invalidateCache && exports.isObject()) {
1647− // we have found a cacheed module -- just need to possibly register it with current parent
1646+ // we have found a cached module -- just need to possibly register it with current parent
16481647 qCDebug(scriptengine_module) << QString("require - using cached module '%1' for '%2' (loaded: %3)")
16491648 .arg(modulePath).arg(moduleId).arg(module.property("loaded").toString());
16501649 registerModuleWithParent(module, parent);
1651− maybeEmitUncaughtException(__FUNCTION__);
1650+ maybeEmitUncaughtException("cached module");
16521651 return exports;
16531652 }
16541653
@@ -2380,9 +2379,12 @@ void ScriptEngine::unloadEntityScript(const EntityItemID& entityID) { · 725 unmodified lines
23802379 const EntityScriptDetails &oldDetails = _entityScripts[entityID];
23812380 if (isEntityScriptRunning(entityID)) {
23822381 callEntityScriptMethod(entityID, "unload");
2383− } else {
2382+ }
2383+#ifdef DEBUG_ENTITY_STATES
2384+ else {
23842385 qCDebug(scriptengine) << "unload called while !running" << entityID << oldDetails.status;
23852386 }
2387+#endif
23862388 if (oldDetails.status != EntityScriptStatus::UNLOADED) {
23872389 EntityScriptDetails newDetails;
23882390 newDetails.status = EntityScriptStatus::UNLOADED;