James,<div><br></div><div>Thanks for the timely feedback. </div><div><br></div><div><br></div><div>Ayush and team members,</div><div><br></div><div>Please make sure that we communicate clearly on list servs on our implementation plans and strategies before we submit patches. Will improve productivity and clarity of work.</div>
<div><br></div><div>Regards,</div><div><br></div><div>Manu </div><div><br></div><div><br><br><div class="gmail_quote">On Thu, Oct 21, 2010 at 7:38 AM, James Cameron <span dir="ltr"><<a href="mailto:quozl@laptop.org">quozl@laptop.org</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;"><div class="im">On Wed, Oct 20, 2010 at 08:00:16PM +0530, Ayush Goyal wrote:<br>
> +        if self.selmove:<br>
> +            size = self.pixmap_sel.get_size()<br>
> +            pix = gtk.gdk.Pixbuf(gtk.gdk.COLORSPACE_RGB, False, 8,<br>
> +                size[0], size[1])<br>
> +            pix.get_from_drawable(self.pixmap_sel,<br>
> +                gtk.gdk.colormap_get_system(), 0, 0, 0, 0, size[0], size[1])<br>
> +        else:<br>
> +            pix = gtk.gdk.Pixbuf(gtk.gdk.COLORSPACE_RGB, False, 8,<br>
> +                width, height)<br>
> +            pix.get_from_drawable(self.pixmap, gtk.gdk.colormap_get_system(),<br>
> +                0, 0, 0, 0, width, height)<br>
> +<br>
> +            pix = pix.flip(horizontal)<br>
<br>
</div>Are you sure this version of the patch works for you for both selected<br>
area and whole image?  The pix.flip is only on the whole image branch of<br>
the if.  Either this version of the patch does not work, or you did not<br>
test it, or I misunderstand.  Lay odds.  ;-)<br>
<div class="im"><br>
> +        if self.selmove:<br>
> +            self.pixmap_sel.draw_pixbuf(self.gc, pix, 0, 0, 0, 0,<br>
> +                size[0], size[1], dither=gtk.gdk.RGB_DITHER_NORMAL,<br>
> +                x_dither=0, y_dither=0)<br>
> +<br>
> +            self.pixmap_temp.draw_drawable(self.gc, self.pixmap, 0, 0, 0, 0,<br>
> +                width, height)<br>
> +            self.pixmap_temp.draw_drawable(self.gc, self.pixmap_sel,<br>
> +                0, 0, self.orig_x, self.orig_y, size[0], size[1])<br>
> +            self.pixmap_temp.draw_rectangle(self.gc_selection, False,<br>
> +                self.orig_x, self.orig_y, size[0], size[1])<br>
> +            self.pixmap_temp.draw_rectangle(self.gc_selection1, False,<br>
> +                self.orig_x - 1, self.orig_y - 1, size[0] + 2, size[1] + 2)<br>
> +<br>
> +        else:<br>
> +            self.pixmap.draw_pixbuf(self.gc, pix, 0, 0, 0, 0, width, height,<br>
> +                dither=gtk.gdk.RGB_DITHER_NORMAL, x_dither=0, y_dither=0)<br>
> +<br>
> +        self.queue_draw()<br>
> +        if not self.selmove:<br>
> +            self.enableUndo(widget)<br>
<br>
</div>I still think there need only be one check for self.selmove, and<br>
duplicated calls for pix.flip and queue_draw in each branch.  Whether<br>
this is done as a set of different functions or not is up to you ...<br>
<br>
... but I'm frustrated in this communication with you because I suggest<br>
n changes and see only a patch come back with n-m changes, with no<br>
discussion of why m changes were not adopted.<br>
<font color="#888888"><br>
--<br>
James Cameron<br>
<a href="http://quozl.linux.org.au/" target="_blank">http://quozl.linux.org.au/</a><br>
_______________________________________________<br>
Sugar-devel mailing list<br>
<a href="mailto:Sugar-devel@lists.sugarlabs.org">Sugar-devel@lists.sugarlabs.org</a><br>
<a href="http://lists.sugarlabs.org/listinfo/sugar-devel" target="_blank">http://lists.sugarlabs.org/listinfo/sugar-devel</a><br>
</font></blockquote></div><br></div>