php.net |  support |  documentation |  report a bug |  advanced search |  search howto |  statistics |  random bug |  login
Doc Bug #54769 Too much optional parameters in w32api_deftype()
Submitted: 2011-05-17 10:43 UTC Modified: 2011-05-19 14:11 UTC
From: gwarnants at gmail dot com Assigned: salathe (profile)
Status: Wont fix Package: Documentation problem
PHP Version: Irrelevant OS:
Private report: No CVE-ID: None
View Add Comment Developer Edit
Welcome! If you don't have a Git account, you can't do anything here.
You can add a comment by following this link or if you reported this bug, you can edit this bug over here.
(description)
Block user comment
Status: Assign to:
Package:
Bug Type:
Summary:
From: gwarnants at gmail dot com
New email:
PHP Version: OS:

 

 [2011-05-17 10:43 UTC] gwarnants at gmail dot com
Description:
------------
---
From manual page: http://www.php.net/function.w32api-deftype#Description
---

There is no reason the extra argument "..." to be written twice in the function protoype. Once is clear enough.

Regards,
Geoffray

Expected result:
----------------
bool w32api_deftype ( string $typename , string $member1_type , string $member1_name [, string $... ] )

Actual result:
--------------
bool w32api_deftype ( string $typename , string $member1_type , string $member1_name [, string $... [, string $... ]] )

Patches

Add a Patch

Pull Requests

Add a Pull Request

History

AllCommentsChangesGit/SVN commitsRelated reports
 [2011-05-19 14:11 UTC] salathe@php.net
-Status: Open +Status: Wont fix -Assigned To: +Assigned To: salathe
 [2011-05-19 14:11 UTC] salathe@php.net
The trailing parameters should be in pairs of $memberN_type and $memberN_name. 
Here having two $... makes sense. 

Also, as with your earlier rand() bug [1] we cannot make the prototype be like

    bool w32api_deftype ( string $typename , string $member1_type , string 
$member1_name [, string $..., string $...] )

[1] http://bugs.php.net/bug.php?id=54746
 
PHP Copyright © 2001-2024 The PHP Group
All rights reserved.
Last updated: Tue Mar 19 09:01:30 2024 UTC