foxygit / doom Log in
commit 0229ae9cfd2e90f587c08280e17833fd7cb72303
Merge: b81ba59d 787ab13a
Author:     Simon Howard <fraggle+github@gmail.com>
AuthorDate: Wed Jul 15 23:48:22 2015 -0400
Commit:     Simon Howard <fraggle+github@gmail.com>
CommitDate: Wed Jul 15 23:48:22 2015 -0400

    Merge pull request #580 from jmtd/aspect-ratio-screenshots

    Write out aspect-corrected 1600x1200 PNGs

    Doom's PCX screenshots are 320x200, which is in the wrong 16:10 aspect
    ratio and doesn't match the 4:3 aspect ratio that the screen is
    supposed to be displayed at. Within the game we stretch the screen to
    the correct aspect ratio, but screenshots are saved wrong.

    When the user has made the choice to use PNG screenshots instead of
    PCX, they've already made the decision to break with Vanilla
    compatibility, so there's an opportunity to fix the aspect ratio.
    To correct aspect ratio losslessly it is necessary to scale up by 5x
    (horizontally) / 6x (vertically). However, PNG files have much better
    compression than PCX so this isn't really an issue - a 1600x1200 PNG
    screenshot is actually smaller than a 320x200 PCX screenshot.

 src/v_video.c | 31 +++++++++++++++++++++++++++----
 1 file changed, 27 insertions(+), 4 deletions(-)