[Themis-devcvs] SF.net SVN: themis:[609] trunk/themis/framework/win.cpp

[CVS C. to T. <the...@li...>]

Revision: 609
          http://themis.svn.sourceforge.net/themis/?rev=609&view=rev
Author:   mark_hellegers
Date:     2011-01-16 13:54:14 +0000 (Sun, 16 Jan 2011)

Log Message:
-----------
Converted the break code to if !=, so it is easier to see how the code flows.

Modified Paths:
--------------
    trunk/themis/framework/win.cpp

Modified: trunk/themis/framework/win.cpp
===================================================================
--- trunk/themis/framework/win.cpp	2011-01-16 13:45:17 UTC (rev 608)
+++ trunk/themis/framework/win.cpp	2011-01-16 13:54:14 UTC (rev 609)
@@ -943,35 +943,32 @@
 				case RENDERVIEW_POINTER: {
 					int32 site_id = 0;
 					message->FindInt32("site_id", &site_id);
-					if (site_id == 0)
-						break;
-
-					int32 tabindex;
-					ThemisTab * tab = FindTabFor(site_id, &tabindex);
-					if (tab == NULL)
-						break;
-
-					BView * renderview = NULL;
-					message->FindPointer("renderview_pointer", (void**)&renderview);
-					if (renderview == NULL)
-						break;
-					
-					/* Attach the renderview to the correct tab. */
-					Lock();
-					/*
-					 * We don't need to resize the renderview here, as this is done in
-					 * ThemisTabView::Select().
-					 */
-					tabview->TabAt(tabindex)->SetView(renderview);
-					
-					if (tabview->Selection() == tabindex) {
-						tabview->Select(tabindex);
-						
-						if (CurrentFocus() != NULL)
-							CurrentFocus()->MakeFocus(false);
-						tabview->TabAt(tabindex)->View()->MakeFocus(true);
+					if (site_id != 0) {
+						int32 tabindex;
+						ThemisTab * tab = FindTabFor(site_id, &tabindex);
+						if (tab != NULL) {
+							BView * renderview = NULL;
+							message->FindPointer("renderview_pointer", (void**)&renderview);
+							if (renderview != NULL) {
+								/* Attach the renderview to the correct tab. */
+								Lock();
+								/*
+								 * We don't need to resize the renderview here, as this is done in
+								 * ThemisTabView::Select().
+								 */
+								tabview->TabAt(tabindex)->SetView(renderview);
+								
+								if (tabview->Selection() == tabindex) {
+									tabview->Select(tabindex);
+									
+									if (CurrentFocus() != NULL)
+										CurrentFocus()->MakeFocus(false);
+									tabview->TabAt(tabindex)->View()->MakeFocus(true);
+								}
+								Unlock();
+							}
+						}
 					}
-					Unlock();
 
 					break;
 				}


This was sent by the SourceForge.net collaborative development platform, the world's largest Open Source development site.