search

mitchcurtis / slate

Pixel Art Editor
GNU General Public License v3.0
1.07k stars 103 forks source link

Improve filling preformance #116

Closed wow2006 closed 5 years ago

wow2006 commented 5 years ago

replace QSet with QVector which is image.width() * image.heigh()

mitchcurtis commented 5 years ago

before (mm:ss:zzz): 19:21:052: attempting to fill starting with pixel at QPoint(381,435) ... 20:14:952: ... filled 1000000 pixels.

after (mm:ss:zzz): 22:34:641: attempting to fill starting with pixel at QPoint(570,507) ... 22:34:731: ... filled 1000000 pixels.

What a drastic improvement! Thank you! :D

mitchcurtis commented 5 years ago

Thanks for the comments @bjorn. Let's see if @wow2006 is interesting in tackling those. If not, I will create a task for them so they're not forgotten.