I'm interested, always have been since you showed it to me a few months ago.
However, my only 'annoyance' is that it's tied with SDL, ever considered removing the dependency for SDL, and support a few others? I think you were talking about doing that before... BTW have you losly based the API on Gtk? If not you could probably borrow a few ideas, I love Gtk and it's very easy to use. Also, being constructive have you thought about 'cleaning' it up a bit? -- You know a few comments here and there, such as
Code:
void rescale_checkbox(struct widget_t *widget) {
}
... why that's blank!? :\
Code:
void save_screenshot(SDL_Surface *screen) {
char filename[14];
FILE *t;
int x;
for(x = 0; x < 10000; x ++) {
sprintf(filename, "shot_%04i.bmp", x);
t = fopen(filename, "r");
if(!t) break;
fclose(t);
}
if(x == 10000) {
log_message(ERROR_TYPE_WARNING, 0, __FILE__, __LINE__,
"Screenshot not saved: shot_0000.bmp through 9999 seem to exist");
}
else {
if(SDL_SaveBMP(screen, filename)) {
log_message(ERROR_TYPE_WARNING, ERROR_FLAG_SDL, __FILE__,
__LINE__, "Can't save screenshot: \"%s\"", filename);
}
else {
log_message(ERROR_TYPE_LOG, 0, __FILE__, __LINE__,
"Screenshot saved: \"%s\"", filename);
}
}
}
Normally I'd say 'brute force for the win', however you could simply chuck a timestamp on the end of the filename. Someone could seriously have 4000 screenshots, considered defining MAX_SCREENSHOTS instead at least .
[off-topic]
How are you finding 64bit Debian? I've thought about changing from OpenSuSE to Debian but need a bit of persuasion
[/off-topic]