code cleanup: typos and set gcc attributes for string formatting.
authorCampbell Barton <ideasman42@gmail.com>
Wed, 25 Apr 2012 06:06:40 +0000 (06:06 +0000)
committerCampbell Barton <ideasman42@gmail.com>
Wed, 25 Apr 2012 06:06:40 +0000 (06:06 +0000)
source/blender/blenkernel/BKE_report.h
source/blender/blenkernel/intern/anim_sys.c
source/blender/blenkernel/intern/cdderivedmesh.c
source/blender/blenkernel/intern/idprop.c
source/blender/blenkernel/intern/mesh.c
source/blender/blenkernel/intern/packedFile.c
source/blender/editors/include/UI_interface.h
source/blender/editors/transform/transform.c

index f630780649199dbf324cb11a898e5596eee112d8..aa625fab3b297f0de031bb4d97c252c9018908d5 100644 (file)
@@ -54,7 +54,11 @@ __attribute__ ((format (printf, 3, 4)))
 ;
 
 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);
index fc58799eb184447af77b41fa84399e2b5450fad6..75b9ae59e26ce49552326e23d1de3e661c5b48f2 100644 (file)
@@ -2171,7 +2171,7 @@ static void animsys_evaluate_overrides (PointerRNA *ptr, AnimData *adt)
  *
  * 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 
index 474aa84fec68b3cf0018c0397e4dfcb7986b3a19..4de7df098c3ce2bfa48bd7fab957c8843b798305 100644 (file)
@@ -230,7 +230,7 @@ static int can_pbvh_draw(Object *ob, DerivedMesh *dm)
        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;
 
index 1fee5cfa3590f6828003a57a0e3e01899ef90ecc..bb51325a6ef02201ded696a3ed30ea02330e6492 100644 (file)
@@ -484,7 +484,7 @@ void IDP_ReplaceGroupInGroup(IDProperty *dest, IDProperty *src)
 }
 /*
  * 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)
 {
index 7280af18493ecdc175a6c46831eb8a21f33d591f..514380dc980f3aa05031cfb84d46c51fdf516e50 100644 (file)
@@ -327,7 +327,7 @@ static void mesh_ensure_tessellation_customdata(Mesh *me)
 
                        /* 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 @@ void mesh_strip_loose_polysloops(Mesh *me)
                        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;
                }
index 8a940e339bc13a35d776c625de496afd7cf219f6..b55033b8493fd1d9ce82327f85a6016601bc9ed2 100644 (file)
@@ -392,7 +392,7 @@ int checkPackedFile(const char *filename, PackedFile *pf)
 /* 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)
index cdb2472706c17e28579226814f36121966c8b26e..d262c650e1cc653c358a0b0ae9eacc8cebbca041 100644 (file)
@@ -322,10 +322,22 @@ uiPopupMenu *uiPupMenuBegin(struct bContext *C, const char *title, int icon);
 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 */
 
index 6b46a7bce8ee4e21096b23ac14969cdac23f17fd..f3819a1b83caf6ca2105a2874c5c708810817058 100644 (file)
@@ -4788,7 +4788,7 @@ void projectSVData(TransInfo *t, int final)
                                        }
 
                                        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() */