[haiku-commits] haiku: hrev44244 - src/apps/deskbar
- From: anevilyak@xxxxxxxxx
- To: haiku-commits@xxxxxxxxxxxxx
- Date: Sat, 16 Jun 2012 22:48:37 +0200 (CEST)
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();
}
Other related posts:
- » [haiku-commits] haiku: hrev44244 - src/apps/deskbar - anevilyak