[Bf-blender-cvs] SVN commit: /data/svn/bf-blender [45957] trunk/blender/source/blender: code cleanup: typos and set gcc attributes for string formatting.

Campbell Barton ideasman42 at gmail.com
Wed Apr 25 08:06:41 CEST 2012


Revision: 45957
          http://projects.blender.org/scm/viewvc.php?view=rev&root=bf-blender&revision=45957
Author:   campbellbarton
Date:     2012-04-25 06:06:40 +0000 (Wed, 25 Apr 2012)
Log Message:
-----------
code cleanup: typos and set gcc attributes for string formatting.

Modified Paths:
--------------
    trunk/blender/source/blender/blenkernel/BKE_report.h
    trunk/blender/source/blender/blenkernel/intern/anim_sys.c
    trunk/blender/source/blender/blenkernel/intern/cdderivedmesh.c
    trunk/blender/source/blender/blenkernel/intern/idprop.c
    trunk/blender/source/blender/blenkernel/intern/mesh.c
    trunk/blender/source/blender/blenkernel/intern/packedFile.c
    trunk/blender/source/blender/editors/include/UI_interface.h
    trunk/blender/source/blender/editors/transform/transform.c

Modified: trunk/blender/source/blender/blenkernel/BKE_report.h
===================================================================
--- trunk/blender/source/blender/blenkernel/BKE_report.h	2012-04-25 04:58:59 UTC (rev 45956)
+++ trunk/blender/source/blender/blenkernel/BKE_report.h	2012-04-25 06:06:40 UTC (rev 45957)
@@ -54,7 +54,11 @@
 ;
 
 void BKE_reports_prepend(ReportList *reports, const char *prepend);
-void BKE_reports_prependf(ReportList *reports, const char *prepend, ...);
+void BKE_reports_prependf(ReportList *reports, const char *prepend, ...)
+#ifdef __GNUC__
+__attribute__ ((format (printf, 2, 3)))
+#endif
+;
 
 ReportType BKE_report_print_level(ReportList *reports);
 void BKE_report_print_level_set(ReportList *reports, ReportType level);

Modified: trunk/blender/source/blender/blenkernel/intern/anim_sys.c
===================================================================
--- trunk/blender/source/blender/blenkernel/intern/anim_sys.c	2012-04-25 04:58:59 UTC (rev 45956)
+++ trunk/blender/source/blender/blenkernel/intern/anim_sys.c	2012-04-25 06:06:40 UTC (rev 45957)
@@ -2171,7 +2171,7 @@
  *
  * Current Status:
  *	- Currently (as of September 2009), overrides we haven't needed to (fully) implement overrides. 
- * 	  However, the code fo this is relatively harmless, so is left in the code for now.
+ * 	  However, the code for this is relatively harmless, so is left in the code for now.
  */
 
 /* Evaluation loop for evaluation animation data 

Modified: trunk/blender/source/blender/blenkernel/intern/cdderivedmesh.c
===================================================================
--- trunk/blender/source/blender/blenkernel/intern/cdderivedmesh.c	2012-04-25 04:58:59 UTC (rev 45956)
+++ trunk/blender/source/blender/blenkernel/intern/cdderivedmesh.c	2012-04-25 06:06:40 UTC (rev 45957)
@@ -230,7 +230,7 @@
 	int deformed= 0;
 
 	/* active modifiers means extra deformation, which can't be handled correct
-	 * on bith of PBVH and sculpt "layer" levels, so use PBVH only for internal brush
+	 * on birth of PBVH and sculpt "layer" levels, so use PBVH only for internal brush
 	 * stuff and show final DerivedMesh so user would see actual object shape */
 	deformed|= ob->sculpt->modifiers_active;
 

Modified: trunk/blender/source/blender/blenkernel/intern/idprop.c
===================================================================
--- trunk/blender/source/blender/blenkernel/intern/idprop.c	2012-04-25 04:58:59 UTC (rev 45956)
+++ trunk/blender/source/blender/blenkernel/intern/idprop.c	2012-04-25 06:06:40 UTC (rev 45957)
@@ -484,7 +484,7 @@
 }
 /*
  * replaces a property with the same name in a group, or adds 
- * it if the propery doesn't exist.
+ * it if the properly doesn't exist.
  */
 void IDP_ReplaceInGroup(IDProperty *group, IDProperty *prop)
 {

Modified: trunk/blender/source/blender/blenkernel/intern/mesh.c
===================================================================
--- trunk/blender/source/blender/blenkernel/intern/mesh.c	2012-04-25 04:58:59 UTC (rev 45956)
+++ trunk/blender/source/blender/blenkernel/intern/mesh.c	2012-04-25 06:06:40 UTC (rev 45957)
@@ -327,7 +327,7 @@
 
 			/* TODO - add some --debug-mesh option */
 			if (G.debug & G_DEBUG) {
-				/* note: this warning may be un-called for if we are inirializing the mesh for the
+				/* note: this warning may be un-called for if we are initializing the mesh for the
 				 * first time from bmesh, rather then giving a warning about this we could be smarter
 				 * and check if there was any data to begin with, for now just print the warning with
 				 * some info to help troubleshoot whats going on - campbell */
@@ -1104,7 +1104,7 @@
 			b++;
 		}
 		else {
-			/* XXX Theorically, we should be able to not do this, as no remaining poly
+			/* XXX Theoretically, we should be able to not do this, as no remaining poly
 			 *     should use any stripped loop. But for security's sake... */
 			new_idx[a] = -a;
 		}

Modified: trunk/blender/source/blender/blenkernel/intern/packedFile.c
===================================================================
--- trunk/blender/source/blender/blenkernel/intern/packedFile.c	2012-04-25 04:58:59 UTC (rev 45956)
+++ trunk/blender/source/blender/blenkernel/intern/packedFile.c	2012-04-25 06:06:40 UTC (rev 45957)
@@ -392,7 +392,7 @@
 /* unpackFile() looks at the existing files (abs_name, local_name) and a packed file.
  *
  * It returns a char *to the existing file name / new file name or NULL when
- * there was an error or when the user desides to cancel the operation.
+ * there was an error or when the user decides to cancel the operation.
  */
 
 char *unpackFile(ReportList *reports, const char *abs_name, const char *local_name, PackedFile *pf, int how)

Modified: trunk/blender/source/blender/editors/include/UI_interface.h
===================================================================
--- trunk/blender/source/blender/editors/include/UI_interface.h	2012-04-25 04:58:59 UTC (rev 45956)
+++ trunk/blender/source/blender/editors/include/UI_interface.h	2012-04-25 06:06:40 UTC (rev 45957)
@@ -322,10 +322,22 @@
 void uiPupMenuEnd(struct bContext *C, struct uiPopupMenu *head);
 struct uiLayout *uiPupMenuLayout(uiPopupMenu *head);
 
-void uiPupMenuOkee(struct bContext *C, const char *opname, const char *str, ...);
+void uiPupMenuOkee(struct bContext *C, const char *opname, const char *str, ...)
+#ifdef __GNUC__
+__attribute__ ((format (printf, 3, 4)))
+#endif
+;
 void uiPupMenuSaveOver(struct bContext *C, struct wmOperator *op, const char *filename);
-void uiPupMenuNotice(struct bContext *C, const char *str, ...);
-void uiPupMenuError(struct bContext *C, const char *str, ...);
+void uiPupMenuNotice(struct bContext *C, const char *str, ...)
+#ifdef __GNUC__
+__attribute__ ((format (printf, 2, 3)))
+#endif
+;
+void uiPupMenuError(struct bContext *C, const char *str, ...)
+#ifdef __GNUC__
+__attribute__ ((format (printf, 2, 3)))
+#endif
+;
 void uiPupMenuReports(struct bContext *C, struct ReportList *reports);
 void uiPupMenuInvoke(struct bContext *C, const char *idname); /* popup registered menu */
 

Modified: trunk/blender/source/blender/editors/transform/transform.c
===================================================================
--- trunk/blender/source/blender/editors/transform/transform.c	2012-04-25 04:58:59 UTC (rev 45956)
+++ trunk/blender/source/blender/editors/transform/transform.c	2012-04-25 06:06:40 UTC (rev 45957)
@@ -4788,7 +4788,7 @@
 					}
 
 					if (e_sel) {
-						/* warning if the UV's are not contiguiys, this will copy from the _wrong_ UVs
+						/* warning if the UV's are not contiguous, this will copy from the _wrong_ UVs
 						 * in fact whenever the face being copied is not 'f_copy' this can happen,
 						 * we could be a lot smarter about this but would need to deal with every UV channel or
 						 * add a way to mask out lauers when calling #BM_loop_interp_from_face() */




More information about the Bf-blender-cvs mailing list