ID:               22047
 Updated by:       [EMAIL PROTECTED]
 Reported By:      [EMAIL PROTECTED]
 Status:           Wont fix
 Bug Type:         *General Issues
 Operating System: Linux
 PHP Version:      4.3.0
 New Comment:

It's fine as it is -> wont fix.


Previous Comments:
------------------------------------------------------------------------

[2003-02-04 11:48:17] [EMAIL PROTECTED]

A NOTICE definitely is NOT sufficient.

Remember many setups running older PHP software set
"error_reporting = E_ALL & ~E_NOTICE", because the
amount of notices on undefined array indices and
undefined variables can be quite overwhelming --
and it is precisely these setups that are likely
to be bitten by this problem.

Redefinition of a constant should be a WARNING,
I must insist.

------------------------------------------------------------------------

[2003-02-04 10:27:54] [EMAIL PROTECTED]

A notice is fine, we're not going to change this as it might break
scripts out there.

------------------------------------------------------------------------

[2003-02-04 10:20:52] [EMAIL PROTECTED]

Good, thanks.

Let me clarify -

PHP does log a NOTICE such as

[04-Feb-2003 17:18:22] PHP Notice:
  Constant T_LIST already defined
  in x.php on line 3

but this definitely needs to be a WARNING
or even an ERROR.

-Tomas

------------------------------------------------------------------------

[2003-02-04 10:13:58] [EMAIL PROTECTED]

That is a good idea, thought it did that. Assigned.

------------------------------------------------------------------------

[2003-02-04 10:09:56] [EMAIL PROTECTED]

Hmm.  I'm convinced PHP should at least file a big fat
warning upon any attempted redefinition of a reserved
constant name, instead of ignoring the instruction
entirely.

Identifiers such as T_LIST are very often used in enums.

------------------------------------------------------------------------

The remainder of the comments for this report are too long. To view
the rest of the comments, please view the bug report online at
    http://bugs.php.net/22047

-- 
Edit this bug report at http://bugs.php.net/?id=22047&edit=1

Reply via email to