sync with github bd3351e572

As always, full log is in GitHub.

git-svn-id: https://rt-thread.googlecode.com/svn/trunk@2234 bbd45198-f89e-11dd-88c7-29a3b14d5316

components/rtgui/SConscript

-Import('RTT_ROOT')
+import os
 from building import *
 
 common_src = Split("""
@@ -72,10 +72,10 @@ widgets/panel.c
 # The set of source files associated with this SConscript file.
 src = common_src + server_src + widgets_src
 
-path = [RTT_ROOT + '/components/rtgui/include',
-	RTT_ROOT + '/components/rtgui/common', 
-	RTT_ROOT + '/components/rtgui/server', 
-	RTT_ROOT + '/components/rtgui/widgets']
+cwd = GetCurrentDir()
+dirs = ['include', 'common', 'server', 'widgets']
+path = [os.path.join(cwd, i) for i in dirs]
+
 group = DefineGroup('RTGUI', src, depend = ['RT_USING_RTGUI'], CPPPATH = path)
 
 Return('group')

components/rtgui/common/filerw.c

@@ -251,21 +251,7 @@ static int mem_read(struct rtgui_filerw *context, void *ptr, rt_size_t size, rt_
 
 static int mem_write(struct rtgui_filerw *context, const void *ptr, rt_size_t size, rt_size_t num)
 {
-#if 0
-	struct rtgui_filerw_mem* mem = (struct rtgui_filerw_mem*)context;
-
-	if ((mem->mem_position + (num * size)) > mem->mem_end)
-	{
-		num = (mem->mem_end - mem->mem_position)/size;
-	}
-
-	rt_memcpy(mem->mem_position, ptr, num*size);
-	mem->mem_position += num*size;
-
-	return num;
-#else
 	return 0; /* not support memory write */
-#endif
 }
 
 static int mem_tell(struct rtgui_filerw* context)

components/rtgui/common/image.c

@@ -82,7 +82,7 @@ static struct rtgui_image_engine* rtgui_image_get_engine(const char* type)
 }
 
 #if defined(RTGUI_USING_DFS_FILERW) || defined(RTGUI_USING_STDIO_FILERW)
-static struct rtgui_image_engine* rtgui_image_get_engine_by_filename(const char* fn)
+struct rtgui_image_engine* rtgui_image_get_engine_by_filename(const char* fn)
 {
 	struct rtgui_list_node *node;
 	struct rtgui_image_engine *engine;

components/rtgui/common/image_bmp.c

@@ -33,22 +33,19 @@ struct rtgui_image_bmp
 {
     rt_bool_t is_loaded;
     rt_uint8_t *pixels;
-	struct rtgui_filerw* filerw;
+    struct rtgui_filerw *filerw;
     rt_uint32_t w, h;
     rt_uint32_t pixel_offset;
-	rt_uint32_t pitch;    rt_uint8_t scale;
+    rt_uint32_t pitch;
+    rt_uint8_t scale;
     rt_uint8_t bit_per_pixel;
     rt_uint8_t pad;
 };
 
-/* static rt_bool_t rtgui_image_bmp_check(struct rtgui_filerw* file,
-    rt_uint32_t *width, rt_uint32_t *height);
-static rt_bool_t rtgui_image_bmp_load(struct rtgui_image* image,
-    struct rtgui_filerw* file, rt_uint8_t scale, rt_bool_t load); */
-static rt_bool_t rtgui_image_bmp_check(struct rtgui_filerw* file);
-static rt_bool_t rtgui_image_bmp_load(struct rtgui_image* image, struct rtgui_filerw* file, rt_bool_t load);
-static void rtgui_image_bmp_unload(struct rtgui_image* image);
-static void rtgui_image_bmp_blit(struct rtgui_image* image, struct rtgui_dc* dc, struct rtgui_rect* rect);
+static rt_bool_t rtgui_image_bmp_check(struct rtgui_filerw *file);
+static rt_bool_t rtgui_image_bmp_load(struct rtgui_image *image, struct rtgui_filerw *file, rt_bool_t load);
+static void rtgui_image_bmp_unload(struct rtgui_image *image);
+static void rtgui_image_bmp_blit(struct rtgui_image *image, struct rtgui_dc *dc, struct rtgui_rect *rect);
 
 
 struct rtgui_image_engine rtgui_image_bmp_engine =
@@ -61,16 +58,14 @@ struct rtgui_image_engine rtgui_image_bmp_engine =
     rtgui_image_bmp_blit
 };
 
-static rt_bool_t rtgui_image_bmp_check(struct rtgui_filerw* file)
-//static rt_bool_t rtgui_image_bmp_check(struct rtgui_filerw* file,
-//    rt_uint32_t *width, rt_uint32_t *height)
+static rt_bool_t rtgui_image_bmp_check(struct rtgui_filerw *file)
 {
     rt_uint8_t buffer[18];
     rt_bool_t is_bmp = RT_FALSE;
 
     do
     {
-        if (!file )
+        if (!file)
         {
             break;
         }
@@ -92,34 +87,33 @@ static rt_bool_t rtgui_image_bmp_check(struct rtgui_filerw* file)
 
         /* Read BMP header size */
         if (*(rt_uint32_t *)&buffer[14] == 12)
-        {   /* Bitmap Header Version 2.x */
+        {
+            /* Bitmap Header Version 2.x */
             if (rtgui_filerw_read(file, (void *)buffer, 8, 1) != 8)
             {
                 break;
             }
             /* Read image size */
-//            *width = (rt_uint32_t)*(rt_uint16_t *)&buffer[0];
-//            *height = (rt_uint32_t)*(rt_uint16_t *)&buffer[2];
             is_bmp = RT_TRUE;
         }
         else
-        {   /* Bitmap Header Version bigger than 2.x */
+        {
+            /* Bitmap Header Version bigger than 2.x */
             if (rtgui_filerw_read(file, (void *)buffer, 8, 1) != 8)
             {
                 break;
             }
             /* Read image size */
-//            *width = *(rt_uint32_t *)&buffer[0];
-//            *height = *(rt_uint32_t *)&buffer[4];
             is_bmp = RT_TRUE;
         }
-    } while(0);
+    }
+    while (0);
 
     return is_bmp;
 }
 
-static struct rtgui_image_palette* rtgui_image_bmp_load_palette(
-    struct rtgui_filerw* file,
+static struct rtgui_image_palette *rtgui_image_bmp_load_palette(
+    struct rtgui_filerw *file,
     rt_uint32_t colorsUsed,
     rt_bool_t alpha)
 {
@@ -163,13 +157,11 @@ static struct rtgui_image_palette* rtgui_image_bmp_load_palette(
     return palette;
 }
 
-static rt_bool_t rtgui_image_bmp_load(struct rtgui_image* image, struct rtgui_filerw* file, rt_bool_t load)
-//static rt_bool_t rtgui_image_bmp_load(struct rtgui_image* image,
-//    struct rtgui_filerw* file, rt_uint8_t scale, rt_bool_t load)
+static rt_bool_t rtgui_image_bmp_load(struct rtgui_image *image, struct rtgui_filerw *file, rt_bool_t load)
 {
-    rt_uint8_t scale = 2;
+    rt_uint8_t scale = 0;
     rt_uint8_t *wrkBuffer;
-	struct rtgui_image_bmp* bmp;
+    struct rtgui_image_bmp *bmp;
     rt_uint32_t bmpHeaderSize;
     rt_uint32_t colorsUsed;
 
@@ -187,7 +179,7 @@ static rt_bool_t rtgui_image_bmp_load(struct rtgui_image* image, struct rtgui_fi
             break;
         }
 
-        bmp = (struct rtgui_image_bmp*)rtgui_malloc(sizeof(struct rtgui_image_bmp));
+        bmp = (struct rtgui_image_bmp *)rtgui_malloc(sizeof(struct rtgui_image_bmp));
         if (bmp == RT_NULL)
         {
             break;
@@ -216,19 +208,21 @@ static rt_bool_t rtgui_image_bmp_load(struct rtgui_image* image, struct rtgui_fi
 //        rt_kprintf("BMP: bmpHeaderSize %d\n", bmpHeaderSize);
         colorsUsed = 0;
         if (bmpHeaderSize == 12)
-        {   /* Bitmap Header Version 2.x */
+        {
+            /* Bitmap Header Version 2.x */
             if (rtgui_filerw_read(file, (void *)wrkBuffer, 8, 1) != 8)
             {
                 break;
             }
             /* Read image size */
-            bmp->w = (rt_uint32_t)*(rt_uint16_t *)&wrkBuffer[0];
-            bmp->h = (rt_uint32_t)*(rt_uint16_t *)&wrkBuffer[2];
+            bmp->w = (rt_uint32_t) * (rt_uint16_t *)&wrkBuffer[0];
+            bmp->h = (rt_uint32_t) * (rt_uint16_t *)&wrkBuffer[2];
             /* Read bits per pixel */
-            bmp->bit_per_pixel = (rt_uint8_t)*(rt_uint16_t *)&wrkBuffer[6];
+            bmp->bit_per_pixel = (rt_uint8_t) * (rt_uint16_t *)&wrkBuffer[6];
         }
         else
-        {   /* Bitmap Header Version bigger than 2.x */
+        {
+            /* Bitmap Header Version bigger than 2.x */
             rt_uint32_t compression;
 
             if (rtgui_filerw_read(file, (void *)wrkBuffer, 36, 1) != 36)
@@ -239,7 +233,7 @@ static rt_bool_t rtgui_image_bmp_load(struct rtgui_image* image, struct rtgui_fi
             bmp->w = *(rt_uint32_t *)&wrkBuffer[0];
             bmp->h = *(rt_uint32_t *)&wrkBuffer[4];
             /* Read bits per pixel */
-            bmp->bit_per_pixel = (rt_uint8_t)*(rt_uint16_t *)&wrkBuffer[10];
+            bmp->bit_per_pixel = (rt_uint8_t) * (rt_uint16_t *)&wrkBuffer[10];
             if (bmp->bit_per_pixel > 32)
             {
                 rt_kprintf("BMP err: unsupported format\n");
@@ -364,7 +358,7 @@ static rt_bool_t rtgui_image_bmp_load(struct rtgui_image* image, struct rtgui_fi
             {
                 dst = bmp->pixels + (image->h - y - 1) * imageWidth;
                 readIndex = 0;
-                skipLength= 0;
+                skipLength = 0;
 
                 /* Process a line */
                 while (readIndex < bmp->pitch)
@@ -446,7 +440,7 @@ static rt_bool_t rtgui_image_bmp_load(struct rtgui_image* image, struct rtgui_fi
                 /* Skip padding bytes  */
                 if (bmp->pad)
                 {
-                    if ( rtgui_filerw_seek(file, bmp->pad, RTGUI_FILE_SEEK_CUR) < 0)
+                    if (rtgui_filerw_seek(file, bmp->pad, RTGUI_FILE_SEEK_CUR) < 0)
                     {
                         error = RT_TRUE;
                         break;
@@ -478,7 +472,8 @@ static rt_bool_t rtgui_image_bmp_load(struct rtgui_image* image, struct rtgui_fi
         /* Release memory */
         rt_free(wrkBuffer);
         return RT_TRUE;
-    } while(0);
+    }
+    while (0);
 
     /* Release memory */
     rt_free(wrkBuffer);
@@ -488,13 +483,13 @@ static rt_bool_t rtgui_image_bmp_load(struct rtgui_image* image, struct rtgui_fi
     return RT_FALSE;
 }
 
-static void rtgui_image_bmp_unload(struct rtgui_image* image)
+static void rtgui_image_bmp_unload(struct rtgui_image *image)
 {
-	struct rtgui_image_bmp* bmp;
+    struct rtgui_image_bmp *bmp;
 
     if (image != RT_NULL)
     {
-		bmp = (struct rtgui_image_bmp*)image->data;
+        bmp = (struct rtgui_image_bmp *)image->data;
 
         /* Release memory */
         rtgui_free(bmp->pixels);
@@ -508,15 +503,15 @@ static void rtgui_image_bmp_unload(struct rtgui_image* image)
     }
 }
 
-static void rtgui_image_bmp_blit(struct rtgui_image* image, struct rtgui_dc* dc, struct rtgui_rect* dst_rect)
+static void rtgui_image_bmp_blit(struct rtgui_image *image, struct rtgui_dc *dc, struct rtgui_rect *dst_rect)
 {
     rt_uint16_t w, h;
-	struct rtgui_image_bmp* bmp;
+    struct rtgui_image_bmp *bmp;
     rt_uint8_t bytePerPixel;
     rt_uint32_t imageWidth;
     rt_bool_t error;
 
-	bmp = (struct rtgui_image_bmp*)image->data;
+    bmp = (struct rtgui_image_bmp *)image->data;
     RT_ASSERT(image != RT_NULL || dc != RT_NULL || dst_rect != RT_NULL || bmp != RT_NULL);
 
     bytePerPixel = bmp->bit_per_pixel / 8;
@@ -560,19 +555,25 @@ static void rtgui_image_bmp_blit(struct rtgui_image* image, struct rtgui_dc* dc,
             rt_uint8_t skipLength;
             rt_uint16_t x, y;
             rt_int8_t scale1, scale2;
+            rt_uint16_t y_start = dst_rect->y1 + h - 1;
 
-            wrkBuffer = (rt_uint8_t *)rt_malloc(BMP_WORKING_BUFFER_SIZE);
-            if (wrkBuffer == RT_NULL)
+            /* Read the pixels.  Note that the bmp image is upside down */
+            if (rtgui_filerw_seek(bmp->filerw, bmp->pixel_offset, RTGUI_FILE_SEEK_SET) < 0)
             {
-                rt_kprintf("BMP err: no mem (%d)\n", BMP_WORKING_BUFFER_SIZE);
                 break;
             }
-
-            /* Read the pixels.  Note that the bmp image is upside down */
-            if (rtgui_filerw_seek(bmp->filerw, bmp->pixel_offset, RTGUI_FILE_SEEK_SET) < 0)
+            /* the image is upside down. So we need to start from middle if the
+             * image is higher than the dst_rect. */
+            if (image->h > rtgui_rect_height(*dst_rect))
             {
+                int hdelta = image->h - rtgui_rect_height(*dst_rect);
+                if (rtgui_filerw_seek(bmp->filerw, hdelta * (bmp->pitch + bmp->pad) * (1 << bmp->scale),
+                            RTGUI_FILE_SEEK_CUR) < 0)
+                {
+                    error = RT_TRUE;
                     break;
                 }
+            }
 
             if (bmp->bit_per_pixel == 1)
             {
@@ -601,19 +602,28 @@ static void rtgui_image_bmp_blit(struct rtgui_image* image, struct rtgui_dc* dc,
                 }
             }
 
+            wrkBuffer = (rt_uint8_t *)rt_malloc(
+					(BMP_WORKING_BUFFER_SIZE > bmp->pitch) ? \
+					bmp->pitch : BMP_WORKING_BUFFER_SIZE);
+            if (wrkBuffer == RT_NULL)
+            {
+                rt_kprintf("BMP err: no mem (%d)\n", BMP_WORKING_BUFFER_SIZE);
+                break;
+            }
+
             /* Process whole image */
             y = 0;
-            while (y < image->h)
+            while (y < h)
             {
                 x = 0;
                 readIndex = 0;
-                skipLength= 0;
+                skipLength = 0;
 
                 /* Process a line */
                 while (readIndex < bmp->pitch)
                 {
                     /* Put progress indicator */
-                    rt_kprintf("\r%lu%%", y * 100UL / image->h);
+                    rt_kprintf("\r%lu%%", y * 100UL / h);
 
                     /* Read data to buffer */
                     readLength = (BMP_WORKING_BUFFER_SIZE > (bmp->pitch - readIndex)) ? \
@@ -629,18 +639,19 @@ static void rtgui_image_bmp_blit(struct rtgui_image* image, struct rtgui_dc* dc,
                     /* Process read buffer */
                     if (bmp->bit_per_pixel == 1)
                     {
-                        rt_uint8_t j;
-                        rtgui_color_t color;
-
                         for (loadIndex = skipLength; loadIndex < readLength; loadIndex += 1 << scale1)
                         {
+                            rt_uint8_t j;
                             for (j = 0; j < 8; j += 1 << scale2)
                             {
+                                rtgui_color_t color;
                                 color = image->palette->colors[(wrkBuffer[loadIndex] & (1 << (7 - j))) >> (7 - j)];
                                 rtgui_dc_draw_color_point(dc,
                                                           dst_rect->x1 + x++,
-                                    dst_rect->y1 + image->h - y,
+                                                          y_start - y,
                                                           color);
+                                if (x >= w)
+                                    break;
                             }
                             if (scale1 && (readLength % (1 << scale1)))
                             {
@@ -650,18 +661,19 @@ static void rtgui_image_bmp_blit(struct rtgui_image* image, struct rtgui_dc* dc,
                     }
                     else if (bmp->bit_per_pixel == 4)
                     {
-                        rt_uint8_t j;
-                        rtgui_color_t color;
-
                         for (loadIndex = skipLength; loadIndex < readLength; loadIndex += 1 << scale1)
                         {
+                            rt_uint8_t j;
                             for (j = 0; j < 8; j += 1 << (2 + scale2))
                             {
+                                rtgui_color_t color;
                                 color = image->palette->colors[(wrkBuffer[loadIndex] & (0x0F << (4 - j))) >> (4 - j)];
                                 rtgui_dc_draw_color_point(dc,
                                                           dst_rect->x1 + x++,
-                                    dst_rect->y1 + image->h - y,
+                                                          y_start - y,
                                                           color);
+                                if (x >= w)
+                                    break;
                             }
                         }
                         if (scale1 && (readLength % (1 << scale1)))
@@ -671,15 +683,16 @@ static void rtgui_image_bmp_blit(struct rtgui_image* image, struct rtgui_dc* dc,
                     }
                     else if (bmp->bit_per_pixel == 8)
                     {
-                        rtgui_color_t color;
-
                         for (loadIndex = skipLength; loadIndex < readLength; loadIndex += 1 << bmp->scale)
                         {
+                            rtgui_color_t color;
                             color = image->palette->colors[wrkBuffer[loadIndex]];
                             rtgui_dc_draw_color_point(dc,
                                                       dst_rect->x1 + x++,
-                                dst_rect->y1 + image->h - y,
+                                                      y_start - y,
                                                       color);
+                            if (x >= w)
+                                break;
                         }
                         if (readLength % (1 << bmp->scale))
                         {
@@ -713,9 +726,11 @@ static void rtgui_image_bmp_blit(struct rtgui_image* image, struct rtgui_dc* dc,
                             blit_line(temp, &wrkBuffer[loadIndex], bytePerPixel);
                             dc->engine->blit_line(dc,
                                                   dst_rect->x1 + x, dst_rect->x1 + x + 1,
-                                dst_rect->y1 + image->h - y,
+                                                  y_start - y,
                                                   temp);
                             x++;
+                            if (x >= w)
+                                break;
                         }
                         if (readLength % (1 << bmp->scale))
                         {
@@ -732,7 +747,7 @@ static void rtgui_image_bmp_blit(struct rtgui_image* image, struct rtgui_dc* dc,
                 /* Skip padding bytes  */
                 if (bmp->pad)
                 {
-                    if ( rtgui_filerw_seek(bmp->filerw, bmp->pad, RTGUI_FILE_SEEK_CUR) < 0)
+                    if (rtgui_filerw_seek(bmp->filerw, bmp->pad, RTGUI_FILE_SEEK_CUR) < 0)
                     {
                         error = RT_TRUE;
                         break;
@@ -761,7 +776,7 @@ static void rtgui_image_bmp_blit(struct rtgui_image* image, struct rtgui_dc* dc,
         else
         {
             rt_uint16_t x, y;
-            rt_uint8_t* ptr;
+            rt_uint8_t *ptr;
 
             for (y = 0; y < h; y ++)
             {
@@ -813,7 +828,8 @@ static void rtgui_image_bmp_blit(struct rtgui_image* image, struct rtgui_dc* dc,
             }
         }
 //        rt_kprintf("BMP: blit ok\n");
-    } while(0);
+    }
+    while (0);
 }
 
 void rtgui_image_bmp_init()

components/rtgui/include/rtgui/image.h

@@ -60,6 +60,7 @@ typedef struct rtgui_image rtgui_image_t;
 void rtgui_system_image_init(void);
 
 #if defined(RTGUI_USING_DFS_FILERW) || defined(RTGUI_USING_STDIO_FILERW)
+struct rtgui_image_engine* rtgui_image_get_engine_by_filename(const char* fn);
 struct rtgui_image* rtgui_image_create_from_file(const char* type, const char* filename, rt_bool_t load);
 struct rtgui_image* rtgui_image_create(const char* filename, rt_bool_t load);
 #endif

components/rtgui/include/rtgui/widgets/notebook.h

@@ -34,6 +34,7 @@ struct rtgui_notebook* rtgui_notebook_create(const rtgui_rect_t* rect, rt_uint8_
 void rtgui_notebook_destroy(struct rtgui_notebook* notebook);
 
 void rtgui_notebook_add(struct rtgui_notebook* notebook, const char* label, struct rtgui_widget* child);
+void rtgui_notebook_remove(struct rtgui_notebook* notebook, rt_uint16_t index);
 struct rtgui_widget* rtgui_notebook_get_current(struct rtgui_notebook* notebook);
 rt_int16_t rtgui_notebook_get_current_index(struct rtgui_notebook* notebook);
 

components/rtgui/include/rtgui/widgets/window.h

@@ -133,10 +133,6 @@ void rtgui_win_move(struct rtgui_win* win, int x, int y);
 /* reset extent of window */
 void rtgui_win_set_rect(rtgui_win_t* win, rtgui_rect_t* rect);
 
-#ifndef RTGUI_USING_SMALL_SIZE
-void rtgui_win_set_box(rtgui_win_t* win, rtgui_box_t* box);
-#endif
-
 void rtgui_win_set_onactivate(rtgui_win_t* win, rtgui_event_handler_ptr handler);
 void rtgui_win_set_ondeactivate(rtgui_win_t* win, rtgui_event_handler_ptr handler);
 void rtgui_win_set_onclose(rtgui_win_t* win, rtgui_event_handler_ptr handler);

components/rtgui/widgets/notebook.c

@@ -231,6 +231,8 @@ void rtgui_notebook_add(struct rtgui_notebook* notebook, const char* label, stru
 void rtgui_notebook_remove(struct rtgui_notebook* notebook, rt_uint16_t index)
 {
 	struct rtgui_notebook_tab tab;
+    rt_bool_t need_update = RT_FALSE;
+
 	RT_ASSERT(notebook != RT_NULL);
 
 	if (index < notebook->count)
@@ -244,8 +246,11 @@ void rtgui_notebook_remove(struct rtgui_notebook* notebook, rt_uint16_t index)
 		}
 		else
 		{
+            if (notebook->current == index)
+                need_update = RT_TRUE;
+
 			tab = notebook->childs[index];
-			for (;index < notebook->count - 1; index ++)
+			for (;index < notebook->count - 1; index++)
 			{
 				notebook->childs[index] = notebook->childs[index + 1];
 			}
@@ -255,16 +260,17 @@ void rtgui_notebook_remove(struct rtgui_notebook* notebook, rt_uint16_t index)
 				sizeof(struct rtgui_notebook_tab) * notebook->count);
 		}
 
-		// FIXME: do we really want to destroy it?
-		rtgui_widget_destroy(tab.widget);
 		rtgui_free(tab.title);
 
-		if (notebook->current == index)
+		if (need_update)
 		{
-			/* update tab */
 			if (notebook->current > notebook->count - 1)
 				notebook->current = notebook->count - 1;
+
+            rtgui_widget_hide(tab.widget);
+            rtgui_widget_show(notebook->childs[notebook->current].widget);
 			rtgui_widget_update(RTGUI_WIDGET(notebook));
+            rtgui_widget_set_parent(tab.widget, RT_NULL);
 		}
 	}
 }

components/rtgui/widgets/widget.c

@@ -74,7 +74,7 @@ static void _rtgui_widget_destructor(rtgui_widget_t *widget)
 {
 	if (widget == RT_NULL) return;
 
-	if (widget->parent != RT_NULL)
+	if (widget->parent != RT_NULL && RTGUI_IS_CONTAINER(widget->parent))
 	{
 		/* remove widget from parent's children list */
 		rtgui_list_remove(&(RTGUI_CONTAINER(widget->parent)->children), &(widget->sibling));
@@ -413,8 +413,7 @@ rt_bool_t rtgui_widget_onupdate_toplvl(struct rtgui_object *object, struct rtgui
 
 rt_bool_t rtgui_widget_event_handler(struct rtgui_object* object, rtgui_event_t* event)
 {
-	RT_ASSERT(object != RT_NULL);
-	RT_ASSERT(event != RT_NULL);
+	RTGUI_WIDGET_EVENT_HANDLER_PREPARE;
 
 	switch (event->type)
 	{
@@ -538,9 +537,10 @@ rt_bool_t rtgui_widget_onshow(struct rtgui_object *object, struct rtgui_event *e
 {
 	struct rtgui_widget *widget = RTGUI_WIDGET(object);
 
-	/* update the clip info of widget */
+    if (!RTGUI_WIDGET_IS_HIDE(RTGUI_WIDGET(object)))
+        return RT_FALSE;
+
 	RTGUI_WIDGET_UNHIDE(widget);
-	rtgui_widget_update_clip(widget);
 
 	if (widget->on_show != RT_NULL)
 		widget->on_show(RTGUI_OBJECT(widget), RT_NULL);
@@ -552,6 +552,9 @@ rt_bool_t rtgui_widget_onhide(struct rtgui_object *object, struct rtgui_event *e
 {
 	struct rtgui_widget *widget = RTGUI_WIDGET(object);
 
+    if (RTGUI_WIDGET_IS_HIDE(RTGUI_WIDGET(object)))
+        return RT_FALSE;
+
 	/* hide this widget */
 	RTGUI_WIDGET_HIDE(widget);
 

components/rtgui/widgets/window.c

@@ -594,16 +594,6 @@ void rtgui_win_set_rect(rtgui_win_t* win, rtgui_rect_t* rect)
 	}
 }
 
-#ifndef RTGUI_USING_SMALL_SIZE
-void rtgui_win_set_box(rtgui_win_t* win, rtgui_box_t* box)
-{
-	if (win == RT_NULL || box == RT_NULL) return;
-
-	rtgui_container_add_child(RTGUI_CONTAINER(win), RTGUI_WIDGET(box));
-	rtgui_widget_set_rect(RTGUI_WIDGET(box), &(RTGUI_WIDGET(win)->extent));
-}
-#endif
-
 void rtgui_win_set_onactivate(rtgui_win_t* win, rtgui_event_handler_ptr handler)
 {
 	if (win != RT_NULL)