hrev44244 adds 1 changeset to branch 'master' old head: 919d3b76016bb150d60488e9224e71a87f780eda new head: a663e9050638c7470918bda00c4f260c30513990 ---------------------------------------------------------------------------- a663e90: Fix #8640. - Automatic whitespace cleanup. - Remove no longer exposed show/hide time setting. The methodology for saving it on exit was broken anyways, since it relied on the current show/hide state of the time view, which would be hidden if the deskbar was currently in Autohide mode. [ Rene Gollent <anevilyak@xxxxxxxxx> ] ---------------------------------------------------------------------------- Revision: hrev44244 Commit: a663e9050638c7470918bda00c4f260c30513990 URL: http://cgit.haiku-os.org/haiku/commit/?id=a663e90 Author: Rene Gollent <anevilyak@xxxxxxxxx> Date: Sat Jun 16 20:45:28 2012 UTC Ticket: https://dev.haiku-os.org/ticket/8640 ---------------------------------------------------------------------------- 3 files changed, 8 insertions(+), 21 deletions(-) src/apps/deskbar/BarApp.cpp | 6 ------ src/apps/deskbar/BarApp.h | 1 - src/apps/deskbar/StatusView.cpp | 22 ++++++++-------------- ---------------------------------------------------------------------------- diff --git a/src/apps/deskbar/BarApp.cpp b/src/apps/deskbar/BarApp.cpp index 91f2ce8..33c5832 100644 --- a/src/apps/deskbar/BarApp.cpp +++ b/src/apps/deskbar/BarApp.cpp @@ -204,7 +204,6 @@ TBarApp::SaveSettings() storedSettings.AddInt32("state", fSettings.state); storedSettings.AddFloat("width", fSettings.width); - storedSettings.AddBool("showTime", fSettings.showTime); storedSettings.AddBool("showSeconds", fSettings.showSeconds); storedSettings.AddBool("showDayOfWeek", fSettings.showDayOfWeek); @@ -244,7 +243,6 @@ TBarApp::InitSettings() settings.vertical = true; settings.left = false; settings.top = true; - settings.showTime = true; settings.showSeconds = false; settings.showDayOfWeek = false; settings.state = kExpandoState; @@ -301,10 +299,6 @@ TBarApp::InitSettings() } if (storedSettings.FindFloat("width", &settings.width) != B_OK) settings.width = 0; - if (storedSettings.FindBool("showTime", &settings.showTime) - != B_OK) { - settings.showTime = true; - } if (storedSettings.FindBool("showSeconds", &settings.showSeconds) != B_OK) { settings.showSeconds = false; diff --git a/src/apps/deskbar/BarApp.h b/src/apps/deskbar/BarApp.h index 6dfac18..2c81757 100644 --- a/src/apps/deskbar/BarApp.h +++ b/src/apps/deskbar/BarApp.h @@ -75,7 +75,6 @@ struct desk_settings { bool vertical; bool left; bool top; - bool showTime; bool showSeconds; bool showDayOfWeek; uint32 state; diff --git a/src/apps/deskbar/StatusView.cpp b/src/apps/deskbar/StatusView.cpp index 08da5e9..e32b769 100644 --- a/src/apps/deskbar/StatusView.cpp +++ b/src/apps/deskbar/StatusView.cpp @@ -181,11 +181,6 @@ TReplicantTray::AttachedToWindow() AddChild(fTime); fTime->MoveTo(Bounds().right - fTime->Bounds().Width() - 1, 2); - if (!((TBarApp*)be_app)->Settings()->showTime) { - fTime->Hide(); - RealignReplicants(); - AdjustPlacement(); - } #ifdef DB_ADDONS // load addons and rehydrate archives @@ -438,13 +433,13 @@ TReplicantTray::InitAddOnSupport() int32 id; BString path; if (fAddOnSettings.Unflatten(&file) == B_OK) { - for (int32 i = 0; fAddOnSettings.FindString(kReplicantPathField, + for (int32 i = 0; fAddOnSettings.FindString(kReplicantPathField, i, &path) == B_OK; i++) { if (entry.SetTo(path.String()) == B_OK && entry.Exists()) { result = LoadAddOn(&entry, &id, false); - } else + } else result = B_ENTRY_NOT_FOUND; - + if (result != B_OK) { fAddOnSettings.RemoveData(kReplicantPathField, i); --i; @@ -460,7 +455,7 @@ void TReplicantTray::DeleteAddOnSupport() { _SaveSettings(); - + for (int32 i = fItemList->CountItems(); i-- > 0 ;) { DeskbarItemInfo* item = (DeskbarItemInfo*)fItemList->RemoveItem(i); if (item) { @@ -641,7 +636,7 @@ TReplicantTray::LoadAddOn(BEntry* entry, int32* id, bool addToSettings) fAddOnSettings.AddString(kReplicantPathField, path.Path()); _SaveSettings(); } - + return B_OK; } @@ -711,7 +706,7 @@ TReplicantTray::RemoveItem(int32 id) if (item->isAddOn) { BPath path(&item->entryRef); BString storedPath; - for (int32 i = 0; + for (int32 i = 0; fAddOnSettings.FindString(kReplicantPathField, i, &storedPath) == B_OK; i++) { if (storedPath == path.Path()) { @@ -720,7 +715,7 @@ TReplicantTray::RemoveItem(int32 id) } } _SaveSettings(); - + BNode node(&item->entryRef); watch_node(&item->nodeRef, B_STOP_WATCHING, this, Window()); } @@ -1225,7 +1220,7 @@ TReplicantTray::_SaveSettings() path.Append(kReplicantSettingsFile); BFile file(path.Path(), B_READ_WRITE | B_CREATE_FILE | B_ERASE_FILE); - if ((result = file.InitCheck()) == B_OK) + if ((result = file.InitCheck()) == B_OK) result = fAddOnSettings.Flatten(&file); } @@ -1240,7 +1235,6 @@ TReplicantTray::SaveTimeSettings() return; desk_settings* settings = ((TBarApp*)be_app)->Settings(); - settings->showTime = !fTime->IsHidden(); settings->showSeconds = fTime->ShowSeconds(); settings->showDayOfWeek = fTime->ShowDayOfWeek(); }