php.net |  support |  documentation |  report a bug |  advanced search |  search howto |  statistics |  random bug |  login

go to bug id or search bugs for

Doc Bug #40522 PATCH: reword encoding param note in mb-strrpos
Submitted: 2007-02-17 18:16 UTC Modified: 2007-02-25 18:14 UTC
From: danielc at analysisandsolutions dot com Assigned:
Status: Closed Package: Documentation problem
PHP Version: Irrelevant OS:
Private report: No CVE-ID: None
View Developer Edit
 [2007-02-17 18:16 UTC] danielc at analysisandsolutions dot com 
Description:
------------
The note for the encoding parameter in the mb-strrpos documentation is ambiguous to me.  It can be mistakenly interpreted as meaning that the parameter was added in 5.2 when in reality it's behavior was modified.

http://www.analysisandsolutions.com/php/mb-strrpos.encoding.explanation.diff

Patches

Pull Requests

History

AllCommentsChangesGit/SVN commitsRelated reports
 [2007-02-24 21:10 UTC] dmytton@php.net 
Your description in the patch was a little too verbose so I have reworded it myself.
 [2007-02-25 04:39 UTC] danielc at analysisandsolutions dot com 
Hi David:

Please switch "back" to "backward" in the phrase "for back compatibility".

In addition, I feel the reworded explanation that got committed is just juggling around the old explanation.  While this is clearer, it's not crystal clear.
 [2007-02-25 10:50 UTC] dmytton@php.net 
I don't feel the change "back" to "backward" is necessary. If you wish to submit a patch that rewords it differently but that is more concise than your original patch, that'd be great.
 [2007-02-25 14:19 UTC] danielc at analysisandsolutions dot com 
There is no such term as "back compatibility" in computer parlance.  The term is "backward compatibility" or "backwards compatibility":

          Google Hits   phpdoc grep
back:          71,100             1*
backward:   1,130,000             7
backwards:  1,160,000             8

* Your entry.
 [2007-02-25 14:31 UTC] danielc at analysisandsolutions dot com 
A less wordy patch:

http://www.analysisandsolutions.com/php/
mb-strrpos.encoding.explanation.v2.diff
 [2007-02-25 18:14 UTC] dmytton@php.net 
This bug has been fixed in the documentation's XML sources. Since the
online and downloadable versions of the documentation need some time
to get updated, we would like to ask you to be a bit patient.

Thank you for the report, and for helping us make our documentation better.
 
PHP Copyright © 2001-2025 The PHP Group
All rights reserved. 		Last updated: Wed Nov 19 17:00:01 2025 UTC