| FazBrowse GitHub Viewer | Trending | | Home |
| Tools: [Download Repo ZIP] [Original HTTPS Page] |
|
Could you make an argument for why this should be removed, rather than documented? |
Sorry, something went wrong.
|
@radarhere Sure! _save_cjpeg(), the save counterpart of this function, was removed without much ceremony by you in #9084. This function might have had some value if it was automatically put into place if actual JPEG loading functions weren't available, but that has never been the case, so, like #9084, I'd call this unused. To match Hugo's research comment for #9084:
Also, there's this 2014 comment expressing desire to get rid of this function, so maybe a decade and change later that could be done. #731 (comment) EDIT: GitHub Code Search seems to find a single call from 10 years ago, from a seemingly abandoned mirror of https://launchpad.net/qa-regression-testing. |
Sorry, something went wrong.
|
I think _save_cjpeg() starting with an underscore is a difference. I would suggest a deprecation, rather than a removal. |
Sorry, something went wrong.
| Back | FazBrowse Home | New Git URL |
Sibling of #9901 (_save_netpbm() for GIFs) and #9084 (save_cjpeg() for JPEGs).
This PR proposes to remove the undocumented JpegImageFile.load_djpeg() function and the testing infrastructure for it.