Revision: 38919
          
http://projects.blender.org/scm/viewvc.php?view=rev&root=bf-blender&revision=38919
Author:   campbellbarton
Date:     2011-08-02 08:12:50 +0000 (Tue, 02 Aug 2011)
Log Message:
-----------
no functional changes.
style edits, also renamed ndof_to_angle_axis --> ndof_to_axis_angle

Modified Paths:
--------------
    trunk/blender/source/blender/editors/space_view3d/view3d_edit.c
    trunk/blender/source/blender/editors/space_view3d/view3d_fly.c
    trunk/blender/source/blender/editors/space_view3d/view3d_intern.h

Modified: trunk/blender/source/blender/editors/space_view3d/view3d_edit.c
===================================================================
--- trunk/blender/source/blender/editors/space_view3d/view3d_edit.c     
2011-08-02 07:49:34 UTC (rev 38918)
+++ trunk/blender/source/blender/editors/space_view3d/view3d_edit.c     
2011-08-02 08:12:50 UTC (rev 38919)
@@ -930,7 +930,7 @@
 
 // NDOF utility functions
 // (should these functions live in this file?)
-float ndof_to_angle_axis(struct wmNDOFMotionData* ndof, float axis[3])
+float ndof_to_axis_angle(struct wmNDOFMotionData* ndof, float axis[3])
 {
        return ndof->dt * normalize_v3_v3(axis, ndof->rvec);
 }
@@ -940,7 +940,7 @@
        float axis[3];
        float angle;
 
-       angle= ndof_to_angle_axis(ndof, axis);
+       angle= ndof_to_axis_angle(ndof, axis);
        axis_angle_to_quat(q, axis, angle);
 }
 
@@ -1023,7 +1023,7 @@
                                mul_qt_qtqt(rot, rot, view_inv_conj);
        #else   // ---------------------------------------- Mike's revised 
version
                                float axis[3];
-                               float angle = rot_sensitivity * 
ndof_to_angle_axis(ndof, axis);
+                               float angle = rot_sensitivity * 
ndof_to_axis_angle(ndof, axis);
        
                                if (invert)
                                        angle = -angle;

Modified: trunk/blender/source/blender/editors/space_view3d/view3d_fly.c
===================================================================
--- trunk/blender/source/blender/editors/space_view3d/view3d_fly.c      
2011-08-02 07:49:34 UTC (rev 38918)
+++ trunk/blender/source/blender/editors/space_view3d/view3d_fly.c      
2011-08-02 08:12:50 UTC (rev 38919)
@@ -109,7 +109,7 @@
        wmKeyMap *keymap= WM_modalkeymap_get(keyconf, "View3D Fly Modal");
 
        /* this function is called for each spacetype, only needs to add map 
once */
-       if(keymap) return;
+       if (keymap) return;
 
        keymap= WM_modalkeymap_add(keyconf, "View3D Fly Modal", modal_items);
 
@@ -261,21 +261,21 @@
        fly->ar = CTX_wm_region(C);
        fly->scene= CTX_data_scene(C);
 
-       #ifdef NDOF_FLY_DEBUG
+#ifdef NDOF_FLY_DEBUG
        puts("\n-- fly begin --");
-       #endif
+#endif
 
-       if(fly->rv3d->persp==RV3D_CAMOB && fly->v3d->camera->id.lib) {
+       if (fly->rv3d->persp==RV3D_CAMOB && fly->v3d->camera->id.lib) {
                BKE_report(op->reports, RPT_ERROR, "Cannot fly a camera from an 
external library");
                return FALSE;
        }
 
-       if(fly->v3d->ob_centre) {
+       if (fly->v3d->ob_centre) {
                BKE_report(op->reports, RPT_ERROR, "Cannot fly when the view is 
locked to an object");
                return FALSE;
        }
 
-       if(fly->rv3d->persp==RV3D_CAMOB && fly->v3d->camera->constraints.first) 
{
+       if (fly->rv3d->persp==RV3D_CAMOB && 
fly->v3d->camera->constraints.first) {
                BKE_report(op->reports, RPT_ERROR, "Cannot fly an object with 
constraints");
                return FALSE;
        }
@@ -319,7 +319,7 @@
        fly->dist_backup= fly->rv3d->dist;
        if (fly->rv3d->persp==RV3D_CAMOB) {
                Object *ob_back;
-               if((U.uiflag & USER_CAM_LOCK_NO_PARENT)==0 && 
(fly->root_parent=fly->v3d->camera->parent)) {
+               if ((U.uiflag & USER_CAM_LOCK_NO_PARENT)==0 && 
(fly->root_parent=fly->v3d->camera->parent)) {
                        while(fly->root_parent->parent)
                                fly->root_parent= fly->root_parent->parent;
                        ob_back= fly->root_parent;
@@ -337,7 +337,8 @@
                negate_v3_v3(fly->rv3d->ofs, fly->v3d->camera->obmat[3]);
 
                fly->rv3d->dist=0.0;
-       } else {
+       }
+       else {
                /* perspective or ortho */
                if (fly->rv3d->persp==RV3D_ORTHO)
                        fly->rv3d->persp= RV3D_PERSP; /*if ortho projection, 
make perspective */
@@ -373,12 +374,12 @@
 
        float upvec[3];
 
-       if(fly->state == FLY_RUNNING)
+       if (fly->state == FLY_RUNNING)
                return OPERATOR_RUNNING_MODAL;
 
-       #ifdef NDOF_FLY_DEBUG
+#ifdef NDOF_FLY_DEBUG
        puts("\n-- fly end --");
-       #endif
+#endif
 
        WM_event_remove_timer(CTX_wm_manager(C), CTX_wm_window(C), fly->timer);
 
@@ -390,14 +391,14 @@
        /* Revert to original view? */
                if (fly->persp_backup==RV3D_CAMOB) { /* a camera view */
                        Object *ob_back;
-                       if(fly->root_parent)ob_back= fly->root_parent;
-                       else                            ob_back= 
fly->v3d->camera;
+                       ob_back= (fly->root_parent) ? fly->root_parent : 
fly->v3d->camera;
 
                        /* store the original camera loc and rot */
                        object_tfm_restore(ob_back, fly->obtfm);
 
                        DAG_id_tag_update(&ob_back->id, OB_RECALC_OB);
-               } else {
+               }
+               else {
                        /* Non Camera we need to reset the view back to the 
original location bacause the user canceled*/
                        copy_qt_qt(rv3d->viewquat, fly->rot_backup);
                        copy_v3_v3(rv3d->ofs, fly->ofs_backup);
@@ -422,13 +423,13 @@
        rv3d->rflag &= ~RV3D_NAVIGATING;
 //XXX2.5       BIF_view3d_previewrender_signal(fly->sa, PR_DBASE|PR_DISPRECT); 
/* not working at the moment not sure why */
 
-       if(fly->obtfm)
+       if (fly->obtfm)
                MEM_freeN(fly->obtfm);
 
-       if(fly->ndof)
+       if (fly->ndof)
                MEM_freeN(fly->ndof);
 
-       if(fly->state == FLY_CONFIRM) {
+       if (fly->state == FLY_CONFIRM) {
                MEM_freeN(fly);
                return OPERATOR_FINISHED;
        }
@@ -451,37 +452,37 @@
                // static const char* tag_name = "3D mouse position";
 
                wmNDOFMotionData* incoming_ndof = (wmNDOFMotionData*) 
event->customdata;
-               switch (incoming_ndof->progress)
-                       {
+               switch (incoming_ndof->progress) {
                        case P_STARTING:
                                // start keeping track of 3D mouse position
-                               #ifdef NDOF_FLY_DEBUG
+#ifdef NDOF_FLY_DEBUG
                                puts("start keeping track of 3D mouse 
position");
-                               #endif
+#endif
                                // fall through...
                        case P_IN_PROGRESS:
                                // update 3D mouse position
-                               #ifdef NDOF_FLY_DEBUG
+#ifdef NDOF_FLY_DEBUG
                                putchar('.'); fflush(stdout);
-                               #endif
-                               if (fly->ndof == NULL)
+#endif
+                               if (fly->ndof == NULL) {
                                        // fly->ndof = 
MEM_mallocN(sizeof(wmNDOFMotionData), tag_name);
                                        fly->ndof = 
MEM_dupallocN(incoming_ndof);
                                        // fly->ndof = 
malloc(sizeof(wmNDOFMotionData));
-                               else
+                               }
+                               else {
                                        memcpy(fly->ndof, incoming_ndof, 
sizeof(wmNDOFMotionData));
+                               }
                                break;
                        case P_FINISHING:
                                // stop keeping track of 3D mouse position
-                               #ifdef NDOF_FLY_DEBUG
+#ifdef NDOF_FLY_DEBUG
                                puts("stop keeping track of 3D mouse position");
-                               #endif
-                               if (fly->ndof)
-                                       {
+#endif
+                               if (fly->ndof) {
                                        MEM_freeN(fly->ndof);
                                        // free(fly->ndof);
                                        fly->ndof = NULL;
-                                       }
+                               }
                                /* update the time else the view will jump when 
2D mouse/timer resume */
                                fly->time_lastdraw= PIL_check_seconds_timer();
                                break;
@@ -511,7 +512,9 @@
                                /*Mouse wheel delays range from 0.5==slow to 
0.01==fast*/
                                time_wheel = 1.0f + (10.0f - (20.0f * 
MIN2(time_wheel, 0.5f))); /* 0-0.5 -> 0-5.0 */
 
-                               if (fly->speed<0.0f) fly->speed= 0.0f;
+                               if (fly->speed < 0.0f) {
+                                       fly->speed= 0.0f;
+                               }
                                else {
                                        if (event->shift)
                                                fly->speed += 
fly->grid*time_wheel * 0.1f;
@@ -530,7 +533,9 @@
                                fly->time_lastwheel = time_currwheel;
                                time_wheel = 1.0f + (10.0f - (20.0f * 
MIN2(time_wheel, 0.5f))); /* 0-0.5 -> 0-5.0 */
 
-                               if (fly->speed>0) fly->speed=0;
+                               if (fly->speed > 0.0f) {
+                                       fly->speed=0;
+                               }
                                else {
                                        if (event->shift)
                                                fly->speed-= 
fly->grid*time_wheel * 0.1f;
@@ -614,7 +619,7 @@
        ID *id_key;
 
        /* transform the parent or the camera? */
-       if(fly->root_parent) {
+       if (fly->root_parent) {
                Object *ob_update;
 
                float view_mat[4][4];
@@ -702,10 +707,10 @@
        unsigned char
        apply_rotation= 1; /* if the user presses shift they can look about 
without movinf the direction there looking*/
 
-       #ifdef NDOF_FLY_DEBUG
+#ifdef NDOF_FLY_DEBUG
        static unsigned int iteration = 1;
        printf("fly timer %d\n", iteration++);
-       #endif
+#endif
 
 
        xmargin= ar->winx/20.0f;
@@ -736,18 +741,18 @@
                 *
                 * the mouse moves isnt linear */
 
-               if(moffset[0]) {
+               if (moffset[0]) {
                        moffset[0] /= ar->winx - (xmargin*2);
                        moffset[0] *= fabsf(moffset[0]);
                }
 
-               if(moffset[1]) {
+               if (moffset[1]) {
                        moffset[1] /= ar->winy - (ymargin*2);
                        moffset[1] *= fabsf(moffset[1]);
                }
 
                /* Should we redraw? */
-               if(fly->speed != 0.0f || moffset[0] || moffset[1] || fly->zlock 
|| fly->xlock || dvec[0] || dvec[1] || dvec[2] ) {
+               if (fly->speed != 0.0f || moffset[0] || moffset[1] || 
fly->zlock || fly->xlock || dvec[0] || dvec[1] || dvec[2] ) {
                        float dvec_tmp[3];
                        double time_current; /*time how fast it takes for us to 
redraw, this is so simple scenes dont fly too fast */
                        float time_redraw;
@@ -781,8 +786,8 @@
 
                                mul_m3_v3(mat, dvec_tmp);
                                mul_v3_fl(dvec_tmp, time_redraw * 200.0f * 
fly->grid);
-
-                       } else {
+                       }
+                       else {
                                float roll; /* similar to the angle between the 
camera's up and the Z-up, but its very rough so just roll*/
 
                                /* rotate about the X axis- look up/down */
@@ -803,23 +808,24 @@
                                if (moffset[0]) {
 
                                        /* if we're upside down invert the 
moffset */
-                                       upvec[0]=0;
-                                       upvec[1]=1;
-                                       upvec[2]=0;
+                                       upvec[0]= 0.0f;
+                                       upvec[1]= 1.0f;
+                                       upvec[2]= 0.0f;
                                        mul_m3_v3(mat, upvec);
 
-                                       if(upvec[2] < 0.0f)
+                                       if (upvec[2] < 0.0f)
                                                moffset[0]= -moffset[0];
 
                                        /* make the lock vectors */
                                        if (fly->zlock) {
-                                               upvec[0]=0;
-                                               upvec[1]=0;
-                                               upvec[2]=1;
-                                       } else {
-                                               upvec[0]=0;
-                                               upvec[1]=1;
-                                               upvec[2]=0;
+                                               upvec[0]= 0.0f;
+                                               upvec[1]= 0.0f;
+                                               upvec[2]= 1.0f;
+                                       }
+                                       else {
+                                               upvec[0]= 0.0f;
+                                               upvec[1]= 1.0f;
+                                               upvec[2]= 0.0f;
                                                mul_m3_v3(mat, upvec);
                                        }
 
@@ -831,25 +837,26 @@
                                }
 
                                if (fly->zlock==2) {
-                                       upvec[0]=1;
-                                       upvec[1]=0;
-                                       upvec[2]=0;
+                                       upvec[0]= 1.0f;
+                                       upvec[1]= 0.0f;
+                                       upvec[2]= 0.0f;
                                        mul_m3_v3(mat, upvec);
 
                                        /*make sure we have some z rolling*/
                                        if (fabsf(upvec[2]) > 0.00001f) {
-                                               roll= upvec[2]*5;
-                                               upvec[0]=0; /*rotate the view 
about this axis*/
-                                               upvec[1]=0;
-                                               upvec[2]=1;
+                                               roll= upvec[2] * 5.0f;
+                                               upvec[0]= 0.0f; /*rotate the 
view about this axis*/
+                                               upvec[1]= 0.0f;
+                                               upvec[2]= 1.0f;
 
                                                mul_m3_v3(mat, upvec);
                                                axis_angle_to_quat( tmp_quat, 
upvec, roll*time_redraw_clamped*fly->zlock_momentum * FLY_ZUP_CORRECT_FAC); /* 
Rotate about the relative up vec */
                                                mul_qt_qtqt(rv3d->viewquat, 
rv3d->viewquat, tmp_quat);
 
                                                fly->zlock_momentum += 
FLY_ZUP_CORRECT_ACCEL;
-                                       } else {
-                                               fly->zlock=1; /* dont check 
until the view rotates again */
+                                       }
+                                       else {
+                                               fly->zlock= 1; /* dont check 
until the view rotates again */
                                                fly->zlock_momentum= 0.0f;
                                        }
                                }
@@ -860,8 +867,8 @@
                                        upvec[2]=1;
                                        mul_m3_v3(mat, upvec);
                                        /*make sure we have some z rolling*/
-                                       if (fabs(upvec[2]) > 0.00001) {
-                                               roll= upvec[2] * -5;
+                                       if (fabs(upvec[2]) > 0.00001f) {
+                                               roll= upvec[2] * -5.0f;
 
                                                upvec[0]= 1.0f; /*rotate the 
view about this axis*/
                                                upvec[1]= 0.0f;

@@ Diff output truncated at 10240 characters. @@
_______________________________________________
Bf-blender-cvs mailing list
[email protected]
http://lists.blender.org/mailman/listinfo/bf-blender-cvs

Reply via email to