Use a thread for ppm exporter about gource

[GoogleCodeExporter]

Good idea.

FYI ppm.cpp has changed a bit (not released yet) as I improved the performance 
by
doing less writes, so the patch will need some merging.

http://github.com/acaudwell/Gource/commit/7857f6fe0e5dd3e25034f7d6b31ec83fd3564e
14

Original comment by [email protected] on 18 Jan 2010 at 7:42

• Added labels: Type-Other
• Removed labels: Type-Defect

[GoogleCodeExporter]

[deleted comment]

[GoogleCodeExporter]

Here's a patch against current git. It's a bit more C++ oriented...

Original comment by [email protected] on 18 Jan 2010 at 8:40

Attachments:

• 0001-Use-a-thread-for-the-exporter.patch

[GoogleCodeExporter]

Nice. I will try it out later this week.

Original comment by [email protected] on 18 Jan 2010 at 8:48

• Changed state: Accepted

[GoogleCodeExporter]

Hi. I've merged your code into this branch:

http://github.com/acaudwell/Gource/tree/ppmthread

http://gource.googlecode.com/files/gource-0.24-beta1.tar.gz

I've changed the code to use the SDL_thread API since that is more portable (the
SDL_thread API is essentially identical). I made a few other changes like adding
double buffering, and doing the pixel flipping on the video card.

In my tests its about 2fps faster (11fps vs 9fps) with the thread than without 
using
this command line:

./gource --stop-at-end --output-ppm-stream - | ffmpeg -y -b 3000K -r 60 -f 
image2pipe
-vcodec ppm -i - -vcodec libx264 gource.mp4

(running against Gource.git)

It'd be good if you could have a look over changes before I add this to the 
master.

Cheers

Original comment by [email protected] on 28 Jan 2010 at 3:02

[GoogleCodeExporter]

I've updated the beta:

http://gource.googlecode.com/files/gource-0.24-beta2.tar.gz

Original comment by [email protected] on 31 Jan 2010 at 10:30

[GoogleCodeExporter]

Original comment by [email protected] on 31 Jan 2010 at 10:37

• Changed state: Fixed