Author: kirilla Date: 2011-04-11 18:55:56 +0200 (Mon, 11 Apr 2011) New Revision: 41216 Changeset: https://dev.haiku-os.org/changeset/41216 Modified: haiku/trunk/src/add-ons/input_server/devices/keyboard/KeyboardInputDevice.cpp haiku/trunk/src/add-ons/input_server/devices/keyboard/Keymap.cpp haiku/trunk/src/add-ons/input_server/devices/keyboard/TeamListItem.cpp haiku/trunk/src/add-ons/input_server/devices/keyboard/TeamListItem.h haiku/trunk/src/add-ons/input_server/devices/keyboard/TeamMonitorWindow.cpp Log: Cleanup. Removal of superfluous UpdateList() call. Modified: haiku/trunk/src/add-ons/input_server/devices/keyboard/KeyboardInputDevice.cpp =================================================================== --- haiku/trunk/src/add-ons/input_server/devices/keyboard/KeyboardInputDevice.cpp 2011-04-11 15:22:40 UTC (rev 41215) +++ haiku/trunk/src/add-ons/input_server/devices/keyboard/KeyboardInputDevice.cpp 2011-04-11 16:55:56 UTC (rev 41216) @@ -48,7 +48,7 @@ fFunctionName.String()); } - ~FunctionTracer() + ~FunctionTracer() { debug_printf("%p -> %s}\n", fPointer, fPrepend.String()); sFunctionDepth--; @@ -208,6 +208,7 @@ return fFD >= 0 ? B_OK : B_ERROR; } + void KeyboardDevice::Stop() { Modified: haiku/trunk/src/add-ons/input_server/devices/keyboard/Keymap.cpp =================================================================== --- haiku/trunk/src/add-ons/input_server/devices/keyboard/Keymap.cpp 2011-04-11 15:22:40 UTC (rev 41215) +++ haiku/trunk/src/add-ons/input_server/devices/keyboard/Keymap.cpp 2011-04-11 16:55:56 UTC (rev 41216) @@ -23,7 +23,7 @@ static void -print_key(char *chars, int32 offset) +print_key(char* chars, int32 offset) { int size = chars[offset++]; @@ -41,7 +41,7 @@ default: { // 2-, 3-, or 4-byte UTF-8 character - char *str = new (std::nothrow) char[size + 1]; + char* str = new (std::nothrow) char[size + 1]; if (str == NULL) break; Modified: haiku/trunk/src/add-ons/input_server/devices/keyboard/TeamListItem.cpp =================================================================== --- haiku/trunk/src/add-ons/input_server/devices/keyboard/TeamListItem.cpp 2011-04-11 15:22:40 UTC (rev 41215) +++ haiku/trunk/src/add-ons/input_server/devices/keyboard/TeamListItem.cpp 2011-04-11 16:55:56 UTC (rev 41216) @@ -48,11 +48,11 @@ void -TeamListItem::DrawItem(BView *owner, BRect frame, bool complete) +TeamListItem::DrawItem(BView* owner, BRect frame, bool complete) { - rgb_color kHighlight = { 140,140,140,0 }; - rgb_color kBlack = { 0,0,0,0 }; - rgb_color kBlue = { 0,0,255,0 }; + rgb_color kHighlight = { 140, 140, 140, 0 }; + rgb_color kBlack = { 0, 0, 0, 0 }; + rgb_color kBlue = { 0, 0, 255, 0 }; BRect r(frame); @@ -139,7 +139,8 @@ firstCall = false; } - if (strncmp(systemServersPath.Path(), fInfo.args, strlen(systemServersPath.Path())) == 0) + if (strncmp(systemServersPath.Path(), fInfo.args, + strlen(systemServersPath.Path())) == 0) return true; if (strncmp(trackerPath.Path(), fInfo.args, strlen(trackerPath.Path())) == 0) Modified: haiku/trunk/src/add-ons/input_server/devices/keyboard/TeamListItem.h =================================================================== --- haiku/trunk/src/add-ons/input_server/devices/keyboard/TeamListItem.h 2011-04-11 15:22:40 UTC (rev 41215) +++ haiku/trunk/src/add-ons/input_server/devices/keyboard/TeamListItem.h 2011-04-11 16:55:56 UTC (rev 41216) @@ -20,14 +20,15 @@ TeamListItem(team_info& info); virtual ~TeamListItem(); - virtual void DrawItem(BView* owner, BRect frame, + virtual void DrawItem(BView* owner, BRect frame, bool complete = false); - virtual void Update(BView* owner, const BFont* font); + virtual void Update(BView* owner, const BFont* font); - const team_info* GetInfo(); - const BBitmap* LargeIcon() { return &fLargeIcon; }; - const BPath* Path() { return &fPath; }; - const BString* AppSignature() { return &fAppSignature; }; + const team_info* GetInfo(); + const BBitmap* LargeIcon() { return &fLargeIcon; }; + const BPath* Path() { return &fPath; }; + const BString* AppSignature() { return &fAppSignature; }; + bool IsSystemServer(); bool IsApplication(); Modified: haiku/trunk/src/add-ons/input_server/devices/keyboard/TeamMonitorWindow.cpp =================================================================== --- haiku/trunk/src/add-ons/input_server/devices/keyboard/TeamMonitorWindow.cpp 2011-04-11 15:22:40 UTC (rev 41215) +++ haiku/trunk/src/add-ons/input_server/devices/keyboard/TeamMonitorWindow.cpp 2011-04-11 16:55:56 UTC (rev 41216) @@ -86,7 +86,7 @@ fListView = new BListView("teams"); fListView->SetSelectionMessage(new BMessage(TM_SELECTED_TEAM)); - BScrollView *scrollView = new BScrollView("scroll_teams", fListView, + BScrollView* scrollView = new BScrollView("scroll_teams", fListView, 0, B_SUPPORTS_LAYOUT, false, true, B_FANCY_BORDER); layout->AddView(scrollView); @@ -111,7 +111,7 @@ groupView = new BGroupView(B_HORIZONTAL); layout->AddView(groupView); - BButton *forceReboot = new BButton("force", "Force Reboot", + BButton* forceReboot = new BButton("force", "Force Reboot", new BMessage(TM_FORCE_REBOOT)); groupView->GroupLayout()->AddView(forceReboot); @@ -164,7 +164,7 @@ void -TeamMonitorWindow::MessageReceived(BMessage *msg) +TeamMonitorWindow::MessageReceived(BMessage* msg) { switch (msg->what) { case SYSTEM_SHUTTING_DOWN: @@ -193,8 +193,8 @@ case TM_KILL_APPLICATION: { - TeamListItem* item = (TeamListItem*)fListView->ItemAt( - fListView->CurrentSelection()); + TeamListItem* item = dynamic_cast<TeamListItem*>(fListView->ItemAt( + fListView->CurrentSelection())); if (item != NULL) { kill_team(item->GetInfo()->team); UpdateList(); @@ -209,7 +209,6 @@ BMessenger messenger(item->AppSignature()->String(), item->GetInfo()->team); messenger.SendMessage(B_QUIT_REQUESTED); - UpdateList(); } break; } @@ -226,10 +225,10 @@ case TM_SELECTED_TEAM: { fKillButton->SetEnabled(fListView->CurrentSelection() >= 0); - TeamListItem* item = (TeamListItem*)fListView->ItemAt( - fListView->CurrentSelection()); + TeamListItem* item = dynamic_cast<TeamListItem*>(fListView->ItemAt( + fListView->CurrentSelection())); fDescriptionView->SetItem(item); - fQuitButton->SetEnabled(item && item->IsApplication()); + fQuitButton->SetEnabled(item != NULL && item->IsApplication()); break; } case TM_CANCEL: @@ -257,8 +256,9 @@ bool changed = false; for (int32 i = 0; i < fListView->CountItems(); i++) { - TeamListItem *item = (TeamListItem*)fListView->ItemAt(i); - item->SetFound(false); + TeamListItem* item = dynamic_cast<TeamListItem*>(fListView->ItemAt(i)); + if (item != NULL) + item->SetFound(false); } int32 cookie = 0; @@ -269,8 +269,8 @@ bool found = false; for (int32 i = 0; i < fListView->CountItems(); i++) { - TeamListItem *item = (TeamListItem*)fListView->ItemAt(i); - if (item->GetInfo()->team == info.team) { + TeamListItem* item = dynamic_cast<TeamListItem*>(fListView->ItemAt(i)); + if (item != NULL && item->GetInfo()->team == info.team) { item->SetFound(true); found = true; } @@ -287,8 +287,8 @@ } for (int32 i = fListView->CountItems() - 1; i >= 0; i--) { - TeamListItem *item = (TeamListItem*)fListView->ItemAt(i); - if (!item->Found()) { + TeamListItem* item = dynamic_cast<TeamListItem*>(fListView->ItemAt(i)); + if (item != NULL && !item->Found()) { if (item == fDescriptionView->Item()) { fDescriptionView->SetItem(NULL); fKillButton->SetEnabled(false); @@ -472,7 +472,7 @@ void -TeamDescriptionView::GetPreferredSize(float *_width, float *_height) +TeamDescriptionView::GetPreferredSize(float* _width, float* _height) { if (_width != NULL) { float width = 0; @@ -503,7 +503,7 @@ void -TeamDescriptionView::SetItem(TeamListItem *item) +TeamDescriptionView::SetItem(TeamListItem* item) { fItem = item; Invalidate();