• Massimo Valentini's avatar
    app: improvements in code executed a lot while painting. · 2aa4426d
    Massimo Valentini authored
    This keeps the same rectangle packing behaviour, so to behave exactly as
    before for what concerns batching the updates, but should be lighter
    when looping to find the first good rectangle to use.
    In rtree_insert(), some conditions in the if tests are implied by
    previous conditions. And therefore the 2 successive for loops are
    actually identical.
    In rtree_node_insert(), it is wrong/harmful to insert zero sized
    rectangles in the tree because they can never be selected and just make
    the list longer. So rtree_node_create() should just return NULL when w
    or h are 0.
    See bug 694917, comments 51 to 61.
gimpdisplayxfer.c 6.89 KB