Doc #77781 [NEW]: fopen() b flag recommendation no longer necessary?
| From: | martin at auswoeger dot com | Date: | Fri, 22 Mar 2019 12:04:12 +0000 |
| Subject: | Doc #77781 [NEW]: fopen() b flag recommendation no longer necessary? | ||
| Groups: | php.doc.bugs | ||
| Request: | Send a blank email to doc-bugs+get-16504@lists.php.net to get a copy of this message | ||
From: martin at auswoeger dot com
Operating system:
PHP version: Irrelevant
Package: Filesystem function related
Bug Type: Documentation Problem
Bug description:fopen() b flag recommendation no longer necessary?
Description:
------------
---
From manual page: https://php.net/function.fopen
---
The documentation for fopen() includes the following note:
For portability, it is strongly recommended that you always use the 'b'
flag when opening files with fopen().
However it seems that since PHP 4.3.2 the 'b' flag is already the
default and it should no longer be necessary.
Looking at the source code it also seems that the 'b' flag is not
necessary anymore:
-
https://github.com/php/php-src/blob/php-7.3.2/main/streams/plain_wrapper.c#L110-L114
-
https://github.com/php/php-src/commit/fa342d2404a9c809d0a182d80798da2abd3a98f7
I think this recommendation note should be removed.
--
Edit bug report at https://bugs.php.net/bug.php?id=77781&edit=1
--
Try a snapshot (PHP 5.4): https://bugs.php.net/fix.php?id=77781&r=trysnapshot54
Try a snapshot (PHP 5.5): https://bugs.php.net/fix.php?id=77781&r=trysnapshot55
Try a snapshot (trunk): https://bugs.php.net/fix.php?id=77781&r=trysnapshottrunk
Fixed in SVN: https://bugs.php.net/fix.php?id=77781&r=fixed
Fixed in release: https://bugs.php.net/fix.php?id=77781&r=alreadyfixed
Need backtrace: https://bugs.php.net/fix.php?id=77781&r=needtrace
Need Reproduce Script: https://bugs.php.net/fix.php?id=77781&r=needscript
Try newer version: https://bugs.php.net/fix.php?id=77781&r=oldversion
Not developer issue: https://bugs.php.net/fix.php?id=77781&r=support
Expected behavior: https://bugs.php.net/fix.php?id=77781&r=notwrong
Not enough info: https://bugs.php.net/fix.php?id=77781&r=notenoughinfo
Submitted twice: https://bugs.php.net/fix.php?id=77781&r=submittedtwice
register_globals: https://bugs.php.net/fix.php?id=77781&r=globals
PHP 4 support discontinued: https://bugs.php.net/fix.php?id=77781&r=php4
Daylight Savings: https://bugs.php.net/fix.php?id=77781&r=dst
IIS Stability: https://bugs.php.net/fix.php?id=77781&r=isapi
Install GNU Sed: https://bugs.php.net/fix.php?id=77781&r=gnused
Floating point limitations: https://bugs.php.net/fix.php?id=77781&r=float
No Zend Extensions: https://bugs.php.net/fix.php?id=77781&r=nozend
MySQL Configuration Error: https://bugs.php.net/fix.php?id=77781&r=mysqlcfg