Menu

#250 Fix initialization of libpaper

v0.9.27
closed
nobody
None
1
2022-04-30
2022-04-29
No

I'm the upstream maintainer of libpaper. I am about to release a new version which requires paperinit() to be called before other functions. In fact, this was always required according to the documentation, it just didn't actually matter.

tuxpaint has a comment where it calls paperinit() in postscript_print.c that says: "FIXME: Should we do this at startup? -bjk 2007.06.25". Yes! We should!

I attach a patch which does it at startup. This works fine with libpaper 1.1.x, and will work with the new version 1.2 as well. That's the only change required to work with the new version.

Thanks for tuxpaint!

1 Attachments

Discussion

  • Reuben Thomas

    Reuben Thomas - 2022-04-29

    By the way, tuxpaint-config calls paperinit() before other functions already, so there's nothing to fix there.

     

    Last edit: Reuben Thomas 2022-04-29
  • William Kendrick

    • status: open --> closed
     
  • William Kendrick

    Applied! Thanks so much!

     

Log in to post a comment.