diff -r aa2539c91954 -r 1c2bb2fc7c87 perfsrv/piprofiler/plugins/PWRplugin/src/PwrPlugin.cpp --- a/perfsrv/piprofiler/plugins/PWRplugin/src/PwrPlugin.cpp Fri Oct 08 14:56:39 2010 +0300 +++ b/perfsrv/piprofiler/plugins/PWRplugin/src/PwrPlugin.cpp Tue Oct 26 16:20:32 2010 +0300 @@ -19,8 +19,8 @@ #include #include -#include -#include +#include +#include #include @@ -324,11 +324,9 @@ iNominalCapa(0), iVoltage(0), iCurrent(0), - iPowerAPI(0) -#ifdef PWR_SAMPLER_BACKLIGHT - ,iLightAPI(0), + iPowerAPI(0), + iLightAPI(0), iBackLightStatus(CHWRMLight::ELightStatusUnknown) -#endif { LOGTEXT(_L("CProfilerPowerListener::CProfilerPowerListener() - konstuktori")); @@ -363,13 +361,12 @@ delete iPowerAPI; iPowerAPI = 0; } -#ifdef PWR_SAMPLER_BACKLIGHT + if (iLightAPI) { delete iLightAPI; iLightAPI = 0; } -#endif LOGTEXT(_L("CProfilerPowerListener::~CProfilerPowerListener() - exit")); } @@ -429,6 +426,8 @@ // Read HWRM reporting settings from central repository CRepository* centRep = CRepository::NewL(KCRUidPowerSettings); + CleanupStack::PushL( centRep ); + TInt baseInterval(0); User::LeaveIfError(centRep->Get(KPowerBaseTimeInterval, baseInterval)); User::LeaveIfError(centRep->Get(KPowerMaxReportingPeriod, iOriginalReportingPeriod)); @@ -437,6 +436,7 @@ LOGSTRING2("CProfilerPowerListener::StartL() - Original HWRM max power reporting period: %d", iOriginalReportingPeriod); User::LeaveIfError(centRep->Set(KPowerMaxReportingPeriod, KReportingPeriodInfinite)); + CleanupStack::PopAndDestroy(); // Power reporting interval reading may return too low value sometimes. Minimum value expected to be 250ms. if ( baseInterval < KMinSampleInterval ) @@ -468,10 +468,8 @@ return status.Int(); } -#ifdef PWR_SAMPLER_BACKLIGHT // Start monitoring backlight status iLightAPI = CHWRMLight::NewL(this); -#endif LOGTEXT(_L("CProfilerPowerListener::StartL() - exit")); return KErrNone; @@ -505,9 +503,7 @@ LOGSTRING2("CProfilerPowerListener::Sample() - Nominal capacitance: %d", iNominalCapa); LOGSTRING2("CProfilerPowerListener::Sample() - Voltage: %d", iVoltage); LOGSTRING2("CProfilerPowerListener::Sample() - Current: %d", iCurrent); -#ifdef PWR_SAMPLER_BACKLIGHT LOGSTRING2("CProfilerPowerListener::Sample() - Backlight status: %d", (TUint8)iBackLightStatus); -#endif iSample[0] = iNominalCapa; iSample[1] = iNominalCapa >> 8; @@ -517,7 +513,6 @@ iSample[5] = iCurrent >> 8; iSample[6] = iCurrent >> 16; iSample[7] = iCurrent >> 24; -#ifdef PWR_SAMPLER_BACKLIGHT iSample[8] = (TUint8)iBackLightStatus; iSample[9] = sampleTime; iSample[10] = sampleTime >> 8; @@ -525,14 +520,6 @@ iSample[12] = sampleTime >> 24; iSampler->AddSample(iSample, 13, 0); -#else - iSample[8] = sampleTime; - iSample[9] = sampleTime >> 8; - iSample[10] = sampleTime >> 16; - iSample[11] = sampleTime >> 24; - - iSampler->AddSample(iSample, 12, 0); -#endif LOGTEXT(_L("CProfilerPowerListener::Sample() - exit")); } @@ -570,14 +557,17 @@ LOGSTRING2("CProfilerPowerListener::Stop() - Failed to restore max sampling period: %d", err); } } + if (centRep) + { + delete centRep; + } } -#ifdef PWR_SAMPLER_BACKLIGHT + if (iLightAPI) { delete iLightAPI; iLightAPI = 0; } -#endif LOGTEXT(_L("CProfilerPowerListener::Stop() - exit")); return KErrNone; @@ -604,7 +594,6 @@ LOGTEXT(_L("CProfilerPowerListener::PowerMeasurement - exit")); } -#ifdef PWR_SAMPLER_BACKLIGHT void CProfilerPowerListener::LightStatusChanged(TInt aTarget, CHWRMLight::TLightStatus aStatus) { LOGTEXT(_L("CProfilerPowerListener::LightStatusChanged - entry")); @@ -620,4 +609,3 @@ } LOGTEXT(_L("CProfilerPowerListener::LightStatusChanged - exit")); } -#endif