[Bf-blender-cvs] [a169a10] master: User Interface: error in own recent commit, replacing loop with memset

Campbell Barton noreply at git.blender.org
Mon Dec 2 23:26:28 CET 2013


Commit: a169a109e5636699fd11915d06257cf26105a105
Author: Campbell Barton
Date:   Tue Dec 3 09:10:55 2013 +1100
http://developer.blender.org/rBa169a109e5636699fd11915d06257cf26105a105

User Interface: error in own recent commit, replacing loop with memset

===================================================================

M	source/blender/editors/interface/interface_handlers.c

===================================================================

diff --git a/source/blender/editors/interface/interface_handlers.c b/source/blender/editors/interface/interface_handlers.c
index d0b0661..f903d06 100644
--- a/source/blender/editors/interface/interface_handlers.c
+++ b/source/blender/editors/interface/interface_handlers.c
@@ -1604,7 +1604,7 @@ void ui_button_text_password_hide(char password_str[UI_MAX_DRAW_STR], uiBut *but
 	}
 	else {
 		/* convert text to hidden test using asterisks (e.g. pass -> ****) */
-		int i, len = BLI_strlen_utf8(but->drawstr);
+		const size_t len = BLI_strlen_utf8(but->drawstr);
 
 		/* remap cursor positions */
 		if (but->pos >= 0) {
@@ -1617,7 +1617,7 @@ void ui_button_text_password_hide(char password_str[UI_MAX_DRAW_STR], uiBut *but
 		BLI_strncpy(password_str, but->drawstr, UI_MAX_DRAW_STR);
 
 		memset(but->drawstr, '*', len);
-		but->drawstr[i] = '\0';
+		but->drawstr[len] = '\0';
 	}
 }




More information about the Bf-blender-cvs mailing list