diff mainwindow.cpp @ 284:a68801b31ceb mainwindow_member_tidy

Make MainWindow members private and m_prefixed
author Chris Cannam
date Mon, 21 Feb 2011 09:08:55 +0000
parents 1ec306df738e
children 3fbafca196e4 ca6edd303c9d
line wrap: on
line diff
--- a/mainwindow.cpp	Fri Feb 18 13:39:16 2011 +0000
+++ b/mainwindow.cpp	Mon Feb 21 09:08:55 2011 +0000
@@ -56,39 +56,39 @@
 
     QString wndTitle;
 
-    showAllFiles = false;
+    m_showAllFiles = false;
 
-    fsWatcher = 0;
-    commitsSincePush = 0;
-    shouldHgStat = true;
+    m_fsWatcher = 0;
+    m_commitsSincePush = 0;
+    m_shouldHgStat = true;
 
     createActions();
     createMenus();
     createToolBars();
     createStatusBar();
 
-    runner = new HgRunner(myDirPath, this);
-    connect(runner, SIGNAL(commandStarting(HgAction)),
+    m_runner = new HgRunner(m_myDirPath, this);
+    connect(m_runner, SIGNAL(commandStarting(HgAction)),
             this, SLOT(commandStarting(HgAction)));
-    connect(runner, SIGNAL(commandCompleted(HgAction, QString)),
+    connect(m_runner, SIGNAL(commandCompleted(HgAction, QString)),
             this, SLOT(commandCompleted(HgAction, QString)));
-    connect(runner, SIGNAL(commandFailed(HgAction, QString)),
+    connect(m_runner, SIGNAL(commandFailed(HgAction, QString)),
             this, SLOT(commandFailed(HgAction, QString)));
-    statusBar()->addPermanentWidget(runner);
+    statusBar()->addPermanentWidget(m_runner);
 
     setWindowTitle(tr("EasyMercurial"));
 
-    remoteRepoPath = "";
-    workFolderPath = "";
+    m_remoteRepoPath = "";
+    m_workFolderPath = "";
 
     readSettings();
 
-    justMerged = false;
+    m_justMerged = false;
 
     QWidget *central = new QWidget(this);
     setCentralWidget(central);
 
-    hgTabs = new HgTabWidget(central, remoteRepoPath, workFolderPath);
+    m_hgTabs = new HgTabWidget(central, m_remoteRepoPath, m_workFolderPath);
     connectTabsSignals();
 
     // Instead of setting the tab widget as our central widget
@@ -96,15 +96,15 @@
     // around it on the Mac where it looks very strange without
 
     QGridLayout *cl = new QGridLayout(central);
-    cl->addWidget(hgTabs, 0, 0);
+    cl->addWidget(m_hgTabs, 0, 0);
 
 #ifndef Q_OS_MAC
     cl->setMargin(0);
 #endif
 
-    connect(hgTabs, SIGNAL(selectionChanged()),
+    connect(m_hgTabs, SIGNAL(selectionChanged()),
             this, SLOT(enableDisableActions()));
-    connect(hgTabs, SIGNAL(showAllChanged(bool)),
+    connect(m_hgTabs, SIGNAL(showAllChanged(bool)),
             this, SLOT(showAllChanged(bool)));
 
     setUnifiedTitleAndToolBarOnMac(true);
@@ -112,7 +112,7 @@
     clearState();
     enableDisableActions();
 
-    if (firstStart) {
+    if (m_firstStart) {
         startupDialog();
     }
 
@@ -131,7 +131,7 @@
 void MainWindow::closeEvent(QCloseEvent *)
 {
     writeSettings();
-    delete fsWatcher;
+    delete m_fsWatcher;
 }
 
 
@@ -195,12 +195,12 @@
 
 void MainWindow::clearSelections()
 {
-    hgTabs->clearSelections();
+    m_hgTabs->clearSelections();
 }
 
 void MainWindow::showAllChanged(bool s)
 {
-    showAllFiles = s;
+    m_showAllFiles = s;
     hgQueryPaths();
 }
 
@@ -216,36 +216,36 @@
     //!!! should we test version output? Really we want at least 1.7.x
     //!!! for options such as merge --tool
     params << "--version";
-    runner->requestAction(HgAction(ACT_TEST_HG, m_myDirPath, params));
+    m_runner->requestAction(HgAction(ACT_TEST_HG, m_myDirPath, params));
 }
 
 void MainWindow::hgTestExtension()
 {
     QStringList params;
     params << "--version";
-    runner->requestAction(HgAction(ACT_TEST_HG_EXT, m_myDirPath, params));
+    m_runner->requestAction(HgAction(ACT_TEST_HG_EXT, m_myDirPath, params));
 }
 
 void MainWindow::hgStat()
 {
     QStringList params;
 
-    if (showAllFiles) {
+    if (m_showAllFiles) {
         params << "stat" << "-A";
     } else {
         params << "stat" << "-ardum";
     }
 
-    lastStatOutput = "";
+    m_lastStatOutput = "";
 
-    runner->requestAction(HgAction(ACT_STAT, workFolderPath, params));
+    m_runner->requestAction(HgAction(ACT_STAT, m_workFolderPath, params));
 }
 
 void MainWindow::hgQueryPaths()
 {
     // Quickest is to just read the file
 
-    QFileInfo hgrc(workFolderPath + "/.hg/hgrc");
+    QFileInfo hgrc(m_workFolderPath + "/.hg/hgrc");
 
     QString path;
 
@@ -255,12 +255,12 @@
         path = s.value("default").toString();
     }
 
-    remoteRepoPath = path;
+    m_remoteRepoPath = path;
 
     // We have to do this here, because commandCompleted won't be called
-    MultiChoiceDialog::addRecentArgument("local", workFolderPath);
-    MultiChoiceDialog::addRecentArgument("remote", remoteRepoPath);
-    hgTabs->setWorkFolderAndRepoNames(workFolderPath, remoteRepoPath);
+    MultiChoiceDialog::addRecentArgument("local", m_workFolderPath);
+    MultiChoiceDialog::addRecentArgument("remote", m_remoteRepoPath);
+    m_hgTabs->setWorkFolderAndRepoNames(m_workFolderPath, m_remoteRepoPath);
     
     hgQueryBranch();
     return;
@@ -269,7 +269,7 @@
 
     QStringList params;
     params << "paths";
-    runner->requestAction(HgAction(ACT_QUERY_PATHS, workFolderPath, params));
+    m_runner->requestAction(HgAction(ACT_QUERY_PATHS, m_workFolderPath, params));
 */
 }
 
@@ -277,7 +277,7 @@
 {
     // Quickest is to just read the file
 
-    QFile hgbr(workFolderPath + "/.hg/branch");
+    QFile hgbr(m_workFolderPath + "/.hg/branch");
 
     QString br = "default";
 
@@ -286,7 +286,7 @@
         br = QString::fromUtf8(ba).trimmed();
     }
     
-    currentBranch = br;
+    m_currentBranch = br;
     
     // We have to do this here, because commandCompleted won't be called
     hgStat();
@@ -296,7 +296,7 @@
 
     QStringList params;
     params << "branch";
-    runner->requestAction(HgAction(ACT_QUERY_BRANCH, workFolderPath, params));
+    m_runner->requestAction(HgAction(ACT_QUERY_BRANCH, m_workFolderPath, params));
 */
 }
 
@@ -309,7 +309,7 @@
     // incremental log will end up with spurious stuff in it because
     // we won't be pruning at the ends of closed branches
     params << "heads" << "--closed";
-    runner->requestAction(HgAction(ACT_QUERY_HEADS, workFolderPath, params));
+    m_runner->requestAction(HgAction(ACT_QUERY_HEADS, m_workFolderPath, params));
 }
 
 void MainWindow::hgLog()
@@ -319,7 +319,7 @@
     params << "--template";
     params << Changeset::getLogTemplate();
     
-    runner->requestAction(HgAction(ACT_LOG, workFolderPath, params));
+    m_runner->requestAction(HgAction(ACT_LOG, m_workFolderPath, params));
 }
 
 void MainWindow::hgLogIncremental(QStringList prune)
@@ -334,26 +334,26 @@
     params << "--template";
     params << Changeset::getLogTemplate();
     
-    runner->requestAction(HgAction(ACT_LOG_INCREMENTAL, workFolderPath, params));
+    m_runner->requestAction(HgAction(ACT_LOG_INCREMENTAL, m_workFolderPath, params));
 }
 
 void MainWindow::hgQueryParents()
 {
     QStringList params;
     params << "parents";
-    runner->requestAction(HgAction(ACT_QUERY_PARENTS, workFolderPath, params));
+    m_runner->requestAction(HgAction(ACT_QUERY_PARENTS, m_workFolderPath, params));
 }
 
 void MainWindow::hgAnnotate()
 {
     QStringList params;
-    QString currentFile;//!!! = hgTabs -> getCurrentFileListLine();
+    QString currentFile;//!!! = m_hgTabs -> getCurrentFileListLine();
     
     if (!currentFile.isEmpty())
     {
         params << "annotate" << "--" << currentFile.mid(2);   //Jump over status marker characters (e.g "M ")
 
-        runner->requestAction(HgAction(ACT_ANNOTATE, workFolderPath, params));
+        m_runner->requestAction(HgAction(ACT_ANNOTATE, m_workFolderPath, params));
     }
 }
 
@@ -362,7 +362,7 @@
     QStringList params;
 
     params << "resolve" << "--list";
-    runner->requestAction(HgAction(ACT_RESOLVE_LIST, workFolderPath, params));
+    m_runner->requestAction(HgAction(ACT_RESOLVE_LIST, m_workFolderPath, params));
 }
 
 void MainWindow::hgAdd()
@@ -372,11 +372,11 @@
     // hgExplorer permitted adding "all" files -- I'm not sure
     // that one is a good idea, let's require the user to select
 
-    QStringList files = hgTabs->getSelectedAddableFiles();
+    QStringList files = m_hgTabs->getSelectedAddableFiles();
 
     if (!files.empty()) {
         params << "add" << "--" << files;
-        runner->requestAction(HgAction(ACT_ADD, workFolderPath, params));
+        m_runner->requestAction(HgAction(ACT_ADD, m_workFolderPath, params));
     }
 }
 
@@ -385,11 +385,11 @@
 {
     QStringList params;
 
-    QStringList files = hgTabs->getSelectedRemovableFiles();
+    QStringList files = m_hgTabs->getSelectedRemovableFiles();
 
     if (!files.empty()) {
         params << "remove" << "--after" << "--force" << "--" << files;
-        runner->requestAction(HgAction(ACT_REMOVE, workFolderPath, params));
+        m_runner->requestAction(HgAction(ACT_REMOVE, m_workFolderPath, params));
     }
 }
 
@@ -398,12 +398,12 @@
     QStringList params;
     QString comment;
 
-    if (justMerged) {
-        comment = mergeCommitComment;
+    if (m_justMerged) {
+        comment = m_mergeCommitComment;
     }
 
-    QStringList files = hgTabs->getSelectedCommittableFiles();
-    QStringList allFiles = hgTabs->getAllCommittableFiles();
+    QStringList files = m_hgTabs->getSelectedCommittableFiles();
+    QStringList allFiles = m_hgTabs->getAllCommittableFiles();
     QStringList reportFiles = files;
     if (reportFiles.empty()) {
         reportFiles = allFiles;
@@ -429,7 +429,7 @@
          comment,
          tr("Commit"))) {
 
-        if (!justMerged && !files.empty()) {
+        if (!m_justMerged && !files.empty()) {
             // User wants to commit selected file(s) (and this is not
             // merge commit, which would fail if we selected files)
             params << "commit" << "--message" << comment
@@ -440,8 +440,8 @@
                    << "--user" << getUserInfo();
         }
         
-        runner->requestAction(HgAction(ACT_COMMIT, workFolderPath, params));
-        mergeCommitComment = "";
+        m_runner->requestAction(HgAction(ACT_COMMIT, m_workFolderPath, params));
+        m_mergeCommitComment = "";
     }
 }
 
@@ -478,7 +478,7 @@
             params << "tag" << "--user" << getUserInfo();
             params << "--rev" << Changeset::hashOf(id) << filterTag(tag);
             
-            runner->requestAction(HgAction(ACT_TAG, workFolderPath, params));
+            m_runner->requestAction(HgAction(ACT_TAG, m_workFolderPath, params));
         }
     }
 }
@@ -489,10 +489,10 @@
     QString hgIgnorePath;
     QStringList params;
     
-    hgIgnorePath = workFolderPath;
+    hgIgnorePath = m_workFolderPath;
     hgIgnorePath += "/.hgignore";
 
-    if (!QDir(workFolderPath).exists()) return;
+    if (!QDir(m_workFolderPath).exists()) return;
     QFile f(hgIgnorePath);
     if (!f.exists()) {
         f.open(QFile::WriteOnly);
@@ -512,10 +512,10 @@
         return;
     }
 
-    HgAction action(ACT_HG_IGNORE, workFolderPath, params);
+    HgAction action(ACT_HG_IGNORE, m_workFolderPath, params);
     action.executable = editor;
 
-    runner->requestAction(action);
+    m_runner->requestAction(action);
 }
 
 QString MainWindow::getDiffBinaryName()
@@ -545,7 +545,7 @@
     
     params << "diff" << "--stat";
 
-    runner->requestAction(HgAction(ACT_DIFF_SUMMARY, workFolderPath, params));
+    m_runner->requestAction(HgAction(ACT_DIFF_SUMMARY, m_workFolderPath, params));
 }
 
 void MainWindow::hgFolderDiff()
@@ -560,9 +560,9 @@
     params << "--config" << "extensions.extdiff=" << "extdiff";
     params << "--program" << diff;
 
-    params << hgTabs->getSelectedCommittableFiles(); // may be none: whole dir
+    params << m_hgTabs->getSelectedCommittableFiles(); // may be none: whole dir
 
-    runner->requestAction(HgAction(ACT_FOLDERDIFF, workFolderPath, params));
+    m_runner->requestAction(HgAction(ACT_FOLDERDIFF, m_workFolderPath, params));
 }
 
 
@@ -579,7 +579,7 @@
     params << "--program" << diff;
     params << "--rev" << Changeset::hashOf(id);
 
-    runner->requestAction(HgAction(ACT_FOLDERDIFF, workFolderPath, params));
+    m_runner->requestAction(HgAction(ACT_FOLDERDIFF, m_workFolderPath, params));
 }
 
 
@@ -597,7 +597,7 @@
     params << "--rev" << Changeset::hashOf(parent)
            << "--rev" << Changeset::hashOf(child);
 
-    runner->requestAction(HgAction(ACT_CHGSETDIFF, workFolderPath, params));
+    m_runner->requestAction(HgAction(ACT_CHGSETDIFF, m_workFolderPath, params));
 }
 
 
@@ -607,7 +607,7 @@
 
     params << "update";
     
-    runner->requestAction(HgAction(ACT_UPDATE, workFolderPath, params));
+    m_runner->requestAction(HgAction(ACT_UPDATE, m_workFolderPath, params));
 }
 
 
@@ -617,7 +617,7 @@
 
     params << "update" << "--rev" << Changeset::hashOf(id) << "--check";
 
-    runner->requestAction(HgAction(ACT_UPDATE, workFolderPath, params));
+    m_runner->requestAction(HgAction(ACT_UPDATE, m_workFolderPath, params));
 }
 
 
@@ -627,8 +627,8 @@
     QString comment;
     bool all = false;
 
-    QStringList files = hgTabs->getSelectedRevertableFiles();
-    QStringList allFiles = hgTabs->getAllRevertableFiles();
+    QStringList files = m_hgTabs->getSelectedRevertableFiles();
+    QStringList allFiles = m_hgTabs->getAllRevertableFiles();
     if (files.empty() || files == allFiles) {
         files = allFiles;
         all = true;
@@ -644,9 +644,9 @@
     // Set up params before asking for confirmation, because there is
     // a failure case here that we would need to report on early
 
-    DEBUG << "hgRevert: justMerged = " << justMerged << ", mergeTargetRevision = " << mergeTargetRevision << endl;
+    DEBUG << "hgRevert: m_justMerged = " << m_justMerged << ", m_mergeTargetRevision = " << m_mergeTargetRevision << endl;
 
-    if (justMerged) {
+    if (m_justMerged) {
 
         // This is a little fiddly.  The proper way to "revert" the
         // whole of an uncommitted merge is with "hg update --clean ."
@@ -660,9 +660,9 @@
         if (all) {
             params << "update" << "--clean" << ".";
         } else {
-            if (mergeTargetRevision != "") {
+            if (m_mergeTargetRevision != "") {
                 params << "revert" << "--rev"
-                       << Changeset::hashOf(mergeTargetRevision)
+                       << Changeset::hashOf(m_mergeTargetRevision)
                        << "--" << files;
             } else {
                 QMessageBox::information
@@ -687,9 +687,9 @@
          files,
          tr("Revert"))) {
 
-        lastRevertedFiles = files;
+        m_lastRevertedFiles = files;
         
-        runner->requestAction(HgAction(ACT_REVERT, workFolderPath, params));
+        m_runner->requestAction(HgAction(ACT_REVERT, m_workFolderPath, params));
     }
 }
 
@@ -706,7 +706,7 @@
         params << "--" << files;
     }
 
-    runner->requestAction(HgAction(ACT_RESOLVE_MARK, workFolderPath, params));
+    m_runner->requestAction(HgAction(ACT_RESOLVE_MARK, m_workFolderPath, params));
 }
 
 
@@ -721,26 +721,26 @@
         params << "--tool" << merge;
     }
 
-    QStringList files = hgTabs->getSelectedUnresolvedFiles();
+    QStringList files = m_hgTabs->getSelectedUnresolvedFiles();
     if (files.empty()) {
         params << "--all";
     } else {
         params << "--" << files;
     }
 
-    if (currentParents.size() == 1) {
-        mergeTargetRevision = currentParents[0]->id();
+    if (m_currentParents.size() == 1) {
+        m_mergeTargetRevision = m_currentParents[0]->id();
     }
 
-    runner->requestAction(HgAction(ACT_RETRY_MERGE, workFolderPath, params));
+    m_runner->requestAction(HgAction(ACT_RETRY_MERGE, m_workFolderPath, params));
 
-    mergeCommitComment = tr("Merge");
+    m_mergeCommitComment = tr("Merge");
 }
 
 
 void MainWindow::hgMerge()
 {
-    if (hgTabs->canResolve()) {
+    if (m_hgTabs->canResolve()) {
         hgRetryMerge();
         return;
     }
@@ -754,13 +754,13 @@
         params << "--tool" << merge;
     }
 
-    if (currentParents.size() == 1) {
-        mergeTargetRevision = currentParents[0]->id();
+    if (m_currentParents.size() == 1) {
+        m_mergeTargetRevision = m_currentParents[0]->id();
     }
 
-    runner->requestAction(HgAction(ACT_MERGE, workFolderPath, params));
+    m_runner->requestAction(HgAction(ACT_MERGE, m_workFolderPath, params));
 
-    mergeCommitComment = tr("Merge");
+    m_mergeCommitComment = tr("Merge");
 }
 
 
@@ -776,26 +776,26 @@
         params << "--tool" << merge;
     }
     
-    if (currentParents.size() == 1) {
-        mergeTargetRevision = currentParents[0]->id();
+    if (m_currentParents.size() == 1) {
+        m_mergeTargetRevision = m_currentParents[0]->id();
     }
 
-    runner->requestAction(HgAction(ACT_MERGE, workFolderPath, params));
+    m_runner->requestAction(HgAction(ACT_MERGE, m_workFolderPath, params));
 
-    mergeCommitComment = "";
+    m_mergeCommitComment = "";
 
-    foreach (Changeset *cs, currentHeads) {
-        if (cs->id() == id && !cs->isOnBranch(currentBranch)) {
+    foreach (Changeset *cs, m_currentHeads) {
+        if (cs->id() == id && !cs->isOnBranch(m_currentBranch)) {
             if (cs->branch() == "" || cs->branch() == "default") {
-                mergeCommitComment = tr("Merge from the default branch");
+                m_mergeCommitComment = tr("Merge from the default branch");
             } else {
-                mergeCommitComment = tr("Merge from branch \"%1\"").arg(cs->branch());
+                m_mergeCommitComment = tr("Merge from branch \"%1\"").arg(cs->branch());
             }
         }
     }
 
-    if (mergeCommitComment == "") {
-        mergeCommitComment = tr("Merge from %1").arg(id);
+    if (m_mergeCommitComment == "") {
+        m_mergeCommitComment = tr("Merge from %1").arg(id);
     }
 }
 
@@ -804,21 +804,21 @@
 {
     QStringList params;
 
-    if (!QDir(workFolderPath).exists()) {
-        if (!QDir().mkpath(workFolderPath)) {
+    if (!QDir(m_workFolderPath).exists()) {
+        if (!QDir().mkpath(m_workFolderPath)) {
             DEBUG << "hgCloneFromRemote: Failed to create target path "
-                  << workFolderPath << endl;
+                  << m_workFolderPath << endl;
             //!!! report error
             return;
         }
     }
 
-    params << "clone" << remoteRepoPath << workFolderPath;
+    params << "clone" << m_remoteRepoPath << m_workFolderPath;
     
-    hgTabs->setWorkFolderAndRepoNames(workFolderPath, remoteRepoPath);
-    hgTabs->updateWorkFolderFileList("");
+    m_hgTabs->setWorkFolderAndRepoNames(m_workFolderPath, m_remoteRepoPath);
+    m_hgTabs->updateWorkFolderFileList("");
 
-    runner->requestAction(HgAction(ACT_CLONEFROMREMOTE, workFolderPath, params));
+    m_runner->requestAction(HgAction(ACT_CLONEFROMREMOTE, m_workFolderPath, params));
 }
 
 void MainWindow::hgInit()
@@ -826,19 +826,19 @@
     QStringList params;
 
     params << "init";
-    params << workFolderPath;
+    params << m_workFolderPath;
 
-    runner->requestAction(HgAction(ACT_INIT, workFolderPath, params));
+    m_runner->requestAction(HgAction(ACT_INIT, m_workFolderPath, params));
 }
 
 void MainWindow::hgIncoming()
 {
     QStringList params;
 
-    params << "incoming" << "--newest-first" << remoteRepoPath;
+    params << "incoming" << "--newest-first" << m_remoteRepoPath;
     params << "--template" << Changeset::getLogTemplate();
 
-    runner->requestAction(HgAction(ACT_INCOMING, workFolderPath, params));
+    m_runner->requestAction(HgAction(ACT_INCOMING, m_workFolderPath, params));
 }
 
 void MainWindow::hgPull()
@@ -847,12 +847,12 @@
         (this, tr("Confirm pull"),
          format3(tr("Confirm pull from remote repository"),
                  tr("You are about to pull changes from the following remote repository:"),
-                 remoteRepoPath),
+                 m_remoteRepoPath),
          tr("Pull"))) {
 
         QStringList params;
-        params << "pull" << remoteRepoPath;
-        runner->requestAction(HgAction(ACT_PULL, workFolderPath, params));
+        params << "pull" << m_remoteRepoPath;
+        m_runner->requestAction(HgAction(ACT_PULL, m_workFolderPath, params));
     }
 }
 
@@ -862,12 +862,12 @@
         (this, tr("Confirm push"),
          format3(tr("Confirm push to remote repository"),
                  tr("You are about to push your changes to the following remote repository:"),
-                 remoteRepoPath),
+                 m_remoteRepoPath),
          tr("Push"))) {
 
         QStringList params;
-        params << "push" << "--new-branch" << remoteRepoPath;
-        runner->requestAction(HgAction(ACT_PUSH, workFolderPath, params));
+        params << "push" << "--new-branch" << m_remoteRepoPath;
+        m_runner->requestAction(HgAction(ACT_PUSH, m_workFolderPath, params));
     }
 }
 
@@ -893,24 +893,24 @@
 
 void MainWindow::clearState()
 {
-    foreach (Changeset *cs, currentParents) delete cs;
-    currentParents.clear();
-    foreach (Changeset *cs, currentHeads) delete cs;
-    currentHeads.clear();
-    currentBranch = "";
-    lastStatOutput = "";
-    lastRevertedFiles.clear();
-    mergeTargetRevision = "";
-    mergeCommitComment = "";
-    stateUnknown = true;
-    needNewLog = true;
-    if (fsWatcher) {
+    foreach (Changeset *cs, m_currentParents) delete cs;
+    m_currentParents.clear();
+    foreach (Changeset *cs, m_currentHeads) delete cs;
+    m_currentHeads.clear();
+    m_currentBranch = "";
+    m_lastStatOutput = "";
+    m_lastRevertedFiles.clear();
+    m_mergeTargetRevision = "";
+    m_mergeCommitComment = "";
+    m_stateUnknown = true;
+    m_needNewLog = true;
+    if (m_fsWatcher) {
         delete m_fsWatcherGeneralTimer;
         m_fsWatcherGeneralTimer = 0;
         delete m_fsWatcherRestoreTimer;
         m_fsWatcherRestoreTimer = 0;
-        delete fsWatcher;
-        fsWatcher = 0;
+        delete m_fsWatcher;
+        m_fsWatcher = 0;
     }
 }
 
@@ -940,17 +940,17 @@
              
     params << "serve";
 
-    runner->requestAction(HgAction(ACT_SERVE, workFolderPath, params));
+    m_runner->requestAction(HgAction(ACT_SERVE, m_workFolderPath, params));
     
     QMessageBox::information(this, tr("Serve"), msg, QMessageBox::Close);
 
-    runner->killCurrentActions();
+    m_runner->killCurrentActions();
 }
 
 void MainWindow::startupDialog()
 {
     StartupDialog *dlg = new StartupDialog(this);
-    if (dlg->exec()) firstStart = false;
+    if (dlg->exec()) m_firstStart = false;
 }
 
 void MainWindow::open()
@@ -1028,13 +1028,13 @@
 {
     // This will involve rewriting the local .hgrc
 
-    QDir hgDir(workFolderPath + "/.hg");
+    QDir hgDir(m_workFolderPath + "/.hg");
     if (!hgDir.exists()) {
         //!!! visible error!
         return;
     }
 
-    QFileInfo hgrc(workFolderPath + "/.hg/hgrc");
+    QFileInfo hgrc(m_workFolderPath + "/.hg/hgrc");
     if (hgrc.exists() && !hgrc.isWritable()) {
         //!!! visible error!
         return;
@@ -1063,7 +1063,7 @@
             s.setValue("default", d->getArgument());
         }
 
-        stateUnknown = true;
+        m_stateUnknown = true;
         hgQueryPaths();
     }
 
@@ -1281,8 +1281,8 @@
         return complainAboutFilePath(local);
     }
 
-    workFolderPath = local;
-    remoteRepoPath = "";
+    m_workFolderPath = local;
+    m_remoteRepoPath = "";
     return true;
 }    
 
@@ -1314,8 +1314,8 @@
         if (local == "") return false;
     }
 
-    workFolderPath = local;
-    remoteRepoPath = remote;
+    m_workFolderPath = local;
+    m_remoteRepoPath = remote;
     hgCloneFromRemote();
 
     return true;
@@ -1346,8 +1346,8 @@
         return complainAboutUnknownFolder(local);
     }
 
-    workFolderPath = local;
-    remoteRepoPath = "";
+    m_workFolderPath = local;
+    m_remoteRepoPath = "";
     hgInit();
     return true;
 }
@@ -1358,7 +1358,7 @@
     settingsDlg->exec();
 
     if (settingsDlg->presentationChanged()) {
-        hgTabs->updateFileStates();
+        m_hgTabs->updateFileStates();
         updateToolBarStyle();
         hgRefresh();
     }
@@ -1410,8 +1410,8 @@
 void MainWindow::updateFileSystemWatcher()
 {
     bool justCreated = false;
-    if (!fsWatcher) {
-        fsWatcher = new QFileSystemWatcher();
+    if (!m_fsWatcher) {
+        m_fsWatcher = new QFileSystemWatcher();
         justCreated = true;
     }
 
@@ -1421,18 +1421,18 @@
     // annoying because it would be the normal case for us.  So we'll
     // check for duplicates ourselves.
     QSet<QString> alreadyWatched;
-    QStringList dl(fsWatcher->directories());
+    QStringList dl(m_fsWatcher->directories());
     foreach (QString d, dl) alreadyWatched.insert(d);
     
     std::deque<QString> pending;
-    pending.push_back(workFolderPath);
+    pending.push_back(m_workFolderPath);
 
     while (!pending.empty()) {
 
         QString path = pending.front();
         pending.pop_front();
         if (!alreadyWatched.contains(path)) {
-            fsWatcher->addPath(path);
+            m_fsWatcher->addPath(path);
             DEBUG << "Added to file system watcher: " << path << endl;
         }
 
@@ -1459,9 +1459,9 @@
     m_fsWatcherGeneralTimer->start();
 
     if (justCreated) {
-        connect(fsWatcher, SIGNAL(directoryChanged(QString)),
+        connect(m_fsWatcher, SIGNAL(directoryChanged(QString)),
                 this, SLOT(fsDirectoryChanged(QString)));
-        connect(fsWatcher, SIGNAL(fileChanged(QString)),
+        connect(m_fsWatcher, SIGNAL(fileChanged(QString)),
                 this, SLOT(fsFileChanged(QString)));
     }
 }
@@ -1469,7 +1469,7 @@
 void MainWindow::suspendFileSystemWatcher()
 {
     DEBUG << "MainWindow::suspendFileSystemWatcher" << endl;
-    if (fsWatcher) {
+    if (m_fsWatcher) {
         m_fsWatcherSuspended = true;
         if (m_fsWatcherRestoreTimer) {
             delete m_fsWatcherRestoreTimer;
@@ -1501,7 +1501,7 @@
 void MainWindow::actuallyRestoreFileSystemWatcher()
 {
     DEBUG << "MainWindow::actuallyRestoreFileSystemWatcher" << endl;
-    if (fsWatcher) {
+    if (m_fsWatcher) {
         m_fsWatcherSuspended = false;
         m_fsWatcherGeneralTimer->start();
     }
@@ -1551,7 +1551,7 @@
 
 void MainWindow::showIncoming(QString output)
 {
-    runner->hide();
+    m_runner->hide();
     IncomingDialog *d = new IncomingDialog(this, output);
     d->exec();
     delete d;
@@ -1581,7 +1581,7 @@
         report = tr("Push complete");
     }
     report = format3(report, tr("The push command output was:"), output);
-    runner->hide();
+    m_runner->hide();
     QMessageBox::information(this, "Push complete", report);
 }
 
@@ -1597,7 +1597,7 @@
         report = tr("Pull complete");
     }
     report = format3(report, tr("The pull command output was:"), output);
-    runner->hide();
+    m_runner->hide();
 
     //!!! and something about updating
 
@@ -1608,19 +1608,19 @@
 {
     bool headsAreLocal = false;
 
-    if (currentParents.size() == 1) {
-        int currentBranchHeads = 0;
+    if (m_currentParents.size() == 1) {
+        int m_currentBranchHeads = 0;
         bool parentIsHead = false;
-        Changeset *parent = currentParents[0];
-        foreach (Changeset *head, currentHeads) {
-            if (head->isOnBranch(currentBranch)) {
-                ++currentBranchHeads;
+        Changeset *parent = m_currentParents[0];
+        foreach (Changeset *head, m_currentHeads) {
+            if (head->isOnBranch(m_currentBranch)) {
+                ++m_currentBranchHeads;
             }
             if (parent->id() == head->id()) {
                 parentIsHead = true;
             }
         }
-        if (currentBranchHeads == 2 && parentIsHead) {
+        if (m_currentBranchHeads == 2 && parentIsHead) {
             headsAreLocal = true;
         }
     }
@@ -1689,7 +1689,7 @@
         return;
     case ACT_CLONEFROMREMOTE:
         // if clone fails, we have no repo
-        workFolderPath = "";
+        m_workFolderPath = "";
         enableDisableActions();
         break;
     case ACT_INCOMING:
@@ -1769,23 +1769,23 @@
         LogList ll = lp.parse();
         DEBUG << ll.size() << " results" << endl;
         if (!ll.empty()) {
-            remoteRepoPath = lp.parse()[0]["default"].trimmed();
-            DEBUG << "Set remote path to " << remoteRepoPath << endl;
+            m_remoteRepoPath = lp.parse()[0]["default"].trimmed();
+            DEBUG << "Set remote path to " << m_remoteRepoPath << endl;
         } else {
-            remoteRepoPath = "";
+            m_remoteRepoPath = "";
         }
-        MultiChoiceDialog::addRecentArgument("local", workFolderPath);
-        MultiChoiceDialog::addRecentArgument("remote", remoteRepoPath);
-        hgTabs->setWorkFolderAndRepoNames(workFolderPath, remoteRepoPath);
+        MultiChoiceDialog::addRecentArgument("local", m_workFolderPath);
+        MultiChoiceDialog::addRecentArgument("remote", m_remoteRepoPath);
+        m_hgTabs->setWorkFolderAndRepoNames(m_workFolderPath, m_remoteRepoPath);
         break;
     }
 
     case ACT_QUERY_BRANCH:
-        currentBranch = output.trimmed();
+        m_currentBranch = output.trimmed();
         break;
 
     case ACT_STAT:
-        lastStatOutput = output;
+        m_lastStatOutput = output;
         updateFileSystemWatcher();
         break;
 
@@ -1800,13 +1800,13 @@
             }
             output = winnowed.join("\n");
         }
-        DEBUG << "lastStatOutput = " << lastStatOutput << endl;
+        DEBUG << "m_lastStatOutput = " << m_lastStatOutput << endl;
         DEBUG << "resolve output = " << output << endl;
-        hgTabs->updateWorkFolderFileList(lastStatOutput + output);
+        m_hgTabs->updateWorkFolderFileList(m_lastStatOutput + output);
         break;
 
     case ACT_RESOLVE_MARK:
-        shouldHgStat = true;
+        m_shouldHgStat = true;
         break;
         
     case ACT_INCOMING:
@@ -1815,12 +1815,12 @@
 
     case ACT_ANNOTATE:
         presentLongStdoutToUser(output);
-        shouldHgStat = true;
+        m_shouldHgStat = true;
         break;
         
     case ACT_PULL:
         showPullResult(output);
-        shouldHgStat = true;
+        m_shouldHgStat = true;
         break;
         
     case ACT_PUSH:
@@ -1828,73 +1828,73 @@
         break;
         
     case ACT_INIT:
-        MultiChoiceDialog::addRecentArgument("init", workFolderPath);
-        MultiChoiceDialog::addRecentArgument("local", workFolderPath);
+        MultiChoiceDialog::addRecentArgument("init", m_workFolderPath);
+        MultiChoiceDialog::addRecentArgument("local", m_workFolderPath);
         enableDisableActions();
-        shouldHgStat = true;
+        m_shouldHgStat = true;
         break;
         
     case ACT_CLONEFROMREMOTE:
-        MultiChoiceDialog::addRecentArgument("local", workFolderPath);
-        MultiChoiceDialog::addRecentArgument("remote", remoteRepoPath);
-        MultiChoiceDialog::addRecentArgument("remote", workFolderPath, true);
+        MultiChoiceDialog::addRecentArgument("local", m_workFolderPath);
+        MultiChoiceDialog::addRecentArgument("remote", m_remoteRepoPath);
+        MultiChoiceDialog::addRecentArgument("remote", m_workFolderPath, true);
         QMessageBox::information(this, tr("Clone"), tr("<qt><h3>Clone successful</h3><pre>%1</pre>").arg(xmlEncode(output)));
         enableDisableActions();
-        shouldHgStat = true;
+        m_shouldHgStat = true;
         break;
         
     case ACT_LOG:
-        hgTabs->setNewLog(output);
-        needNewLog = false;
+        m_hgTabs->setNewLog(output);
+        m_needNewLog = false;
         break;
         
     case ACT_LOG_INCREMENTAL:
-        hgTabs->addIncrementalLog(output);
+        m_hgTabs->addIncrementalLog(output);
         break;
         
     case ACT_QUERY_PARENTS:
     {
-        foreach (Changeset *cs, currentParents) delete cs;
-        currentParents = Changeset::parseChangesets(output);
-        QStringList parentIds = Changeset::getIds(currentParents);
-        hgTabs->setCurrent(parentIds, currentBranch);
+        foreach (Changeset *cs, m_currentParents) delete cs;
+        m_currentParents = Changeset::parseChangesets(output);
+        QStringList parentIds = Changeset::getIds(m_currentParents);
+        m_hgTabs->setCurrent(parentIds, m_currentBranch);
     }
         break;
         
     case ACT_QUERY_HEADS:
     {
-        oldHeadIds = Changeset::getIds(currentHeads);
+        oldHeadIds = Changeset::getIds(m_currentHeads);
         Changesets newHeads = Changeset::parseChangesets(output);
         QStringList newHeadIds = Changeset::getIds(newHeads);
         if (oldHeadIds != newHeadIds) {
             DEBUG << "Heads changed, will prompt an incremental log if appropriate" << endl;
             headsChanged = true;
-            foreach (Changeset *cs, currentHeads) delete cs;
-            currentHeads = newHeads;
+            foreach (Changeset *cs, m_currentHeads) delete cs;
+            m_currentHeads = newHeads;
         }
     }
         break;
 
     case ACT_COMMIT:
-        hgTabs->clearSelections();
-        justMerged = false;
-        shouldHgStat = true;
+        m_hgTabs->clearSelections();
+        m_justMerged = false;
+        m_shouldHgStat = true;
         break;
 
     case ACT_REVERT:
-        hgMarkResolved(lastRevertedFiles);
-        justMerged = false;
+        hgMarkResolved(m_lastRevertedFiles);
+        m_justMerged = false;
         break;
         
     case ACT_REMOVE:
     case ACT_ADD:
-        hgTabs->clearSelections();
-        shouldHgStat = true;
+        m_hgTabs->clearSelections();
+        m_shouldHgStat = true;
         break;
 
     case ACT_TAG:
-        needNewLog = true;
-        shouldHgStat = true;
+        m_needNewLog = true;
+        m_shouldHgStat = true;
         break;
 
     case ACT_DIFF_SUMMARY:
@@ -1908,26 +1908,26 @@
     case ACT_CHGSETDIFF:
     case ACT_SERVE:
     case ACT_HG_IGNORE:
-        shouldHgStat = true;
+        m_shouldHgStat = true;
         break;
         
     case ACT_UPDATE:
         QMessageBox::information(this, tr("Update"), tr("<qt><h3>Update successful</h3><p>%1</p>").arg(xmlEncode(output)));
-        shouldHgStat = true;
+        m_shouldHgStat = true;
         break;
         
     case ACT_MERGE:
         //!!! use format3?
         QMessageBox::information(this, tr("Merge"), tr("<qt><h3>Merge successful</h3><pre>%1</pre>").arg(xmlEncode(output)));
-        shouldHgStat = true;
-        justMerged = true;
+        m_shouldHgStat = true;
+        m_justMerged = true;
         break;
         
     case ACT_RETRY_MERGE:
         QMessageBox::information(this, tr("Resolved"),
                                  tr("<qt><h3>Merge resolved</h3><p>Merge resolved successfully.</p>"));
-        shouldHgStat = true;
-        justMerged = true;
+        m_shouldHgStat = true;
+        m_justMerged = true;
         break;
         
     default:
@@ -1954,7 +1954,7 @@
         settings.beginGroup("General");
         if (settings.value("useextension", true).toBool()) {
             hgTestExtension();
-        } else if (workFolderPath == "") {
+        } else if (m_workFolderPath == "") {
             open();
         } else {
             hgQueryPaths();
@@ -1963,7 +1963,7 @@
     }
         
     case ACT_TEST_HG_EXT:
-        if (workFolderPath == "") {
+        if (m_workFolderPath == "") {
             open();
         } else{
             hgQueryPaths();
@@ -1987,7 +1987,7 @@
         break;
 
     case ACT_QUERY_HEADS:
-        if (headsChanged && !needNewLog) {
+        if (headsChanged && !m_needNewLog) {
             hgLogIncremental(oldHeadIds);
         } else {
             hgQueryParents();
@@ -1999,7 +1999,7 @@
         break;
 
     case ACT_QUERY_PARENTS:
-        if (needNewLog) {
+        if (m_needNewLog) {
             hgLog();
         } else {
             // we're done
@@ -2013,8 +2013,8 @@
         break;
 
     default:
-        if (shouldHgStat) {
-            shouldHgStat = false;
+        if (m_shouldHgStat) {
+            m_shouldHgStat = false;
             hgQueryPaths();
         } else {
             noMore = true;
@@ -2023,66 +2023,66 @@
     }
 
     if (noMore) {
-        stateUnknown = false;
+        m_stateUnknown = false;
         enableDisableActions();
-        hgTabs->updateHistory();
+        m_hgTabs->updateHistory();
     }
 }
 
 void MainWindow::connectActions()
 {
-    connect(exitAct, SIGNAL(triggered()), this, SLOT(close()));
-    connect(aboutAct, SIGNAL(triggered()), this, SLOT(about()));
+    connect(m_exitAct, SIGNAL(triggered()), this, SLOT(close()));
+    connect(m_aboutAct, SIGNAL(triggered()), this, SLOT(about()));
 
-    connect(hgRefreshAct, SIGNAL(triggered()), this, SLOT(hgRefresh()));
-    connect(hgRemoveAct, SIGNAL(triggered()), this, SLOT(hgRemove()));
-    connect(hgAddAct, SIGNAL(triggered()), this, SLOT(hgAdd()));
-    connect(hgCommitAct, SIGNAL(triggered()), this, SLOT(hgCommit()));
-    connect(hgFolderDiffAct, SIGNAL(triggered()), this, SLOT(hgFolderDiff()));
-    connect(hgUpdateAct, SIGNAL(triggered()), this, SLOT(hgUpdate()));
-    connect(hgRevertAct, SIGNAL(triggered()), this, SLOT(hgRevert()));
-    connect(hgMergeAct, SIGNAL(triggered()), this, SLOT(hgMerge()));
-    connect(hgIgnoreAct, SIGNAL(triggered()), this, SLOT(hgIgnore()));
+    connect(m_hgRefreshAct, SIGNAL(triggered()), this, SLOT(hgRefresh()));
+    connect(m_hgRemoveAct, SIGNAL(triggered()), this, SLOT(hgRemove()));
+    connect(m_hgAddAct, SIGNAL(triggered()), this, SLOT(hgAdd()));
+    connect(m_hgCommitAct, SIGNAL(triggered()), this, SLOT(hgCommit()));
+    connect(m_hgFolderDiffAct, SIGNAL(triggered()), this, SLOT(hgFolderDiff()));
+    connect(m_hgUpdateAct, SIGNAL(triggered()), this, SLOT(hgUpdate()));
+    connect(m_hgRevertAct, SIGNAL(triggered()), this, SLOT(hgRevert()));
+    connect(m_hgMergeAct, SIGNAL(triggered()), this, SLOT(hgMerge()));
+    connect(m_hgIgnoreAct, SIGNAL(triggered()), this, SLOT(hgIgnore()));
 
-    connect(settingsAct, SIGNAL(triggered()), this, SLOT(settings()));
-    connect(openAct, SIGNAL(triggered()), this, SLOT(open()));
-    connect(changeRemoteRepoAct, SIGNAL(triggered()), this, SLOT(changeRemoteRepo()));
+    connect(m_settingsAct, SIGNAL(triggered()), this, SLOT(settings()));
+    connect(m_openAct, SIGNAL(triggered()), this, SLOT(open()));
+    connect(m_changeRemoteRepoAct, SIGNAL(triggered()), this, SLOT(changeRemoteRepo()));
 
-    connect(hgIncomingAct, SIGNAL(triggered()), this, SLOT(hgIncoming()));
-    connect(hgPullAct, SIGNAL(triggered()), this, SLOT(hgPull()));
-    connect(hgPushAct, SIGNAL(triggered()), this, SLOT(hgPush()));
+    connect(m_hgIncomingAct, SIGNAL(triggered()), this, SLOT(hgIncoming()));
+    connect(m_hgPullAct, SIGNAL(triggered()), this, SLOT(hgPull()));
+    connect(m_hgPushAct, SIGNAL(triggered()), this, SLOT(hgPush()));
 
-    connect(hgAnnotateAct, SIGNAL(triggered()), this, SLOT(hgAnnotate()));
-    connect(hgServeAct, SIGNAL(triggered()), this, SLOT(hgServe()));
+    connect(m_hgAnnotateAct, SIGNAL(triggered()), this, SLOT(hgAnnotate()));
+    connect(m_hgServeAct, SIGNAL(triggered()), this, SLOT(hgServe()));
 }
 
 void MainWindow::connectTabsSignals()
 {
-    connect(hgTabs, SIGNAL(commit()),
+    connect(m_hgTabs, SIGNAL(commit()),
             this, SLOT(hgCommit()));
     
-    connect(hgTabs, SIGNAL(revert()),
+    connect(m_hgTabs, SIGNAL(revert()),
             this, SLOT(hgRevert()));
     
-    connect(hgTabs, SIGNAL(diffWorkingFolder()),
+    connect(m_hgTabs, SIGNAL(diffWorkingFolder()),
             this, SLOT(hgFolderDiff()));
     
-    connect(hgTabs, SIGNAL(showSummary()),
+    connect(m_hgTabs, SIGNAL(showSummary()),
             this, SLOT(hgShowSummary()));
 
-    connect(hgTabs, SIGNAL(updateTo(QString)),
+    connect(m_hgTabs, SIGNAL(updateTo(QString)),
             this, SLOT(hgUpdateToRev(QString)));
 
-    connect(hgTabs, SIGNAL(diffToCurrent(QString)),
+    connect(m_hgTabs, SIGNAL(diffToCurrent(QString)),
             this, SLOT(hgDiffToCurrent(QString)));
 
-    connect(hgTabs, SIGNAL(diffToParent(QString, QString)),
+    connect(m_hgTabs, SIGNAL(diffToParent(QString, QString)),
             this, SLOT(hgDiffToParent(QString, QString)));
 
-    connect(hgTabs, SIGNAL(mergeFrom(QString)),
+    connect(m_hgTabs, SIGNAL(mergeFrom(QString)),
             this, SLOT(hgMergeFrom(QString)));
 
-    connect(hgTabs, SIGNAL(tag(QString)),
+    connect(m_hgTabs, SIGNAL(tag(QString)),
             this, SLOT(hgTag(QString)));
 }    
 
@@ -2090,7 +2090,7 @@
 {
     DEBUG << "MainWindow::enableDisableActions" << endl;
 
-    QString dirname = QDir(workFolderPath).dirName();
+    QString dirname = QDir(m_workFolderPath).dirName();
     if (dirname != "") {
         setWindowTitle(tr("EasyMercurial: %1").arg(dirname));
     } else {
@@ -2105,32 +2105,32 @@
     bool workFolderExist = true;
     bool localRepoExist = true;
 
-    remoteRepoActionsEnabled = true;
-    if (remoteRepoPath.isEmpty()) {
-        remoteRepoActionsEnabled = false;
+    m_remoteRepoActionsEnabled = true;
+    if (m_remoteRepoPath.isEmpty()) {
+        m_remoteRepoActionsEnabled = false;
     }
 
-    localRepoActionsEnabled = true;
-    if (workFolderPath.isEmpty()) {
-        localRepoActionsEnabled = false;
+    m_localRepoActionsEnabled = true;
+    if (m_workFolderPath.isEmpty()) {
+        m_localRepoActionsEnabled = false;
         workFolderExist = false;
     }
 
-    if (workFolderPath == "" || !workFolderDir.exists(workFolderPath)) {
-        localRepoActionsEnabled = false;
+    if (m_workFolderPath == "" || !workFolderDir.exists(m_workFolderPath)) {
+        m_localRepoActionsEnabled = false;
         workFolderExist = false;
     } else {
         workFolderExist = true;
     }
 
-    if (!localRepoDir.exists(workFolderPath + "/.hg")) {
-        localRepoActionsEnabled = false;
+    if (!localRepoDir.exists(m_workFolderPath + "/.hg")) {
+        m_localRepoActionsEnabled = false;
         localRepoExist = false;
     }
 
-    hgIncomingAct -> setEnabled(remoteRepoActionsEnabled && remoteRepoActionsEnabled);
-    hgPullAct -> setEnabled(remoteRepoActionsEnabled && remoteRepoActionsEnabled);
-    hgPushAct -> setEnabled(remoteRepoActionsEnabled && remoteRepoActionsEnabled);
+    m_hgIncomingAct -> setEnabled(m_remoteRepoActionsEnabled && m_remoteRepoActionsEnabled);
+    m_hgPullAct -> setEnabled(m_remoteRepoActionsEnabled && m_remoteRepoActionsEnabled);
+    m_hgPushAct -> setEnabled(m_remoteRepoActionsEnabled && m_remoteRepoActionsEnabled);
 
     bool haveDiff = false;
     QSettings settings;
@@ -2140,26 +2140,26 @@
     }
     settings.endGroup();
 
-    hgRefreshAct -> setEnabled(localRepoActionsEnabled);
-    hgFolderDiffAct -> setEnabled(localRepoActionsEnabled && haveDiff);
-    hgRevertAct -> setEnabled(localRepoActionsEnabled);
-    hgAddAct -> setEnabled(localRepoActionsEnabled);
-    hgRemoveAct -> setEnabled(localRepoActionsEnabled);
-    hgUpdateAct -> setEnabled(localRepoActionsEnabled);
-    hgCommitAct -> setEnabled(localRepoActionsEnabled);
-    hgMergeAct -> setEnabled(localRepoActionsEnabled);
-    hgAnnotateAct -> setEnabled(localRepoActionsEnabled);
-    hgServeAct -> setEnabled(localRepoActionsEnabled);
-    hgIgnoreAct -> setEnabled(localRepoActionsEnabled);
+    m_hgRefreshAct -> setEnabled(m_localRepoActionsEnabled);
+    m_hgFolderDiffAct -> setEnabled(m_localRepoActionsEnabled && haveDiff);
+    m_hgRevertAct -> setEnabled(m_localRepoActionsEnabled);
+    m_hgAddAct -> setEnabled(m_localRepoActionsEnabled);
+    m_hgRemoveAct -> setEnabled(m_localRepoActionsEnabled);
+    m_hgUpdateAct -> setEnabled(m_localRepoActionsEnabled);
+    m_hgCommitAct -> setEnabled(m_localRepoActionsEnabled);
+    m_hgMergeAct -> setEnabled(m_localRepoActionsEnabled);
+    m_hgAnnotateAct -> setEnabled(m_localRepoActionsEnabled);
+    m_hgServeAct -> setEnabled(m_localRepoActionsEnabled);
+    m_hgIgnoreAct -> setEnabled(m_localRepoActionsEnabled);
 
-    DEBUG << "localRepoActionsEnabled = " << localRepoActionsEnabled << endl;
-    DEBUG << "canCommit = " << hgTabs->canCommit() << endl;
+    DEBUG << "m_localRepoActionsEnabled = " << m_localRepoActionsEnabled << endl;
+    DEBUG << "canCommit = " << m_hgTabs->canCommit() << endl;
 
-    hgAddAct->setEnabled(localRepoActionsEnabled && hgTabs->canAdd());
-    hgRemoveAct->setEnabled(localRepoActionsEnabled && hgTabs->canRemove());
-    hgCommitAct->setEnabled(localRepoActionsEnabled && hgTabs->canCommit());
-    hgRevertAct->setEnabled(localRepoActionsEnabled && hgTabs->canRevert());
-    hgFolderDiffAct->setEnabled(localRepoActionsEnabled && hgTabs->canDiff());
+    m_hgAddAct->setEnabled(m_localRepoActionsEnabled && m_hgTabs->canAdd());
+    m_hgRemoveAct->setEnabled(m_localRepoActionsEnabled && m_hgTabs->canRemove());
+    m_hgCommitAct->setEnabled(m_localRepoActionsEnabled && m_hgTabs->canCommit());
+    m_hgRevertAct->setEnabled(m_localRepoActionsEnabled && m_hgTabs->canRevert());
+    m_hgFolderDiffAct->setEnabled(m_localRepoActionsEnabled && m_hgTabs->canDiff());
 
     // A default merge makes sense if:
     //  * there is only one parent (if there are two, we have an uncommitted merge) and
@@ -2176,24 +2176,24 @@
     bool emptyRepo = false;
     bool noWorkingCopy = false;
     bool newBranch = false;
-    int currentBranchHeads = 0;
+    int m_currentBranchHeads = 0;
 
-    if (currentParents.size() == 1) {
+    if (m_currentParents.size() == 1) {
         bool parentIsHead = false;
-        Changeset *parent = currentParents[0];
-        foreach (Changeset *head, currentHeads) {
-            DEBUG << "head branch " << head->branch() << ", current branch " << currentBranch << endl;
-            if (head->isOnBranch(currentBranch)) {
-                ++currentBranchHeads;
+        Changeset *parent = m_currentParents[0];
+        foreach (Changeset *head, m_currentHeads) {
+            DEBUG << "head branch " << head->branch() << ", current branch " << m_currentBranch << endl;
+            if (head->isOnBranch(m_currentBranch)) {
+                ++m_currentBranchHeads;
             }
             if (parent->id() == head->id()) {
                 parentIsHead = true;
             }
         }
-        if (currentBranchHeads == 2 && parentIsHead) {
+        if (m_currentBranchHeads == 2 && parentIsHead) {
             canMerge = true;
         }
-        if (currentBranchHeads == 0 && parentIsHead) {
+        if (m_currentBranchHeads == 0 && parentIsHead) {
             // Just created a new branch
             newBranch = true;
         }
@@ -2201,13 +2201,13 @@
             canUpdate = true;
             DEBUG << "parent id = " << parent->id() << endl;
             DEBUG << " head ids "<<endl;
-            foreach (Changeset *h, currentHeads) {
+            foreach (Changeset *h, m_currentHeads) {
                 DEBUG << "head id = " << h->id() << endl;
             }
         }
-        justMerged = false;
-    } else if (currentParents.size() == 0) {
-        if (currentHeads.size() == 0) {
+        m_justMerged = false;
+    } else if (m_currentParents.size() == 0) {
+        if (m_currentHeads.size() == 0) {
             // No heads -> empty repo
             emptyRepo = true;
         } else {
@@ -2217,125 +2217,125 @@
             noWorkingCopy = true;
             canUpdate = true;
         }
-        justMerged = false;
+        m_justMerged = false;
     } else {
         haveMerge = true;
-        justMerged = true;
+        m_justMerged = true;
     }
         
-    hgMergeAct->setEnabled(localRepoActionsEnabled &&
-                           (canMerge || hgTabs->canResolve()));
-    hgUpdateAct->setEnabled(localRepoActionsEnabled &&
-                            (canUpdate && !hgTabs->haveChangesToCommit()));
+    m_hgMergeAct->setEnabled(m_localRepoActionsEnabled &&
+                           (canMerge || m_hgTabs->canResolve()));
+    m_hgUpdateAct->setEnabled(m_localRepoActionsEnabled &&
+                            (canUpdate && !m_hgTabs->haveChangesToCommit()));
 
     // Set the state field on the file status widget
 
     QString branchText;
-    if (currentBranch == "" || currentBranch == "default") {
+    if (m_currentBranch == "" || m_currentBranch == "default") {
         branchText = tr("the default branch");
     } else {
-        branchText = tr("branch \"%1\"").arg(currentBranch);
+        branchText = tr("branch \"%1\"").arg(m_currentBranch);
     }
 
-    if (stateUnknown) {
-        if (workFolderPath == "") {
-            hgTabs->setState(tr("No repository open"));
+    if (m_stateUnknown) {
+        if (m_workFolderPath == "") {
+            m_hgTabs->setState(tr("No repository open"));
         } else {
-            hgTabs->setState(tr("(Examining repository)"));
+            m_hgTabs->setState(tr("(Examining repository)"));
         }
     } else if (emptyRepo) {
-        hgTabs->setState(tr("Nothing committed to this repository yet"));
+        m_hgTabs->setState(tr("Nothing committed to this repository yet"));
     } else if (noWorkingCopy) {
-        hgTabs->setState(tr("No working copy yet: consider updating"));
+        m_hgTabs->setState(tr("No working copy yet: consider updating"));
     } else if (canMerge) {
-        hgTabs->setState(tr("<b>Awaiting merge</b> on %1").arg(branchText));
-    } else if (!hgTabs->getAllUnresolvedFiles().empty()) {
-        hgTabs->setState(tr("Have unresolved files following merge on %1").arg(branchText));
+        m_hgTabs->setState(tr("<b>Awaiting merge</b> on %1").arg(branchText));
+    } else if (!m_hgTabs->getAllUnresolvedFiles().empty()) {
+        m_hgTabs->setState(tr("Have unresolved files following merge on %1").arg(branchText));
     } else if (haveMerge) {
-        hgTabs->setState(tr("Have merged but not yet committed on %1").arg(branchText));
+        m_hgTabs->setState(tr("Have merged but not yet committed on %1").arg(branchText));
     } else if (newBranch) {
-        hgTabs->setState(tr("On %1.  New branch: has not yet been committed").arg(branchText));
+        m_hgTabs->setState(tr("On %1.  New branch: has not yet been committed").arg(branchText));
     } else if (canUpdate) {
-        if (hgTabs->haveChangesToCommit()) {
+        if (m_hgTabs->haveChangesToCommit()) {
             // have uncommitted changes
-            hgTabs->setState(tr("On %1. Not at the head of the branch").arg(branchText));
+            m_hgTabs->setState(tr("On %1. Not at the head of the branch").arg(branchText));
         } else {
             // no uncommitted changes
-            hgTabs->setState(tr("On %1. Not at the head of the branch: consider updating").arg(branchText));
+            m_hgTabs->setState(tr("On %1. Not at the head of the branch: consider updating").arg(branchText));
         }
-    } else if (currentBranchHeads > 1) {
-        hgTabs->setState(tr("At one of %n heads of %1", "", currentBranchHeads).arg(branchText));
+    } else if (m_currentBranchHeads > 1) {
+        m_hgTabs->setState(tr("At one of %n heads of %1", "", m_currentBranchHeads).arg(branchText));
     } else {
-        hgTabs->setState(tr("At the head of %1").arg(branchText));
+        m_hgTabs->setState(tr("At the head of %1").arg(branchText));
     }
 }
 
 void MainWindow::createActions()
 {
     //File actions
-    openAct = new QAction(QIcon(":/images/fileopen.png"), tr("Open..."), this);
-    openAct -> setStatusTip(tr("Open an existing repository or working folder"));
+    m_openAct = new QAction(QIcon(":/images/fileopen.png"), tr("Open..."), this);
+    m_openAct -> setStatusTip(tr("Open an existing repository or working folder"));
 
-    changeRemoteRepoAct = new QAction(tr("Change Remote Location..."), this);
-    changeRemoteRepoAct->setStatusTip(tr("Change the default remote repository for pull and push actions"));
+    m_changeRemoteRepoAct = new QAction(tr("Change Remote Location..."), this);
+    m_changeRemoteRepoAct->setStatusTip(tr("Change the default remote repository for pull and push actions"));
 
-    settingsAct = new QAction(QIcon(":/images/settings.png"), tr("Settings..."), this);
-    settingsAct -> setStatusTip(tr("View and change application settings"));
+    m_settingsAct = new QAction(QIcon(":/images/settings.png"), tr("Settings..."), this);
+    m_settingsAct -> setStatusTip(tr("View and change application settings"));
 
-    exitAct = new QAction(QIcon(":/images/exit.png"), tr("Quit"), this);
-    exitAct->setShortcuts(QKeySequence::Quit);
-    exitAct->setStatusTip(tr("Quit EasyMercurial"));
+    m_exitAct = new QAction(QIcon(":/images/exit.png"), tr("Quit"), this);
+    m_exitAct->setShortcuts(QKeySequence::Quit);
+    m_exitAct->setStatusTip(tr("Quit EasyMercurial"));
 
     //Repository actions
-    hgRefreshAct = new QAction(QIcon(":/images/status.png"), tr("Refresh"), this);
-    hgRefreshAct->setStatusTip(tr("Refresh the window to show the current state of the working folder"));
+    m_hgRefreshAct = new QAction(QIcon(":/images/status.png"), tr("Refresh"), this);
+    m_hgRefreshAct->setStatusTip(tr("Refresh the window to show the current state of the working folder"));
 
-    hgIncomingAct = new QAction(QIcon(":/images/incoming.png"), tr("Preview"), this);
-    hgIncomingAct -> setStatusTip(tr("See what changes are available in the remote repository waiting to be pulled"));
+    m_hgIncomingAct = new QAction(QIcon(":/images/incoming.png"), tr("Preview"), this);
+    m_hgIncomingAct -> setStatusTip(tr("See what changes are available in the remote repository waiting to be pulled"));
 
-    hgPullAct = new QAction(QIcon(":/images/pull.png"), tr("Pull"), this);
-    hgPullAct -> setStatusTip(tr("Pull changes from the remote repository to the local repository"));
+    m_hgPullAct = new QAction(QIcon(":/images/pull.png"), tr("Pull"), this);
+    m_hgPullAct -> setStatusTip(tr("Pull changes from the remote repository to the local repository"));
 
-    hgPushAct = new QAction(QIcon(":/images/push.png"), tr("Push"), this);
-    hgPushAct->setStatusTip(tr("Push changes from the local repository to the remote repository"));
+    m_hgPushAct = new QAction(QIcon(":/images/push.png"), tr("Push"), this);
+    m_hgPushAct->setStatusTip(tr("Push changes from the local repository to the remote repository"));
 
     //Workfolder actions
-    hgFolderDiffAct   = new QAction(QIcon(":/images/folderdiff.png"), tr("Diff"), this);
-    hgFolderDiffAct->setStatusTip(tr("See what has changed in the working folder compared with the last committed state"));
+    m_hgFolderDiffAct   = new QAction(QIcon(":/images/folderdiff.png"), tr("Diff"), this);
+    m_hgFolderDiffAct->setStatusTip(tr("See what has changed in the working folder compared with the last committed state"));
 
-    hgRevertAct = new QAction(QIcon(":/images/undo.png"), tr("Revert"), this);
-    hgRevertAct->setStatusTip(tr("Throw away your changes and return to the last committed state"));
+    m_hgRevertAct = new QAction(QIcon(":/images/undo.png"), tr("Revert"), this);
+    m_hgRevertAct->setStatusTip(tr("Throw away your changes and return to the last committed state"));
 
-    hgAddAct = new QAction(QIcon(":/images/add.png"), tr("Add"), this);
-    hgAddAct -> setStatusTip(tr("Mark the selected file(s) to be added on the next commit"));
+    m_hgAddAct = new QAction(QIcon(":/images/add.png"), tr("Add"), this);
+    m_hgAddAct -> setStatusTip(tr("Mark the selected file(s) to be added on the next commit"));
 
     //!!! needs to be modified for number
-    hgRemoveAct = new QAction(QIcon(":/images/remove.png"), tr("Remove"), this);
-    hgRemoveAct -> setStatusTip(tr("Mark the selected file(s) to be removed from version control on the next commit"));
+    m_hgRemoveAct = new QAction(QIcon(":/images/remove.png"), tr("Remove"), this);
+    m_hgRemoveAct -> setStatusTip(tr("Mark the selected file(s) to be removed from version control on the next commit"));
 
-    hgUpdateAct = new QAction(QIcon(":/images/update.png"), tr("Update"), this);
-    hgUpdateAct->setStatusTip(tr("Update the working folder to the head of the current repository branch"));
+    m_hgUpdateAct = new QAction(QIcon(":/images/update.png"), tr("Update"), this);
+    m_hgUpdateAct->setStatusTip(tr("Update the working folder to the head of the current repository branch"));
 
     //!!! needs to be modified when files selected
-    hgCommitAct = new QAction(QIcon(":/images/commit.png"), tr("Commit"), this);
-    hgCommitAct->setStatusTip(tr("Commit your changes to the local repository"));
+    m_hgCommitAct = new QAction(QIcon(":/images/commit.png"), tr("Commit"), this);
+    m_hgCommitAct->setStatusTip(tr("Commit your changes to the local repository"));
 
-    hgMergeAct = new QAction(QIcon(":/images/merge.png"), tr("Merge"), this);
-    hgMergeAct->setStatusTip(tr("Merge the two independent sets of changes in the local repository into the working folder"));
+    m_hgMergeAct = new QAction(QIcon(":/images/merge.png"), tr("Merge"), this);
+    m_hgMergeAct->setStatusTip(tr("Merge the two independent sets of changes in the local repository into the working folder"));
 
     //Advanced actions
     //!!! needs to be modified for number
-    hgAnnotateAct = new QAction(tr("Annotate"), this);
-    hgAnnotateAct -> setStatusTip(tr("Show line-by-line version information for selected file"));
+    m_hgAnnotateAct = new QAction(tr("Annotate"), this);
+    m_hgAnnotateAct -> setStatusTip(tr("Show line-by-line version information for selected file"));
 
-    hgIgnoreAct = new QAction(tr("Edit .hgignore File"), this);
-    hgIgnoreAct -> setStatusTip(tr("Edit the .hgignore file, containing the names of files that should be ignored by Mercurial"));
+    m_hgIgnoreAct = new QAction(tr("Edit .hgignore File"), this);
+    m_hgIgnoreAct -> setStatusTip(tr("Edit the .hgignore file, containing the names of files that should be ignored by Mercurial"));
 
-    hgServeAct = new QAction(tr("Serve via HTTP"), this);
-    hgServeAct -> setStatusTip(tr("Serve local repository via http for workgroup access"));
+    m_hgServeAct = new QAction(tr("Serve via HTTP"), this);
+    m_hgServeAct -> setStatusTip(tr("Serve local repository via http for workgroup access"));
 
     //Help actions
-    aboutAct = new QAction(tr("About EasyMercurial"), this);
+    m_aboutAct = new QAction(tr("About EasyMercurial"), this);
 
     // Miscellaneous
     QShortcut *clearSelectionsShortcut = new QShortcut(Qt::Key_Escape, this);
@@ -2345,56 +2345,56 @@
 
 void MainWindow::createMenus()
 {
-    fileMenu = menuBar()->addMenu(tr("File"));
+    m_fileMenu = menuBar()->addMenu(tr("File"));
 
-    fileMenu -> addAction(openAct);
-    fileMenu -> addAction(changeRemoteRepoAct);
-    fileMenu -> addSeparator();
+    m_fileMenu -> addAction(m_openAct);
+    m_fileMenu -> addAction(m_changeRemoteRepoAct);
+    m_fileMenu -> addSeparator();
 
-    advancedMenu = fileMenu->addMenu(tr("Advanced"));
+    m_advancedMenu = m_fileMenu->addMenu(tr("Advanced"));
 
-    fileMenu -> addAction(settingsAct);
+    m_fileMenu -> addAction(m_settingsAct);
 
-    fileMenu -> addSeparator();
-    fileMenu -> addAction(exitAct);
+    m_fileMenu -> addSeparator();
+    m_fileMenu -> addAction(m_exitAct);
 
-    advancedMenu -> addAction(hgIgnoreAct);
-    advancedMenu -> addSeparator();
-    advancedMenu -> addAction(hgServeAct);
+    m_advancedMenu -> addAction(m_hgIgnoreAct);
+    m_advancedMenu -> addSeparator();
+    m_advancedMenu -> addAction(m_hgServeAct);
 
-    helpMenu = menuBar()->addMenu(tr("Help"));
-    helpMenu->addAction(aboutAct);
+    m_helpMenu = menuBar()->addMenu(tr("Help"));
+    m_helpMenu->addAction(m_aboutAct);
 }
 
 void MainWindow::createToolBars()
 {
-    fileToolBar = addToolBar(tr("File"));
-    fileToolBar -> setIconSize(QSize(MY_ICON_SIZE, MY_ICON_SIZE));
-    fileToolBar -> addAction(openAct);
-    fileToolBar -> addAction(hgRefreshAct);
-    fileToolBar -> addSeparator();
-    fileToolBar -> setMovable(false);
+    m_fileToolBar = addToolBar(tr("File"));
+    m_fileToolBar -> setIconSize(QSize(MY_ICON_SIZE, MY_ICON_SIZE));
+    m_fileToolBar -> addAction(m_openAct);
+    m_fileToolBar -> addAction(m_hgRefreshAct);
+    m_fileToolBar -> addSeparator();
+    m_fileToolBar -> setMovable(false);
 
-    repoToolBar = addToolBar(tr(REPOMENU_TITLE));
-    repoToolBar -> setIconSize(QSize(MY_ICON_SIZE, MY_ICON_SIZE));
-    repoToolBar->addAction(hgIncomingAct);
-    repoToolBar->addAction(hgPullAct);
-    repoToolBar->addAction(hgPushAct);
-    repoToolBar -> setMovable(false);
+    m_repoToolBar = addToolBar(tr(REPOMENU_TITLE));
+    m_repoToolBar -> setIconSize(QSize(MY_ICON_SIZE, MY_ICON_SIZE));
+    m_repoToolBar->addAction(m_hgIncomingAct);
+    m_repoToolBar->addAction(m_hgPullAct);
+    m_repoToolBar->addAction(m_hgPushAct);
+    m_repoToolBar -> setMovable(false);
 
-    workFolderToolBar = addToolBar(tr(WORKFOLDERMENU_TITLE));
-    addToolBar(Qt::LeftToolBarArea, workFolderToolBar);
-    workFolderToolBar -> setIconSize(QSize(MY_ICON_SIZE, MY_ICON_SIZE));
-    workFolderToolBar->addAction(hgFolderDiffAct);
-    workFolderToolBar->addSeparator();
-    workFolderToolBar->addAction(hgRevertAct);
-    workFolderToolBar->addAction(hgUpdateAct);
-    workFolderToolBar->addAction(hgCommitAct);
-    workFolderToolBar->addAction(hgMergeAct);
-    workFolderToolBar->addSeparator();
-    workFolderToolBar->addAction(hgAddAct);
-    workFolderToolBar->addAction(hgRemoveAct);
-    workFolderToolBar -> setMovable(false);
+    m_workFolderToolBar = addToolBar(tr(WORKFOLDERMENU_TITLE));
+    addToolBar(Qt::LeftToolBarArea, m_workFolderToolBar);
+    m_workFolderToolBar -> setIconSize(QSize(MY_ICON_SIZE, MY_ICON_SIZE));
+    m_workFolderToolBar->addAction(m_hgFolderDiffAct);
+    m_workFolderToolBar->addSeparator();
+    m_workFolderToolBar->addAction(m_hgRevertAct);
+    m_workFolderToolBar->addAction(m_hgUpdateAct);
+    m_workFolderToolBar->addAction(m_hgCommitAct);
+    m_workFolderToolBar->addAction(m_hgMergeAct);
+    m_workFolderToolBar->addSeparator();
+    m_workFolderToolBar->addAction(m_hgAddAct);
+    m_workFolderToolBar->addAction(m_hgRemoveAct);
+    m_workFolderToolBar -> setMovable(false);
 
     updateToolBarStyle();
 }
@@ -2427,16 +2427,16 @@
 
     QSettings settings;
 
-    remoteRepoPath = settings.value("remoterepopath", "").toString();
-    workFolderPath = settings.value("workfolderpath", "").toString();
-    if (!workFolder.exists(workFolderPath))
+    m_remoteRepoPath = settings.value("remoterepopath", "").toString();
+    m_workFolderPath = settings.value("workfolderpath", "").toString();
+    if (!workFolder.exists(m_workFolderPath))
     {
-        workFolderPath = "";
+        m_workFolderPath = "";
     }
 
     QPoint pos = settings.value("pos", QPoint(200, 200)).toPoint();
     QSize size = settings.value("size", QSize(400, 400)).toSize();
-    firstStart = settings.value("firststart", QVariant(true)).toBool();
+    m_firstStart = settings.value("firststart", QVariant(true)).toBool();
 
 //!!!    initialFileTypesBits = (unsigned char) settings.value("viewFileTypes", QVariant(DEFAULT_HG_STAT_BITS)).toInt();
     resize(size);
@@ -2449,10 +2449,10 @@
     QSettings settings;
     settings.setValue("pos", pos());
     settings.setValue("size", size());
-    settings.setValue("remoterepopath", remoteRepoPath);
-    settings.setValue("workfolderpath", workFolderPath);
-    settings.setValue("firststart", firstStart);
-    //!!!settings.setValue("viewFileTypes", hgTabs -> getFileTypesBits());
+    settings.setValue("remoterepopath", m_remoteRepoPath);
+    settings.setValue("workfolderpath", m_workFolderPath);
+    settings.setValue("firststart", m_firstStart);
+    //!!!settings.setValue("viewFileTypes", m_hgTabs -> getFileTypesBits());
 }