Skip to content

Commit

Permalink
Sounds: make private methods private.
Browse files Browse the repository at this point in the history
  • Loading branch information
pulkomandy committed Oct 29, 2014
1 parent c54aa3d commit 3a8e734
Show file tree
Hide file tree
Showing 2 changed files with 118 additions and 116 deletions.
222 changes: 111 additions & 111 deletions src/preferences/sounds/HWindow.cpp
Expand Up @@ -50,7 +50,7 @@ HWindow::HWindow(BRect rect, const char* name)
fFilePanel(NULL),
fPlayer(NULL)
{
InitGUI();
_InitGUI();

fFilePanel = new BFilePanel();
fFilePanel->SetTarget(this);
Expand Down Expand Up @@ -91,62 +91,11 @@ HWindow::~HWindow()


void
HWindow::InitGUI()
HWindow::DispatchMessage(BMessage* message, BHandler* handler)
{
fEventList = new HEventList();
fEventList->SetType(BMediaFiles::B_SOUNDS);
fEventList->SetSelectionMode(B_SINGLE_SELECTION_LIST);

BMenu* menu = new BMenu("file");
menu->SetRadioMode(true);
menu->SetLabelFromMarked(true);
menu->AddSeparatorItem();
menu->AddItem(new BMenuItem(B_TRANSLATE("<none>"),
new BMessage(M_NONE_MESSAGE)));
menu->AddItem(new BMenuItem(B_TRANSLATE("Other" B_UTF8_ELLIPSIS),
new BMessage(M_OTHER_MESSAGE)));

BString label(B_TRANSLATE("Sound file:"));
BMenuField* menuField = new BMenuField("filemenu", label, menu);
menuField->SetDivider(menuField->StringWidth(label) + 10);

BSize buttonsSize(be_plain_font->Size() * 2.5, be_plain_font->Size() * 2.5);

BButton* stopbutton = new BButton("stop", "\xE2\x96\xA0",
new BMessage(M_STOP_MESSAGE));
stopbutton->SetEnabled(false);
stopbutton->SetExplicitSize(buttonsSize);

// We need at least one view to trigger B_PULSE_NEEDED events which we will
// intercept in DispatchMessage to trigger the buttons enabling or disabling.
stopbutton->SetFlags(stopbutton->Flags() | B_PULSE_NEEDED);

BButton* playbutton = new BButton("play", "\xE2\x96\xB6",
new BMessage(M_PLAY_MESSAGE));
playbutton->SetEnabled(false);
playbutton->SetExplicitSize(buttonsSize);

const float kInset = be_control_look->DefaultItemSpacing();

SetLayout(new BGroupLayout(B_HORIZONTAL));
AddChild(BGroupLayoutBuilder(B_VERTICAL)
.SetInsets(kInset, kInset, kInset, kInset)
.Add(fEventList)
.AddGroup(B_HORIZONTAL)
.SetInsets(0, 0, 0, 0)
.Add(menuField)
.AddGroup(B_HORIZONTAL, 0)
.Add(playbutton)
.Add(stopbutton)
.End()
.End()
);

// setup file menu
SetupMenuField();
BMenuItem* noneItem = menu->FindItem(B_TRANSLATE("<none>"));
if (noneItem != NULL)
noneItem->SetMarked(true);
if (message->what == B_PULSE)
_Pulse();
BWindow::DispatchMessage(message, handler);
}


Expand Down Expand Up @@ -299,8 +248,113 @@ HWindow::MessageReceived(BMessage* message)
}


bool
HWindow::QuitRequested()
{
fFrame = Frame();

fEventList->RemoveAll();
be_app->PostMessage(B_QUIT_REQUESTED);
return true;
}


void
HWindow::_InitGUI()
{
fEventList = new HEventList();
fEventList->SetType(BMediaFiles::B_SOUNDS);
fEventList->SetSelectionMode(B_SINGLE_SELECTION_LIST);

BMenu* menu = new BMenu("file");
menu->SetRadioMode(true);
menu->SetLabelFromMarked(true);
menu->AddSeparatorItem();
menu->AddItem(new BMenuItem(B_TRANSLATE("<none>"),
new BMessage(M_NONE_MESSAGE)));
menu->AddItem(new BMenuItem(B_TRANSLATE("Other" B_UTF8_ELLIPSIS),
new BMessage(M_OTHER_MESSAGE)));

BString label(B_TRANSLATE("Sound file:"));
BMenuField* menuField = new BMenuField("filemenu", label, menu);
menuField->SetDivider(menuField->StringWidth(label) + 10);

BSize buttonsSize(be_plain_font->Size() * 2.5, be_plain_font->Size() * 2.5);

BButton* stopbutton = new BButton("stop", "\xE2\x96\xA0",
new BMessage(M_STOP_MESSAGE));
stopbutton->SetEnabled(false);
stopbutton->SetExplicitSize(buttonsSize);

// We need at least one view to trigger B_PULSE_NEEDED events which we will
// intercept in DispatchMessage to trigger the buttons enabling or disabling.
stopbutton->SetFlags(stopbutton->Flags() | B_PULSE_NEEDED);

BButton* playbutton = new BButton("play", "\xE2\x96\xB6",
new BMessage(M_PLAY_MESSAGE));
playbutton->SetEnabled(false);
playbutton->SetExplicitSize(buttonsSize);

const float kInset = be_control_look->DefaultItemSpacing();

SetLayout(new BGroupLayout(B_HORIZONTAL));
AddChild(BGroupLayoutBuilder(B_VERTICAL)
.SetInsets(kInset, kInset, kInset, kInset)
.Add(fEventList)
.AddGroup(B_HORIZONTAL)
.SetInsets(0, 0, 0, 0)
.Add(menuField)
.AddGroup(B_HORIZONTAL, 0)
.Add(playbutton)
.Add(stopbutton)
.End()
.End()
);

// setup file menu
_SetupMenuField();
BMenuItem* noneItem = menu->FindItem(B_TRANSLATE("<none>"));
if (noneItem != NULL)
noneItem->SetMarked(true);
}


void
HWindow::_Pulse()
{
HEventRow* row = (HEventRow*)fEventList->CurrentSelection();
BMenuField* menufield = dynamic_cast<BMenuField*>(FindView("filemenu"));
BButton* button = dynamic_cast<BButton*>(FindView("play"));
BButton* stop = dynamic_cast<BButton*>(FindView("stop"));

if (menufield == NULL || button == NULL || stop == NULL)
return;

if (row != NULL) {
menufield->SetEnabled(true);

const char* path = row->Path();
if (path != NULL && strcmp(path, ""))
button->SetEnabled(true);
else
button->SetEnabled(false);
} else {
menufield->SetEnabled(false);
button->SetEnabled(false);
}

if (fPlayer != NULL) {
if (fPlayer->IsPlaying())
stop->SetEnabled(true);
else
stop->SetEnabled(false);
} else
stop->SetEnabled(false);
}


void
HWindow::SetupMenuField()
HWindow::_SetupMenuField()
{
BMenuField* menufield = dynamic_cast<BMenuField*>(FindView("filemenu"));
if (menufield == NULL)
Expand Down Expand Up @@ -360,57 +414,3 @@ HWindow::SetupMenuField()
}
}
}


void
HWindow::Pulse()
{
HEventRow* row = (HEventRow*)fEventList->CurrentSelection();
BMenuField* menufield = dynamic_cast<BMenuField*>(FindView("filemenu"));
BButton* button = dynamic_cast<BButton*>(FindView("play"));
BButton* stop = dynamic_cast<BButton*>(FindView("stop"));

if (menufield == NULL || button == NULL || stop == NULL)
return;

if (row != NULL) {
menufield->SetEnabled(true);

const char* path = row->Path();
if (path != NULL && strcmp(path, ""))
button->SetEnabled(true);
else
button->SetEnabled(false);
} else {
menufield->SetEnabled(false);
button->SetEnabled(false);
}

if (fPlayer != NULL) {
if (fPlayer->IsPlaying())
stop->SetEnabled(true);
else
stop->SetEnabled(false);
} else
stop->SetEnabled(false);
}


void
HWindow::DispatchMessage(BMessage* message, BHandler* handler)
{
if (message->what == B_PULSE)
Pulse();
BWindow::DispatchMessage(message, handler);
}


bool
HWindow::QuitRequested()
{
fFrame = Frame();

fEventList->RemoveAll();
be_app->PostMessage(B_QUIT_REQUESTED);
return true;
}
12 changes: 7 additions & 5 deletions src/preferences/sounds/HWindow.h
Expand Up @@ -38,13 +38,15 @@ class HWindow : public BWindow {
HWindow(BRect rect, const char* name);
virtual ~HWindow();

virtual void MessageReceived(BMessage* message);
virtual bool QuitRequested();
virtual void DispatchMessage(BMessage* message,
BHandler* handler);
void InitGUI();
void SetupMenuField();
void Pulse();
virtual void MessageReceived(BMessage* message);
virtual bool QuitRequested();

private:
void _InitGUI();
void _Pulse();
void _SetupMenuField();

private:
HEventList* fEventList;
Expand Down

0 comments on commit 3a8e734

Please sign in to comment.