Deprecating ldap_sort the right way

Author: Côme Bernigaud

ext/ldap/ldap.c

@@ -2001,8 +2001,6 @@ PHP_FUNCTION(ldap_sort)
 	size_t sflen;
 	zend_resource *le;
 
-    php_error_docref(NULL, E_DEPRECATED, "Usage of ldap_sort is deprecated and will be removed in one of the next PHP-versions");
-
 	if (zend_parse_parameters(ZEND_NUM_ARGS(), "rrs", &link, &result, &sortfilter, &sflen) != SUCCESS) {
 		RETURN_FALSE;
 	}
@@ -3279,7 +3277,7 @@ const zend_function_entry ldap_functions[] = {
 	PHP_FE(ldap_err2str,								arginfo_ldap_err2str)
 	PHP_FE(ldap_error,									arginfo_ldap_resource)
 	PHP_FE(ldap_compare,								arginfo_ldap_compare)
-	PHP_FE(ldap_sort,									arginfo_ldap_sort)
+	PHP_DEP_FE(ldap_sort,									arginfo_ldap_sort)
 
 #if (LDAP_API_VERSION > 2000) || HAVE_NSLDAP || HAVE_ORALDAP
 	PHP_FE(ldap_rename,									arginfo_ldap_rename)

ext/ldap/tests/ldap_sort_basic.phpt

@@ -51,7 +51,7 @@ ldap_delete($link, "cn=userE,$base");
 remove_dummy_data($link, $base);
 ?>
 --EXPECTF--
-Deprecated: ldap_sort(): Usage of ldap_sort is deprecated and will be removed in one of the next PHP-versions in %s.php on line %d
+Deprecated: Function ldap_sort() is deprecated in %s.php on line %d
 bool(true)
 array(7) {
   ["count"]=>

ext/ldap/tests/ldap_sort_error.phpt

@@ -18,27 +18,27 @@ var_dump(ldap_sort($link, $link, "sn"));
 ?>
 ===DONE===
 --EXPECTF--
-Deprecated: ldap_sort(): Usage of ldap_sort is deprecated and will be removed in one of the next PHP-versions in %s.php on line %d
+Deprecated: Function ldap_sort() is deprecated in %s.php on line %d
 
 Warning: ldap_sort() expects exactly 3 parameters, 1 given in %s on line %d
 bool(false)
 
-Deprecated: ldap_sort(): Usage of ldap_sort is deprecated and will be removed in one of the next PHP-versions in %s.php on line %d
+Deprecated: Function ldap_sort() is deprecated in %s.php on line %d
 
 Warning: ldap_sort() expects exactly 3 parameters, 2 given in %s on line %d
 bool(false)
 
-Deprecated: ldap_sort(): Usage of ldap_sort is deprecated and will be removed in one of the next PHP-versions in %s.php on line %d
+Deprecated: Function ldap_sort() is deprecated in %s.php on line %d
 
 Warning: ldap_sort() expects exactly 3 parameters, 4 given in %s on line %d
 bool(false)
 
-Deprecated: ldap_sort(): Usage of ldap_sort is deprecated and will be removed in one of the next PHP-versions in %s.php on line %d
+Deprecated: Function ldap_sort() is deprecated in %s.php on line %d
 
 Warning: ldap_sort() expects parameter 3 to be %binary_string_optional%, resource given in %s on line %d
 bool(false)
 
-Deprecated: ldap_sort(): Usage of ldap_sort is deprecated and will be removed in one of the next PHP-versions in %s.php on line %d
+Deprecated: Function ldap_sort() is deprecated in %s.php on line %d
 
 Warning: ldap_sort(): Supplied resource is not a valid ldap result resource in %s on line %d
 bool(false)

ext/ldap/tests/ldap_sort_variation.phpt

@@ -51,7 +51,7 @@ ldap_delete($link, "cn=userE,$base");
 remove_dummy_data($link, $base);
 ?>
 --EXPECTF--
-Deprecated: ldap_sort(): Usage of ldap_sort is deprecated and will be removed in one of the next PHP-versions in %s.php on line %d
+Deprecated: Function ldap_sort() is deprecated in %s.php on line %d
 bool(true)
 array(7) {
   ["count"]=>