style cleanup
authorCampbell Barton <ideasman42@gmail.com>
Wed, 4 Jul 2012 20:47:12 +0000 (20:47 +0000)
committerCampbell Barton <ideasman42@gmail.com>
Wed, 4 Jul 2012 20:47:12 +0000 (20:47 +0000)
18 files changed:
source/blender/blenlib/BLI_args.h
source/blender/blenlib/intern/BLI_args.c
source/blender/collada/GeometryExporter.h
source/blender/collada/ImageExporter.cpp
source/blender/compositor/nodes/COM_LensDistortionNode.cpp
source/blender/compositor/operations/COM_ProjectorLensDistortionOperation.cpp
source/blender/compositor/operations/COM_ScreenLensDistortionOperation.cpp
source/blender/compositor/operations/COM_VariableSizeBokehBlurOperation.cpp
source/blender/compositor/operations/COM_WriteBufferOperation.cpp
source/blender/editors/space_view3d/drawarmature.c
source/blender/editors/space_view3d/drawvolume.c
source/blender/editors/space_view3d/view3d_edit.c
source/blender/editors/space_view3d/view3d_view.c
source/blender/editors/transform/transform.c
source/blender/editors/transform/transform_snap.c
source/blender/imbuf/intern/readimage.c
source/blender/nodes/texture/nodes/node_texture_viewer.c
source/blender/windowmanager/intern/wm_gesture.c

index 7a240e0e08a33ce14e08293dd6e7820ce2e1b782..6758fa399a4d7e182787be9cb4e996fbdba1b291 100644 (file)
 struct bArgs;
 typedef struct bArgs bArgs;
 
-/* returns the number of extra arguments consumed by the function. 0 is normal value, -1 stops parsing arguments, other negative indicates skip */
+/**
+ * Returns the number of extra arguments consumed by the function.
+ * -  0 is normal value,
+ * - -1 stops parsing arguments, other negative indicates skip
+ */
 typedef int (*BA_ArgCallback)(int argc, const char **argv, void *data);
 
 struct bArgs *BLI_argsInit(int argc, const char **argv);
 void BLI_argsFree(struct bArgs *ba);
 
-/* pass starts at 1, -1 means valid all the time
+/**
+ * Pass starts at 1, -1 means valid all the time
  * short_arg or long_arg can be null to specify no short or long versions
- * */
-void BLI_argsAdd(struct bArgs *ba, int pass, const char *short_arg, const char *long_arg, const char *doc, BA_ArgCallback cb, void *data);
-/* short_case and long_case specify if those arguments are case specific */
-void BLI_argsAddCase(struct bArgs *ba, int pass, const char *short_arg, int short_case, const char *long_arg, int long_case, const char *doc, BA_ArgCallback cb, void *data);
+ */
+void BLI_argsAdd(struct bArgs *ba, int pass,
+                 const char *short_arg, const char *long_arg,
+                 const char *doc, BA_ArgCallback cb, void *data);
+
+/**
+ * Short_case and long_case specify if those arguments are case specific
+ */
+void BLI_argsAddCase(struct bArgs *ba, int pass,
+                     const char *short_arg, int short_case,
+                     const char *long_arg, int long_case,
+                     const char *doc, BA_ArgCallback cb, void *data);
 
 void BLI_argsParse(struct bArgs *ba, int pass, BA_ArgCallback default_cb, void *data);
 
index 785c9f9f32b0c4d91da0adc2d140fcf40142279e..22b939485124b0ac8cc4437444768f0a0dfd387f 100644 (file)
@@ -173,7 +173,8 @@ static bArgDoc *internalDocs(struct bArgs *ba, const char *short_arg, const char
        return d;
 }
 
-static void internalAdd(struct bArgs *ba, const char *arg, int pass, int case_str, BA_ArgCallback cb, void *data, bArgDoc *d)
+static void internalAdd(struct bArgs *ba, const char *arg, int pass,
+                        int case_str, BA_ArgCallback cb, void *data, bArgDoc *d)
 {
        bArgument *a;
        bAKey *key;
@@ -182,8 +183,10 @@ static void internalAdd(struct bArgs *ba, const char *arg, int pass, int case_st
 
        if (a) {
                printf("WARNING: conflicting argument\n");
-               printf("\ttrying to add '%s' on pass %i, %scase sensitive\n", arg, pass, case_str == 1 ? "not " : "");
-               printf("\tconflict with '%s' on pass %i, %scase sensitive\n\n", a->key->arg, (int)a->key->pass, a->key->case_str == 1 ? "not " : "");
+               printf("\ttrying to add '%s' on pass %i, %scase sensitive\n",
+                      arg, pass, case_str == 1 ? "not " : "");
+               printf("\tconflict with '%s' on pass %i, %scase sensitive\n\n",
+                      a->key->arg, (int)a->key->pass, a->key->case_str == 1 ? "not " : "");
        }
 
        a = MEM_callocN(sizeof(bArgument), "bArgument");
@@ -201,7 +204,10 @@ static void internalAdd(struct bArgs *ba, const char *arg, int pass, int case_st
        BLI_ghash_insert(ba->items, key, a);
 }
 
-void BLI_argsAddCase(struct bArgs *ba, int pass, const char *short_arg, int short_case, const char *long_arg, int long_case, const char *doc, BA_ArgCallback cb, void *data)
+void BLI_argsAddCase(struct bArgs *ba, int pass,
+                     const char *short_arg, int short_case,
+                     const char *long_arg, int long_case,
+                     const char *doc, BA_ArgCallback cb, void *data)
 {
        bArgDoc *d = internalDocs(ba, short_arg, long_arg, doc);
 
@@ -214,7 +220,9 @@ void BLI_argsAddCase(struct bArgs *ba, int pass, const char *short_arg, int shor
 
 }
 
-void BLI_argsAdd(struct bArgs *ba, int pass, const char *short_arg, const char *long_arg, const char *doc, BA_ArgCallback cb, void *data)
+void BLI_argsAdd(struct bArgs *ba, int pass,
+                 const char *short_arg, const char *long_arg,
+                 const char *doc, BA_ArgCallback cb, void *data)
 {
        BLI_argsAddCase(ba, pass, short_arg, 0, long_arg, 0, doc, cb, data);
 }
index f14775b9f440517769dcf53de883668983af5a6c..7161bb751dd779677b3329c7e85f10830a064eea 100644 (file)
@@ -119,8 +119,7 @@ struct GeometryFunctor {
                LinkNode *node;
                for (node=export_set; node; node = node->next) {
                        Object *ob = (Object *)node->link;
-                       if (ob->type == OB_MESH)
-                       {
+                       if (ob->type == OB_MESH) {
                                f(ob);
                        }
                }
index f789cfe3660aa15ebb6eb2a11c1d9eeeb3be0708..b9e96445dcb750154b97d331df22357ec5a867fb 100644 (file)
@@ -135,7 +135,7 @@ void ImagesExporter::export_UV_Image(Image *image, bool use_copies)
 
                COLLADASW::Image img(COLLADABU::URI(COLLADABU::URI::nativePathToUri(export_path)), translated_name, translated_name); /* set name also to mNameNC. This helps other viewers import files exported from Blender better */
                img.add(mSW);
-               fprintf(stdout, "Collada export: Added image: %s\n",export_file);
+               fprintf(stdout, "Collada export: Added image: %s\n", export_file);
                mImages.push_back(translated_name);
        }
 }
@@ -145,7 +145,7 @@ void ImagesExporter::export_UV_Images()
        std::set<Image *> uv_textures;
        LinkNode *node;
        bool use_copies = this->export_settings->use_texture_copies;
-       for (node=this->export_settings->export_set; node; node=node->next) {
+       for (node = this->export_settings->export_set; node; node = node->next) {
                Object *ob = (Object *)node->link;
                if (ob->type == OB_MESH && ob->totcol) {
                        Mesh *me     = (Mesh *) ob->data;
@@ -160,7 +160,7 @@ void ImagesExporter::export_UV_Images()
                                                if (ima == NULL)
                                                        continue;
 
-                                               bool not_in_list = uv_textures.find(ima)==uv_textures.end();
+                                               bool not_in_list = uv_textures.find(ima) == uv_textures.end();
                                                if (not_in_list) {
                                                                uv_textures.insert(ima);
                                                                export_UV_Image(ima, use_copies);
@@ -177,7 +177,7 @@ bool ImagesExporter::hasImages(Scene *sce)
 {
        LinkNode *node;
        
-       for (node=this->export_settings->export_set; node; node=node->next) {
+       for (node = this->export_settings->export_set; node; node = node->next) {
                Object *ob = (Object *)node->link;
                int a;
                for (a = 0; a < ob->totcol; a++) {
index 4870e252b74f40918a4f039bb0d51745d6efbd24..3913b4ac2b6af9d712af9dc132e0e072399b0ad1 100644 (file)
@@ -50,8 +50,7 @@ void LensDistortionNode::convertToOperations(ExecutionSystem *graph, CompositorC
                ScreenLensDistortionOperation *operation = new ScreenLensDistortionOperation();
                operation->setbNode(editorNode);
                operation->setData(data);
-               if (!(this->getInputSocket(1)->isConnected() || this->getInputSocket(2)->isConnected())) 
-               {
+               if (!(this->getInputSocket(1)->isConnected() || this->getInputSocket(2)->isConnected())) {
                        // no nodes connected to the distortion and dispersion. We can precalculate some values
                        float distortion = ((const bNodeSocketValueFloat *)this->getInputSocket(1)->getbNodeSocket()->default_value)->value;
                        float dispersion = ((const bNodeSocketValueFloat *)this->getInputSocket(2)->getbNodeSocket()->default_value)->value;
index 74145c52a5de13b695143e4410120c7e0c57f81c..12674153bc8c76ddaba0be128e8b5bc3cd7de436 100644 (file)
@@ -79,10 +79,10 @@ bool ProjectorLensDistortionOperation::determineDependingAreaOfInterest(rcti *in
                newInput.xmin = input->xmin - this->m_kr2 - 2;
                newInput.xmax = input->xmax + this->m_kr2 + 2;
        } else {
-               newInput.xmin = input->xmin-7; //(0.25f*20*1)+2 == worse case dispersion
+               newInput.xmin = input->xmin - 7; //(0.25f*20*1)+2 == worse case dispersion
                newInput.ymin = input->ymin;
                newInput.ymax = input->ymax;
-               newInput.xmax = input->xmax+7; //(0.25f*20*1)+2 == worse case dispersion
+               newInput.xmax = input->xmax + 7; //(0.25f*20*1)+2 == worse case dispersion
        }
        return NodeOperation::determineDependingAreaOfInterest(&newInput, readOperation, output);
 }
index ea8483734d3452b79f4ec982c088b843b7db0b71..c1ad2efc384b0660006f1f7478cf06db31380b3a 100644 (file)
@@ -256,7 +256,7 @@ bool ScreenLensDistortionOperation::determineDependingAreaOfInterest(rcti *input
                UPDATE_INPUT;
                determineUV(coords, input->xmax, input->ymin);
                UPDATE_INPUT;
-               margin = (ABS(this->m_distortion)+this->m_dispersion)*MARGIN;
+               margin = (ABS(this->m_distortion) + this->m_dispersion) * MARGIN;
        } 
        else 
        {
@@ -283,7 +283,7 @@ bool ScreenLensDistortionOperation::determineDependingAreaOfInterest(rcti *input
                UPDATE_INPUT;
                determineUV(coords, input->xmax, input->ymin, 1.0f, 1.0f);
                UPDATE_INPUT;
-               margin=MARGIN;
+               margin = MARGIN;
        }
 
 #undef UPDATE_INPUT
index 46101f6355dc3b78b904cfa2a94ae98cef979b09..1368476e9b436e6b397bdaed7add2e237a8882dc 100644 (file)
@@ -74,7 +74,7 @@ void VariableSizeBokehBlurOperation::executePixel(float *color, int x, int y, Me
 
 #ifdef COM_DEFOCUS_SEARCH
        float search[4];
-       this->inputSearchProgram->read(search, x/InverseSearchRadiusOperation::DIVIDER, y/InverseSearchRadiusOperation::DIVIDER, inputBuffers, NULL);
+       this->inputSearchProgram->read(search, x/InverseSearchRadiusOperation::DIVIDER, y / InverseSearchRadiusOperation::DIVIDER, inputBuffers, NULL);
        int minx = search[0];
        int miny = search[1];
        int maxx = search[2];
@@ -168,10 +168,10 @@ bool VariableSizeBokehBlurOperation::determineDependingAreaOfInterest(rcti *inpu
        }
 #ifdef COM_DEFOCUS_SEARCH
        rcti searchInput;
-       searchInput.xmax = (input->xmax/InverseSearchRadiusOperation::DIVIDER)+1;
-       searchInput.xmin = (input->xmin/InverseSearchRadiusOperation::DIVIDER)-1;
-       searchInput.ymax = (input->ymax/InverseSearchRadiusOperation::DIVIDER)+1;
-       searchInput.ymin = (input->ymin/InverseSearchRadiusOperation::DIVIDER)-1;
+       searchInput.xmax = (input->xmax / InverseSearchRadiusOperation::DIVIDER) + 1;
+       searchInput.xmin = (input->xmin / InverseSearchRadiusOperation::DIVIDER) - 1;
+       searchInput.ymax = (input->ymax / InverseSearchRadiusOperation::DIVIDER) + 1;
+       searchInput.ymin = (input->ymin / InverseSearchRadiusOperation::DIVIDER) - 1;
        operation = getInputOperation(4);
        if (operation->determineDependingAreaOfInterest(&searchInput, readOperation, output) ) {
                return true;
index b8c7478b2f7f4e686017f5b4dd023e47049a6b14..191b21b921743173df486bb850312118ee252534 100644 (file)
@@ -64,7 +64,7 @@ void WriteBufferOperation::executeRegion(rcti *rect, unsigned int tileNumber, Me
        float *buffer = memoryBuffer->getBuffer();
        if (this->m_input->isComplex()) {
                bNode* bnode = this->m_input->getbNode();
-               if (bnode&& bnode->original) bnode->original->highlight++;
+               if (bnode && bnode->original) bnode->original->highlight++;
 
                void *data = this->m_input->initializeTileData(rect, memoryBuffers);
                int x1 = rect->xmin;
@@ -90,7 +90,7 @@ void WriteBufferOperation::executeRegion(rcti *rect, unsigned int tileNumber, Me
                        this->m_input->deinitializeTileData(rect, memoryBuffers, data);
                        data = NULL;
                }
-               if (bnode&& bnode->original) bnode->original->highlight++;
+               if (bnode && bnode->original) bnode->original->highlight++;
        }
        else {
                int x1 = rect->xmin;
@@ -144,7 +144,7 @@ void WriteBufferOperation::executeOpenCLRegion(OpenCLDevice* device, rcti *rect,
        clMemToCleanUp->push_back(clOutputBuffer);
        list<cl_kernel> *clKernelsToCleanUp = new list<cl_kernel>();
        bNode* bnode = this->m_input->getbNode();
-       if (bnode&& bnode->original) bnode->original->highlight++;
+       if (bnode && bnode->original) bnode->original->highlight++;
 
        this->m_input->executeOpenCL(device, outputBuffer, clOutputBuffer, inputMemoryBuffers, clMemToCleanUp, clKernelsToCleanUp);
 
@@ -163,7 +163,7 @@ void WriteBufferOperation::executeOpenCLRegion(OpenCLDevice* device, rcti *rect,
        
        this->getMemoryProxy()->getBuffer()->copyContentFrom(outputBuffer);
 
-       if (bnode&& bnode->original) bnode->original->highlight++;
+       if (bnode && bnode->original) bnode->original->highlight++;
        // STEP 4
 
        
index 33aada7b8110fb1dd051db4d3c38f766ff6a00cf..cdaf7d5690f9092bc60ef0c3e61ddfebd94ac180 100644 (file)
@@ -709,9 +709,9 @@ static void draw_sphere_bone_dist(float smat[][4], float imat[][4], bPoseChannel
                glBegin(GL_QUAD_STRIP);
                
                for (a = 0; a < 16; a++) {
-                       vec[0] = -*(si + a) * dirvec[0] + *(co + a) * norvec[0];
-                       vec[1] = -*(si + a) * dirvec[1] + *(co + a) * norvec[1];
-                       vec[2] = -*(si + a) * dirvec[2] + *(co + a) * norvec[2];
+                       vec[0] = -si[a] * dirvec[0] + co[a] * norvec[0];
+                       vec[1] = -si[a] * dirvec[1] + co[a] * norvec[1];
+                       vec[2] = -si[a] * dirvec[2] + co[a] * norvec[2];
 
                        madd_v3_v3v3fl(vec1, headvec, vec, head);
                        madd_v3_v3v3fl(vec2, headvec, vec, head + dist);
@@ -723,9 +723,9 @@ static void draw_sphere_bone_dist(float smat[][4], float imat[][4], bPoseChannel
                }
                
                for (a = 15; a >= 0; a--) {
-                       vec[0] = *(si + a) * dirvec[0] + *(co + a) * norvec[0];
-                       vec[1] = *(si + a) * dirvec[1] + *(co + a) * norvec[1];
-                       vec[2] = *(si + a) * dirvec[2] + *(co + a) * norvec[2];
+                       vec[0] = si[a] * dirvec[0] + co[a] * norvec[0];
+                       vec[1] = si[a] * dirvec[1] + co[a] * norvec[1];
+                       vec[2] = si[a] * dirvec[2] + co[a] * norvec[2];
 
                        madd_v3_v3v3fl(vec1, tailvec, vec, tail);
                        madd_v3_v3v3fl(vec2, tailvec, vec, tail + dist);
@@ -737,9 +737,9 @@ static void draw_sphere_bone_dist(float smat[][4], float imat[][4], bPoseChannel
                }
                /* make it cyclic... */
                
-               vec[0] = -*(si) * dirvec[0] + *(co) * norvec[0];
-               vec[1] = -*(si) * dirvec[1] + *(co) * norvec[1];
-               vec[2] = -*(si) * dirvec[2] + *(co) * norvec[2];
+               vec[0] = -si[0] * dirvec[0] + co[0] * norvec[0];
+               vec[1] = -si[0] * dirvec[1] + co[0] * norvec[1];
+               vec[2] = -si[0] * dirvec[2] + co[0] * norvec[2];
 
                madd_v3_v3v3fl(vec1, headvec, vec, head);
                madd_v3_v3v3fl(vec2, headvec, vec, head + dist);
index 7d39a89a130f47ec8dedaff62e7c71e31dc9e803..60b3b911b1b6fc2fbf8d6b89db935980bfc40b5f 100644 (file)
@@ -373,7 +373,7 @@ void draw_volume(ARegion *ar, GPUTexture *tex, float min[3], float max[3], int r
 
        /* d0 = (viewnormal[0]*cv[i][0] + viewnormal[1]*cv[i][1] + viewnormal[2]*cv[i][2]); */ /* UNUSED */
        ds = (ABS(viewnormal[0]) * size[0] + ABS(viewnormal[1]) * size[1] + ABS(viewnormal[2]) * size[2]);
-       dd = ds/96.f;
+       dd = ds / 96.f;
        n = 0;
        good_index = i;
 
index cc3d2d383b84f8e89209bd5d02638433a1c9815e..218aa71d621f2d6acccc4e9ee7e38d8871e34758 100644 (file)
@@ -2873,7 +2873,7 @@ static int viewnumpad_exec(bContext *C, wmOperator *op)
                        break;
 
                case RV3D_VIEW_BACK:
-                       axis_set_view(C, v3d, ar, 0.0, 0.0, (float)-cos(M_PI / 4.0), (float)-cos(M_PI / 4.0),
+                       axis_set_view(C, v3d, ar, 0.0, 0.0, -cosf(M_PI / 4.0), -cosf(M_PI / 4.0),
                                      viewnum, nextperspo, align_active);
                        break;
 
@@ -2888,7 +2888,7 @@ static int viewnumpad_exec(bContext *C, wmOperator *op)
                        break;
 
                case RV3D_VIEW_FRONT:
-                       axis_set_view(C, v3d, ar, (float)cos(M_PI / 4.0), (float)-sin(M_PI / 4.0), 0.0, 0.0,
+                       axis_set_view(C, v3d, ar, cosf(M_PI / 4.0), -sinf(M_PI / 4.0), 0.0, 0.0,
                                      viewnum, nextperspo, align_active);
                        break;
 
index 4782b95ba1c7321ca409f443464e1bcaa76706d6..b435084755258c1d97b3ed65b7cc76eb6ca51416 100644 (file)
@@ -1125,7 +1125,7 @@ int ED_view3d_lock(RegionView3D *rv3d)
                        break;
 
                case RV3D_VIEW_BACK:
-                       QUATSET(rv3d->viewquat, 0.0, 0.0, (float)-cos(M_PI / 4.0), (float)-cos(M_PI / 4.0));
+                       QUATSET(rv3d->viewquat, 0.0, 0.0, -cosf(M_PI / 4.0), -cosf(M_PI / 4.0));
                        break;
 
                case RV3D_VIEW_LEFT:
@@ -1137,7 +1137,7 @@ int ED_view3d_lock(RegionView3D *rv3d)
                        break;
 
                case RV3D_VIEW_FRONT:
-                       QUATSET(rv3d->viewquat, (float)cos(M_PI / 4.0), (float)-sin(M_PI / 4.0), 0.0, 0.0);
+                       QUATSET(rv3d->viewquat, (float)cos(M_PI / 4.0), -sinf(M_PI / 4.0), 0.0, 0.0);
                        break;
 
                case RV3D_VIEW_RIGHT:
index 164c1dd7c1276ec3be23c0821da480f0e95cc736..44860258c89afc9472a0c4ec22dc9872c3cd86fd 100644 (file)
@@ -5114,7 +5114,7 @@ void projectSVData(TransInfo *t, int final)
 
                                }
                                
-                               if(!affected)
+                               if (!affected)
                                        continue;
 
                                /* only loop data, no vertex data since that contains shape keys,
index c49e1715f34544aeb705105041db0b067fa5c6df..3847087111ecd520a82e58ea06588e6cf91bfbd7 100644 (file)
@@ -463,12 +463,10 @@ static void initSnappingMode(TransInfo *t)
                        t->tsnap.mode = SCE_SNAP_MODE_INCREMENT;
                }
        }
-       else if (t->spacetype == SPACE_NODE)
-       {
+       else if (t->spacetype == SPACE_NODE) {
                setSnappingCallback(t);
                
-               if (t->tsnap.applySnap != NULL)
-               {
+               if (t->tsnap.applySnap != NULL) {
                        t->tsnap.modeSelect = SNAP_NOT_SELECTED;
                }
                else {
index e78c4a558430d3f69cde493eaa766c40b37968cb..cb6cc2c082b4f943b6df629c526c04b4f2e51746 100644 (file)
@@ -90,7 +90,7 @@ ImBuf *IMB_loadifffile(int file, int flags, const char *descr)
        size = BLI_file_descriptor_size(file);
 
        mem = mmap(NULL, size, PROT_READ, MAP_SHARED, file, 0);
-       if (mem == (unsigned char *)-1) {
+       if (mem == (unsigned char *) -1) {
                fprintf(stderr, "%s: couldn't get mapping %s\n", __func__, descr);
                return NULL;
        }
@@ -178,7 +178,7 @@ static void imb_loadtilefile(ImBuf *ibuf, int file, int tx, int ty, unsigned int
        size = BLI_file_descriptor_size(file);
 
        mem = mmap(NULL, size, PROT_READ, MAP_SHARED, file, 0);
-       if (mem == (unsigned char *)-1) {
+       if (mem == (unsigned char *) -1) {
                fprintf(stderr, "Couldn't get memory mapping for %s\n", ibuf->cachename);
                return;
        }
index 03a94d1df675d2ed05676d420629b3b7053239e9..bc31bef59d8884e6070e17c2b0366ee508d00ba0 100644 (file)
 #include "NOD_texture.h"
 #include <math.h>
 
-static bNodeSocketTemplate inputs[]= {
+static bNodeSocketTemplate inputs[] = {
        { SOCK_RGBA, 1, N_("Color"), 1.0f, 0.0f, 0.0f, 1.0f },
        { -1, 0, "" }
 };
-static bNodeSocketTemplate outputs[]= {
+static bNodeSocketTemplate outputs[] = {
        { -1, 0, "" }
 };
 
index e4b7d1c5d5fccf18cd5e2a035c44071bbe83eebd..b280b979280efeb7f0d586e1879a620c690d9742 100644 (file)
@@ -224,7 +224,7 @@ static void wm_gesture_draw_circle(wmGesture *gt)
        glutil_draw_lined_arc(0.0, M_PI * 2.0, rect->xmax, 40);
        
        glDisable(GL_LINE_STIPPLE);
-       glTranslatef((float)-rect->xmin, (float)-rect->ymin, 0.0f);
+       glTranslatef(-rect->xmin, -rect->ymin, 0.0f);
        
 }