Screenshot Example UP in the CODE Repository

[SG57]

http://www.psp-programming.com/code/index.php?id=c:screenshot

Just made it...  Take a look...

[Insert_Witty_Name]

Looks good.

Your link to the header file is messed up though, I see _ instead of / in the path.

Also think you should at least credit Shine for this as well.

[SG57]

Shine?  I gave credit to the real devleoper in the header and in the example...

it was Jason who made it...

Ill get right on that link, and i think its Yeldarbs fault because he said he was moving the header to a more permanent spot...

[Insert_Witty_Name]

http://forums.ps2dev.org/viewtopic.php?t=3112

12th post down - by Shine.

Jason's was a BMP/TGA version which didn't work.

The version you have is Shine's work, but edited by Jason (a few post below that link I gave).

[SG57]

Well i guess Jason is a little sly pooch...  ill edit it really quick...

[harleyg]

* You MUST have the Portable Networks Graphic Library (PNGLib) installed to complie this *  Created By: SG57

good work.

[Insert_Witty_Name]

Also, no offence, but this function is already implemented in libpng.

This line alone will take a screenshot:

Code:

saveImage("Screenshot.png", getVramDisplayBuffer(), SCREEN_WIDTH, SCREEN_HEIGHT, PSP_LINE_SIZE, 0);

I use the following code in AniMate to take screenshots when select is pressed.

[EDIT] Damn letter count limit in posts....
Link to Code: http://www.psp-programming.com/animate/screenshotcode.htm

Also you need to initialise the ScreenShotNumber variable somewhere at the start of your program with:

Code:

int ScreenShotNumber = 0;

The above code will auto-increment the screenshot number -- and unlike some other code snippets, will not overwrite any existing files (for example from a previous day of playing the game).

This code follows the KISS principal  

[SG57]

?  I did that completely...  And i know that's in the PNG Lib already, but it requires alot of parameters that is kinda of pointless if your capturing the entire screen, so why should you have to input how large the screen, is, just have the ACTUAL_WIDTH defined already without the need for it... Plus it requires the getVramDisplayBuffer() which is a pain to write each time when doing it.

[Insert_Witty_Name]

Eh?

It's one line versus a whole function. All you need to to is the SCREEN_WIDTH and SCREEN_HEIGHT.

getVramDisplayBuffer() is no problem at all... you don't have to do anything... ?

I Don't see your point, sorry

[SG57]

1. SCREEN_WIDTH
2. SCREEN_HEIGHT
3. getVramDisplayBuffer()
4. string as filename

Thats 4 things versus 1:

1. String as filename

Plus getVramDisplayBuffer() is just a hassle to remember as a parameter name...

Plus this is more of an example of how LUA specific functions can be converted...

[Insert_Witty_Name]

You're forgetting the fact that the code you're calling is wrapped up in a function with 50 lines of code or whatever....

It'd take all of two seconds to wrap my line up in a function that only accepts one parameter (like yours does) and it'd be less than 5 lines...

Don't get me wrong, I'm not being hostile, I just can't see the point of making a function that is IDENTICAL to a function you already have access to!