rfc:deprecate-png-jpeg-2wbmp

PHP RFC: Deprecate png2wbmp() and jpeg2wbmp()

Introduction

png2wbmp() and jpeg2wbmp() are the only direct image format conversion functions available in ext/gd which makes them rather special, particularly as libgd doesn't offer any such functions 1). Furthermore WBMP 2) has been invented to support WAP 3), which appears to be mostly obsolete nowadays.

Proposal

Therefore I propose to deprecate the png2wbmp() and jpeg2wbmp() functions as of PHP 7.2, and to remove them as of PHP 8.0. During the deprecation phase calling either function should emit an E_DEPRECATED error.

The main benefit in doing so would be easier maintainance of the libgd bindings (_php_image_convert() alone accounts for rougly 200 lines). An additional benefit would be to have a slightly cleaner and less confusing API (Why are there no other conversion functions? Why don't these functions follow the general “image” prefix rule?)

Backward Incompatible Changes

Obviously, code still using png2wbmp() or jpeg2wbmp() would break. However, fallback functions could easily implemented in userland. The supposedly long deprecation period should give users ample time to cater to the removal.

Proposed PHP Version(s)

  • Deprecation as of next PHP 7.x
  • Removal as of next major (supposedly PHP 8.0)

Open Issues

Currently none.

Unaffected PHP Functionality

imagewbmp() and image2wbmp() are not subject of this RFC, and as such their functionality will remain unmodified.

Proposed Voting Choices

Deprecate and remove the functions: yes or no.

This RFC requires a 50%+1 majority.

Voting ends on 2016-11-13, 22:00 UTC.

Deprecate png2wbmp() and jpeg2wbmp()
Real name Yes No
ajf (ajf)  
bwoebi (bwoebi)  
cmb (cmb)  
colinodell (colinodell)  
daverandom (daverandom)  
galvao (galvao)  
guilhermeblanco (guilhermeblanco)  
jhdxr (jhdxr)  
kalle (kalle)  
kguest (kguest)  
levim (levim)  
mbeccati (mbeccati)  
mgocobachi (mgocobachi)  
mrook (mrook)  
ocramius (ocramius)  
pierrick (pierrick)  
sammyk (sammyk)  
santiagolizardo (santiagolizardo)  
svpernova09 (svpernova09)  
zimt (zimt)  
Final result: 20 0
This poll has been closed.

Patches and Tests

PR implementing the deprecation: https://github.com/php/php-src/pull/2164.

Implementation

  1. PR #2164 has been merged into master (what is supposed to become PHP-7.2) with https://github.com/php/php-src/commit/189bbb58
  2. The deprecation and removal has been documented with http://svn.php.net/viewvc?view=revision&revision=341003
  3. The language specification is not affected by this RFC

References

1)
instead they are defined in the libgd binding using libgd's image reading and writing functions
2)
Wireless Application Protocol Bitmap Format
3)
Wireless Application Protocol
rfc/deprecate-png-jpeg-2wbmp.txt · Last modified: 2019/01/28 13:52 by cmb