php / php/php-src

Incorrect lineno for property and class const variance check

未关闭
#14,874 0 条评论 0 个 reaction 已指派 0 人 在 GitHub 查看

还没有人认领这个 Issue。

Category: Engine Feature Status: Verified
主要语言
C
星标
40.4k
派生
8.2k
平均合并
2 天 13 小时
30 天内合并 PR
96

描述

Description

The following code:

<?php

class C extends P {
    public const int X = 42;
}

class P {
    public const string X = 'X';
}

https://3v4l.org/IKY5H#v8.3.9

Resulted in this output:

Fatal error: Type of C::X must be compatible with P::X of type string in /in/ZAZYv on line 3

But I expected this output instead:

Fatal error: Type of C::X must be compatible with P::X of type string in /in/ZAZYv on line 4

Same goes for properties. From property hooks feedback:

https://github.com/php/php-src/pull/13455#discussion_r1499322256

PHP Version

*

Operating System

No response

贡献指南

打开贡献指南

从这里开始

  1. 先读完整个 Issue,再读项目的贡献指南。
  2. 在 Issue 下留言说明你要接手 —— 这能避免两个人做同样的事。
  3. Fork 仓库,在一个分支上完成修改。
  4. 提交 Pull Request,并在描述里引用这个 Issue 编号。

调研方向

复现类常量示例,并将报告的行与预期的第 4 行进行比较,然后调查 PHP 解释器中的方差检查诊断路径。检查 Property Hooks 反馈中提到的对应属性案例;当属性错误和类常量错误都报告不兼容子成员的声明行时,即视为完成。

由索引模型根据 Issue 内容生成。

评估

技术栈
c, php
领域
compilers
Issue 类型
缺陷
难度
4/5
预计耗时
3-5 天
活跃度
停滞
描述清晰度
基本清楚
新手友好度
35/100

把新 issue 发到你的邮箱

精选适合新手参与的 GitHub issue 摘要。