<br><div class="gmail_quote"><blockquote class="gmail_quote" style="margin: 0pt 0pt 0pt 0.8ex; border-left: 1px solid rgb(204, 204, 204); padding-left: 1ex;"><div>
&gt; +        if (width &gt; preview_width) or (height &gt; preview_height):<br>
&gt; +            scale_x = float(width) / preview_width<br>
&gt; +            scale_y = float(height) / preview_height<br>
&gt; +            scale = max(scale_x, scale_y)<br>
&gt; +<br>
&gt; +            pixbuf = pixbuf.scale_simple(float(width) / scale, height / scale,<br>
&gt; +                                     gtk.gdk.INTERP_BILINEAR)<br> 
<br></div></blockquote><blockquote class="gmail_quote" style="margin: 0pt 0pt 0pt 0.8ex; border-left: 1px solid rgb(204, 204, 204); padding-left: 1ex;"><div>
</div>I guess you meant &quot;width / scale&quot;? scale_simple() expects integers.<br></blockquote><div><br>Yes<br> </div><blockquote class="gmail_quote" style="margin: 0pt 0pt 0pt 0.8ex; border-left: 1px solid rgb(204, 204, 204); padding-left: 1ex;">


If you use scale(), you can get rid of one pixbuf (currently you use up<br>
to three).<br>
<div><br></div></blockquote><div>Ok.<br><br> </div><blockquote class="gmail_quote" style="margin: 0pt 0pt 0pt 0.8ex; border-left: 1px solid rgb(204, 204, 204); padding-left: 1ex;"><div>
&gt; +        width, height = pixbuf.get_width(), pixbuf.get_height()<br>
<br>
&gt; +        pixbuf2 = gtk.gdk.Pixbuf(gtk.gdk.COLORSPACE_RGB, pixbuf.get_has_alpha(),<br>
&gt; +            pixbuf.get_bits_per_sample(), preview_width, preview_height)<br>
<br>
</div>This creates a border around an image that&#39;s smaller than the preview<br>
size, thus wasting disk space. Please use width and height instead.<br>
<br></blockquote></div><br>That isn&#39;t true. If the image doesn&#39;t have the right proportions, it&#39;s <span id="result_box" class="short_text"><span style="" title="">stretch when</span></span> displayed.<br><br>
<br>-- <br>Gonzalo Odiard<br>SugarLabs Argentina<br><br>