[haiku-commits] r41216 - haiku/trunk/src/add-ons/input_server/devices/keyboard

  • From: jonas@xxxxxxxxxxx
  • To: haiku-commits@xxxxxxxxxxxxx
  • Date: Mon, 11 Apr 2011 18:55:56 +0200 (CEST)

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();


Other related posts:

  • » [haiku-commits] r41216 - haiku/trunk/src/add-ons/input_server/devices/keyboard - jonas