diff -r 41042d2af523 -r 91123d004e8f basiclocationinfodisplay/blid/ui/src/CBlidBaseView.cpp --- a/basiclocationinfodisplay/blid/ui/src/CBlidBaseView.cpp Fri Mar 12 15:42:12 2010 +0200 +++ b/basiclocationinfodisplay/blid/ui/src/CBlidBaseView.cpp Mon Mar 15 12:40:01 2010 +0200 @@ -179,7 +179,7 @@ } case R_BLID_MAIN_MENU: { - CBlidBaseView::HandleHelpFeature( aMenuPane ); + CBlidBaseView::HandleHelpFeatureL( aMenuPane ); break; } @@ -274,11 +274,11 @@ } // --------------------------------------------------------- -// CBlidBaseView::HandleHelpFeature +// CBlidBaseView::HandleHelpFeatureL // Handles the Help option command // --------------------------------------------------------- // -void CBlidBaseView::HandleHelpFeature( CEikMenuPane* aMenuPane ) const +void CBlidBaseView::HandleHelpFeatureL( CEikMenuPane* aMenuPane ) const { //implementation for req 406-917. Hide position view based on feature variation flag TInt helpSettingValue = ELocHelpNotHidden; @@ -306,7 +306,6 @@ // TBool CBlidBaseView::InsertWaypointL() { - TInt index(KErrCancel); TBuf textData; StringLoader::Load( textData, R_BLID_WAYPOINT_NAME, iCoeEnv ); TInt errorCode(KErrNone); @@ -330,58 +329,41 @@ }while( CheckIfExistL( textData, errorCode, removedIndex ) ); - if(errorCode != KErrNone) - { - return EFalse; - } + if(errorCode != KErrNone) + { + return EFalse; + } - /// if KMaxNumberOfWaypoints or more than that is already present - /// display note. - if ( iRouter->Count() >= KMaxNumberOfWaypoints ) - { - static_cast(AppUi() - )->ShowDialogL( BlidNotes::EBlidOverWriteWaypointNote ); - return EFalse; - } + /// if KMaxNumberOfWaypoints or more than that is already present + /// display note. + if ( iRouter->Count() >= KMaxNumberOfWaypoints ) + { + static_cast(AppUi() + )->ShowDialogL( BlidNotes::EBlidOverWriteWaypointNote ); + return EFalse; + } - TInt retVal(KErrNone); - if ( index >=0 ) - { - if ( iRouter->IndexOfWaypoint() >=0 && - index >= iRouter->IndexOfWaypoint() ) - { - index++; - } - retVal = iRouter->RemoveL( index ); - if(retVal == KErrDiskFull) - { - BlidNotes::OutOfMemoryNoteL(); - } - } - - if(retVal == KErrNone) - { - TNamedCoordinate* waypoint = new(ELeave)TNamedCoordinate(); - CleanupStack::PushL( waypoint ); - waypoint->SetName( textData ); - TPosition position = iLocation->GetCurrentPosition(); - waypoint->SetCoordinate( position.Latitude(), - position.Longitude(), - position.Altitude() ); - position.SetHorizontalAccuracy(position.HorizontalAccuracy()); - waypoint->SetAccuracy(position.HorizontalAccuracy()); - //iRouter takes waypoint's ownership - TInt retVal = iRouter->AppendL( waypoint, ETrue); - if(retVal == KErrDiskFull) - { - delete waypoint; - BlidNotes::OutOfMemoryNoteL(); - returnValue = EFalse; - } - CleanupStack::Pop(); //waypoint - } - return returnValue; - } + TNamedCoordinate* waypoint = new(ELeave)TNamedCoordinate(); + CleanupStack::PushL( waypoint ); + waypoint->SetName( textData ); + TPosition position = iLocation->GetCurrentPosition(); + waypoint->SetCoordinate( position.Latitude(), + position.Longitude(), + position.Altitude() ); + position.SetHorizontalAccuracy(position.HorizontalAccuracy()); + waypoint->SetAccuracy(position.HorizontalAccuracy()); + //iRouter takes waypoint's ownership + TInt retVal = iRouter->AppendL( waypoint, ETrue); + if(retVal == KErrDiskFull) + { + delete waypoint; + BlidNotes::OutOfMemoryNoteL(); + returnValue = EFalse; + } + CleanupStack::Pop(); //waypoint + + return returnValue; + } // ---------------------------------------------------------------------------- // CBlidBaseView::SelectWaypoint