Revision: 24024 http://projects.blender.org/plugins/scmsvn/viewcvs.php?view=rev&root=bf-blender&revision=24024 Author: blendix Date: 2009-10-20 23:05:22 +0200 (Tue, 20 Oct 2009)
Log Message: ----------- Region post redraw is now split up in a view space and pixel space part. This fixes a bug where transform help line drawing would not work with view clipping and mess up the z-buffer. This avoids the transform code having to figure out what kind of opengl state is enabled and disable it temporarily. Modified Paths: -------------- trunk/blender/source/blender/editors/include/ED_space_api.h trunk/blender/source/blender/editors/mesh/loopcut.c trunk/blender/source/blender/editors/space_api/spacetypes.c trunk/blender/source/blender/editors/space_image/image_draw.c trunk/blender/source/blender/editors/space_image/image_ops.c trunk/blender/source/blender/editors/space_image/space_image.c trunk/blender/source/blender/editors/space_view3d/view3d_draw.c trunk/blender/source/blender/editors/transform/transform.c trunk/blender/source/blender/editors/transform/transform.h trunk/blender/source/blender/editors/transform/transform_generics.c Modified: trunk/blender/source/blender/editors/include/ED_space_api.h =================================================================== --- trunk/blender/source/blender/editors/include/ED_space_api.h 2009-10-20 20:59:02 UTC (rev 24023) +++ trunk/blender/source/blender/editors/include/ED_space_api.h 2009-10-20 21:05:22 UTC (rev 24024) @@ -60,8 +60,8 @@ void ED_file_init(void); void ED_file_exit(void); -#define REGION_DRAW_PRE 1 -#define REGION_DRAW_POST 0 +#define REGION_DRAW_POST_VIEW 0 +#define REGION_DRAW_POST_PIXEL 1 void *ED_region_draw_cb_activate(struct ARegionType *, void (*draw)(const struct bContext *, struct ARegion *, void *), Modified: trunk/blender/source/blender/editors/mesh/loopcut.c =================================================================== --- trunk/blender/source/blender/editors/mesh/loopcut.c 2009-10-20 20:59:02 UTC (rev 24023) +++ trunk/blender/source/blender/editors/mesh/loopcut.c 2009-10-20 21:05:22 UTC (rev 24024) @@ -301,7 +301,7 @@ /* assign the drawing handle for drawing preview line... */ lcd->ar= CTX_wm_region(C); - lcd->draw_handle= ED_region_draw_cb_activate(lcd->ar->type, ringsel_draw, lcd, REGION_DRAW_POST); + lcd->draw_handle= ED_region_draw_cb_activate(lcd->ar->type, ringsel_draw, lcd, REGION_DRAW_POST_VIEW); lcd->ob = CTX_data_edit_object(C); lcd->em= BKE_mesh_get_editmesh((Mesh *)lcd->ob->data); lcd->extend = do_cut ? 0 : RNA_boolean_get(op->ptr, "extend"); Modified: trunk/blender/source/blender/editors/space_api/spacetypes.c =================================================================== --- trunk/blender/source/blender/editors/space_api/spacetypes.c 2009-10-20 20:59:02 UTC (rev 24023) +++ trunk/blender/source/blender/editors/space_api/spacetypes.c 2009-10-20 21:05:22 UTC (rev 24024) @@ -151,10 +151,6 @@ /* ********************** custom drawcall api ***************** */ -/* type */ -#define REGION_DRAW_PRE 1 -#define REGION_DRAW_POST 0 - typedef struct RegionDrawCB { struct RegionDrawCB *next, *prev; Modified: trunk/blender/source/blender/editors/space_image/image_draw.c =================================================================== --- trunk/blender/source/blender/editors/space_image/image_draw.c 2009-10-20 20:59:02 UTC (rev 24023) +++ trunk/blender/source/blender/editors/space_image/image_draw.c 2009-10-20 21:05:22 UTC (rev 24024) @@ -173,8 +173,6 @@ { char str[256]; int ofs; - - ED_region_pixelspace(ar); ofs= sprintf(str, "X: %d Y: %d ", x, y); if(cp) Modified: trunk/blender/source/blender/editors/space_image/image_ops.c =================================================================== --- trunk/blender/source/blender/editors/space_image/image_ops.c 2009-10-20 20:59:02 UTC (rev 24023) +++ trunk/blender/source/blender/editors/space_image/image_ops.c 2009-10-20 21:05:22 UTC (rev 24024) @@ -1539,7 +1539,7 @@ info= MEM_callocN(sizeof(ImageSampleInfo), "ImageSampleInfo"); info->art= ar->type; - info->draw_handle = ED_region_draw_cb_activate(ar->type, sample_draw, info, REGION_DRAW_POST); + info->draw_handle = ED_region_draw_cb_activate(ar->type, sample_draw, info, REGION_DRAW_POST_PIXEL); op->customdata= info; sample_apply(C, op, event); Modified: trunk/blender/source/blender/editors/space_image/space_image.c =================================================================== --- trunk/blender/source/blender/editors/space_image/space_image.c 2009-10-20 20:59:02 UTC (rev 24023) +++ trunk/blender/source/blender/editors/space_image/space_image.c 2009-10-20 21:05:22 UTC (rev 24024) @@ -431,15 +431,17 @@ UI_view2d_view_ortho(C, v2d); draw_uvedit_main(sima, ar, scene, obedit); - ED_region_draw_cb_draw(C, ar, REGION_DRAW_POST); + ED_region_draw_cb_draw(C, ar, REGION_DRAW_POST_VIEW); /* Grease Pencil too (in addition to UV's) */ draw_image_grease_pencil((bContext *)C, 1); UI_view2d_view_restore(C); - + /* draw Grease Pencil - screen space only */ draw_image_grease_pencil((bContext *)C, 0); + + ED_region_draw_cb_draw(C, ar, REGION_DRAW_POST_PIXEL); /* scrollers? */ /*scrollers= UI_view2d_scrollers_calc(C, v2d, V2D_UNIT_VALUES, V2D_GRID_CLAMP, V2D_ARG_DUMMY, V2D_ARG_DUMMY); Modified: trunk/blender/source/blender/editors/space_view3d/view3d_draw.c =================================================================== --- trunk/blender/source/blender/editors/space_view3d/view3d_draw.c 2009-10-20 20:59:02 UTC (rev 24023) +++ trunk/blender/source/blender/editors/space_view3d/view3d_draw.c 2009-10-20 21:05:22 UTC (rev 24024) @@ -2056,7 +2056,7 @@ view3d_update_depths(ar, v3d); } - ED_region_draw_cb_draw(C, ar, REGION_DRAW_POST); + ED_region_draw_cb_draw(C, ar, REGION_DRAW_POST_VIEW); // REEB_draw(); @@ -2123,6 +2123,8 @@ ob= OBACT; if(U.uiflag & USER_DRAWVIEWINFO) draw_selected_name(scene, ob, v3d); + + ED_region_draw_cb_draw(C, ar, REGION_DRAW_POST_PIXEL); /* XXX here was the blockhandlers for floating panels */ Modified: trunk/blender/source/blender/editors/transform/transform.c =================================================================== --- trunk/blender/source/blender/editors/transform/transform.c 2009-10-20 20:59:02 UTC (rev 24023) +++ trunk/blender/source/blender/editors/transform/transform.c 2009-10-20 21:05:22 UTC (rev 24024) @@ -1124,15 +1124,8 @@ projectFloatView(t, vecrot, cent); // no overflow in extreme cases - glDisable(GL_DEPTH_TEST); - - glMatrixMode(GL_PROJECTION); glPushMatrix(); - glMatrixMode(GL_MODELVIEW); - glPushMatrix(); - ED_region_pixelspace(t->ar); - switch(t->helpline) { case HLP_SPRING: @@ -1238,22 +1231,23 @@ } } - glMatrixMode(GL_PROJECTION); glPopMatrix(); - glMatrixMode(GL_MODELVIEW); - glPopMatrix(); - - glEnable(GL_DEPTH_TEST); } } -void drawTransform(const struct bContext *C, struct ARegion *ar, void *arg) +void drawTransformView(const struct bContext *C, struct ARegion *ar, void *arg) { TransInfo *t = arg; drawConstraint(C, t); drawPropCircle(C, t); drawSnapping(C, t); +} + +void drawTransformPixel(const struct bContext *C, struct ARegion *ar, void *arg) +{ + TransInfo *t = arg; + drawHelpline(C, t); } @@ -1373,11 +1367,13 @@ //calc_manipulator_stats(curarea); initTransformOrientation(C, t); - t->draw_handle = ED_region_draw_cb_activate(t->ar->type, drawTransform, t, REGION_DRAW_POST); + t->draw_handle_view = ED_region_draw_cb_activate(t->ar->type, drawTransformView, t, REGION_DRAW_POST_VIEW); + t->draw_handle_pixel = ED_region_draw_cb_activate(t->ar->type, drawTransformPixel, t, REGION_DRAW_POST_PIXEL); } else if(t->spacetype == SPACE_IMAGE) { Mat3One(t->spacemtx); - t->draw_handle = ED_region_draw_cb_activate(t->ar->type, drawTransform, t, REGION_DRAW_POST); + t->draw_handle_view = ED_region_draw_cb_activate(t->ar->type, drawTransformView, t, REGION_DRAW_POST_VIEW); + t->draw_handle_pixel = ED_region_draw_cb_activate(t->ar->type, drawTransformPixel, t, REGION_DRAW_POST_PIXEL); } else Mat3One(t->spacemtx); Modified: trunk/blender/source/blender/editors/transform/transform.h =================================================================== --- trunk/blender/source/blender/editors/transform/transform.h 2009-10-20 20:59:02 UTC (rev 24023) +++ trunk/blender/source/blender/editors/transform/transform.h 2009-10-20 21:05:22 UTC (rev 24024) @@ -302,7 +302,8 @@ struct wmTimer *animtimer; short mval[2]; /* current mouse position */ struct Object *obedit; - void *draw_handle; + void *draw_handle_view; + void *draw_handle_pixel; } TransInfo; Modified: trunk/blender/source/blender/editors/transform/transform_generics.c =================================================================== --- trunk/blender/source/blender/editors/transform/transform_generics.c 2009-10-20 20:59:02 UTC (rev 24023) +++ trunk/blender/source/blender/editors/transform/transform_generics.c 2009-10-20 21:05:22 UTC (rev 24024) @@ -1045,10 +1045,10 @@ { TransData *td; - if (t->draw_handle) - { - ED_region_draw_cb_exit(t->ar->type, t->draw_handle); - } + if (t->draw_handle_view) + ED_region_draw_cb_exit(t->ar->type, t->draw_handle_view); + if (t->draw_handle_pixel) + ED_region_draw_cb_exit(t->ar->type, t->draw_handle_pixel); if (t->customFree) { _______________________________________________ Bf-blender-cvs mailing list Bf-blender-cvs@blender.org http://lists.blender.org/mailman/listinfo/bf-blender-cvs