diff options
author | Campbell Barton <ideasman42@gmail.com> | 2011-02-28 06:17:20 +0300 |
---|---|---|
committer | Campbell Barton <ideasman42@gmail.com> | 2011-02-28 06:17:20 +0300 |
commit | 3919c46a09aa243815e1c2306991ab880448ee71 (patch) | |
tree | afb994557c8b38b728c5b653f9fa10c9ba79c24c /source/blender/editors | |
parent | 1af13dea622eb230d488219b368ca721ad41b858 (diff) |
minor change for image invert operator
- dirty var wasn't being used.
- only call IMB_rect_from_float if there was previously a rect buffer.
Diffstat (limited to 'source/blender/editors')
-rw-r--r-- | source/blender/editors/space_image/image_ops.c | 33 |
1 files changed, 16 insertions, 17 deletions
diff --git a/source/blender/editors/space_image/image_ops.c b/source/blender/editors/space_image/image_ops.c index 93c891239f8..978c0739abb 100644 --- a/source/blender/editors/space_image/image_ops.c +++ b/source/blender/editors/space_image/image_ops.c @@ -1366,19 +1366,18 @@ static int image_invert_exec(bContext *C, wmOperator *op) { Image *ima= CTX_data_edit_image(C); ImBuf *ibuf= BKE_image_get_ibuf(ima, NULL); - + // flags indicate if this channel should be inverted - short r,g,b,a; - int i, dirty = 0; - + const short r= RNA_boolean_get(op->ptr, "invert_r"); + const short g= RNA_boolean_get(op->ptr, "invert_g"); + const short b= RNA_boolean_get(op->ptr, "invert_b"); + const short a= RNA_boolean_get(op->ptr, "invert_a"); + + int i; + if( ibuf == NULL) // TODO: this should actually never happen, but does for render-results -> cleanup return OPERATOR_CANCELLED; - - r = RNA_boolean_get(op->ptr, "invert_r"); - g = RNA_boolean_get(op->ptr, "invert_g"); - b = RNA_boolean_get(op->ptr, "invert_b"); - a = RNA_boolean_get(op->ptr, "invert_a"); - + /* TODO: make this into an IMB_invert_channels(ibuf,r,g,b,a) method!? */ if (ibuf->rect_float) { @@ -1389,8 +1388,10 @@ static int image_invert_exec(bContext *C, wmOperator *op) if( b ) fp[2] = 1.0f - fp[2]; if( a ) fp[3] = 1.0f - fp[3]; } - dirty = 1; - IMB_rect_from_float(ibuf); + + if(ibuf->rect) { + IMB_rect_from_float(ibuf); + } } else if(ibuf->rect) { @@ -1401,16 +1402,14 @@ static int image_invert_exec(bContext *C, wmOperator *op) if( b ) cp[2] = 255 - cp[2]; if( a ) cp[3] = 255 - cp[3]; } - dirty = 1; } - else + else { return OPERATOR_CANCELLED; + } - ibuf->userflags |= IB_BITMAPDIRTY; // mark as modified + ibuf->userflags |= IB_BITMAPDIRTY; WM_event_add_notifier(C, NC_IMAGE|NA_EDITED, ima); - return OPERATOR_FINISHED; - } void IMAGE_OT_invert(wmOperatorType *ot) |