Page MenuHomePhabricator

Don't set `YYINITDEPTH`
ClosedPublic

Authored by joshuaspence on Nov 20 2015, 6:28 AM.
Tags
None
Referenced Files
F14018239: D14529.id35145.diff
Tue, Nov 5, 6:49 AM
F14015694: D14529.id35380.diff
Sun, Nov 3, 11:43 PM
F13992148: D14529.id.diff
Tue, Oct 22, 2:39 PM
Unknown Object (File)
Fri, Oct 11, 12:32 PM
Unknown Object (File)
Fri, Oct 11, 12:32 PM
Unknown Object (File)
Fri, Oct 11, 12:32 PM
Unknown Object (File)
Fri, Oct 11, 12:21 PM
Unknown Object (File)
Wed, Oct 9, 3:34 AM
Subscribers

Details

Summary

I have no idea why this was added originally, but it doesn't seem to be necessary anymore.

Test Plan

Ran arc unit --everything in rARC, rPHU and rP.

Diff Detail

Repository
rPHU libphutil
Branch
master
Lint
Lint Passed
Unit
No Test Coverage
Build Status
Buildable 8955
Build 10512: Run Core Tests
Build 10511: arc lint + arc unit

Event Timeline

joshuaspence retitled this revision from to Don't set `YYINITDEPTH`.
joshuaspence updated this object.
joshuaspence edited the test plan for this revision. (Show Details)
joshuaspence edited the test plan for this revision. (Show Details)
joshuaspence edited edge metadata.

Without this, does the parser handle this file correctly?

if (0) {
} else if (0) {
} else if (0) {
... add about 450 more clauses ...
} else {
}

Use the following snippet:

<?php

if (0) {
    // empty
} else if (1) {
    // empty
} else if (2) {
    // empty
} else if (3) {
    // empty
} else if (4) {
    // empty
} else if (5) {
    // empty
} else if (6) {
    // empty
} else if (7) {
    // empty
} else if (8) {
    // empty
} else if (9) {
    // empty
} else if (10) {
    // empty
} else if (11) {
    // empty
} else if (12) {
    // empty
} else if (13) {
    // empty
} else if (14) {
    // empty
} else if (15) {
    // empty
} else if (16) {
    // empty
} else if (17) {
    // empty
} else if (18) {
    // empty
} else if (19) {
    // empty
} else if (20) {
    // empty
} else if (21) {
    // empty
} else if (22) {
    // empty
} else if (23) {
    // empty
} else if (24) {
    // empty
} else if (25) {
    // empty
} else if (26) {
    // empty
} else if (27) {
    // empty
} else if (28) {
    // empty
} else if (29) {
    // empty
} else if (30) {
    // empty
} else if (31) {
    // empty
} else if (32) {
    // empty
} else if (33) {
    // empty
} else if (34) {
    // empty
} else if (35) {
    // empty
} else if (36) {
    // empty
} else if (37) {
    // empty
} else if (38) {
    // empty
} else if (39) {
    // empty
} else if (40) {
    // empty
} else if (41) {
    // empty
} else if (42) {
    // empty
} else if (43) {
    // empty
} else if (44) {
    // empty
} else if (45) {
    // empty
} else if (46) {
    // empty
} else if (47) {
    // empty
} else if (48) {
    // empty
} else if (49) {
    // empty
} else if (50) {
    // empty
} else if (51) {
    // empty
} else if (52) {
    // empty
} else if (53) {
    // empty
} else if (54) {
    // empty
} else if (55) {
    // empty
} else if (56) {
    // empty
} else if (57) {
    // empty
} else if (58) {
    // empty
} else if (59) {
    // empty
} else if (60) {
    // empty
} else if (61) {
    // empty
} else if (62) {
    // empty
} else if (63) {
    // empty
} else if (64) {
    // empty
} else if (65) {
    // empty
} else if (66) {
    // empty
} else if (67) {
    // empty
} else if (68) {
    // empty
} else if (69) {
    // empty
} else if (70) {
    // empty
} else if (71) {
    // empty
} else if (72) {
    // empty
} else if (73) {
    // empty
} else if (74) {
    // empty
} else if (75) {
    // empty
} else if (76) {
    // empty
} else if (77) {
    // empty
} else if (78) {
    // empty
} else if (79) {
    // empty
} else if (80) {
    // empty
} else if (81) {
    // empty
} else if (82) {
    // empty
} else if (83) {
    // empty
} else if (84) {
    // empty
} else if (85) {
    // empty
} else if (86) {
    // empty
} else if (87) {
    // empty
} else if (88) {
    // empty
} else if (89) {
    // empty
} else if (90) {
    // empty
} else if (91) {
    // empty
} else if (92) {
    // empty
} else if (93) {
    // empty
} else if (94) {
    // empty
} else if (95) {
    // empty
} else if (96) {
    // empty
} else if (97) {
    // empty
} else if (98) {
    // empty
} else if (99) {
    // empty
} else if (100) {
    // empty
} else if (101) {
    // empty
} else if (102) {
    // empty
} else if (103) {
    // empty
} else if (104) {
    // empty
} else if (105) {
    // empty
} else if (106) {
    // empty
} else if (107) {
    // empty
} else if (108) {
    // empty
} else if (109) {
    // empty
} else if (110) {
    // empty
} else if (111) {
    // empty
} else if (112) {
    // empty
} else if (113) {
    // empty
} else if (114) {
    // empty
} else if (115) {
    // empty
} else if (116) {
    // empty
} else if (117) {
    // empty
} else if (118) {
    // empty
} else if (119) {
    // empty
} else if (120) {
    // empty
} else if (121) {
    // empty
} else if (122) {
    // empty
} else if (123) {
    // empty
} else if (124) {
    // empty
} else if (125) {
    // empty
} else if (126) {
    // empty
} else if (127) {
    // empty
} else if (128) {
    // empty
} else if (129) {
    // empty
} else if (130) {
    // empty
} else if (131) {
    // empty
} else if (132) {
    // empty
} else if (133) {
    // empty
} else if (134) {
    // empty
} else if (135) {
    // empty
} else if (136) {
    // empty
} else if (137) {
    // empty
} else if (138) {
    // empty
} else if (139) {
    // empty
} else if (140) {
    // empty
} else if (141) {
    // empty
} else if (142) {
    // empty
} else if (143) {
    // empty
} else if (144) {
    // empty
} else if (145) {
    // empty
} else if (146) {
    // empty
} else if (147) {
    // empty
} else if (148) {
    // empty
} else if (149) {
    // empty
} else if (150) {
    // empty
} else if (151) {
    // empty
} else if (152) {
    // empty
} else if (153) {
    // empty
} else if (154) {
    // empty
} else if (155) {
    // empty
} else if (156) {
    // empty
} else if (157) {
    // empty
} else if (158) {
    // empty
} else if (159) {
    // empty
} else if (160) {
    // empty
} else if (161) {
    // empty
} else if (162) {
    // empty
} else if (163) {
    // empty
} else if (164) {
    // empty
} else if (165) {
    // empty
} else if (166) {
    // empty
} else if (167) {
    // empty
} else if (168) {
    // empty
} else if (169) {
    // empty
} else if (170) {
    // empty
} else if (171) {
    // empty
} else if (172) {
    // empty
} else if (173) {
    // empty
} else if (174) {
    // empty
} else if (175) {
    // empty
} else if (176) {
    // empty
} else if (177) {
    // empty
} else if (178) {
    // empty
} else if (179) {
    // empty
} else if (180) {
    // empty
} else if (181) {
    // empty
} else if (182) {
    // empty
} else if (183) {
    // empty
} else if (184) {
    // empty
} else if (185) {
    // empty
} else if (186) {
    // empty
} else if (187) {
    // empty
} else if (188) {
    // empty
} else if (189) {
    // empty
} else if (190) {
    // empty
} else if (191) {
    // empty
} else if (192) {
    // empty
} else if (193) {
    // empty
} else if (194) {
    // empty
} else if (195) {
    // empty
} else if (196) {
    // empty
} else if (197) {
    // empty
} else if (198) {
    // empty
} else if (199) {
    // empty
} else if (200) {
    // empty
} else if (201) {
    // empty
} else if (202) {
    // empty
} else if (203) {
    // empty
} else if (204) {
    // empty
} else if (205) {
    // empty
} else if (206) {
    // empty
} else if (207) {
    // empty
} else if (208) {
    // empty
} else if (209) {
    // empty
} else if (210) {
    // empty
} else if (211) {
    // empty
} else if (212) {
    // empty
} else if (213) {
    // empty
} else if (214) {
    // empty
} else if (215) {
    // empty
} else if (216) {
    // empty
} else if (217) {
    // empty
} else if (218) {
    // empty
} else if (219) {
    // empty
} else if (220) {
    // empty
} else if (221) {
    // empty
} else if (222) {
    // empty
} else if (223) {
    // empty
} else if (224) {
    // empty
} else if (225) {
    // empty
} else if (226) {
    // empty
} else if (227) {
    // empty
} else if (228) {
    // empty
} else if (229) {
    // empty
} else if (230) {
    // empty
} else if (231) {
    // empty
} else if (232) {
    // empty
} else if (233) {
    // empty
} else if (234) {
    // empty
} else if (235) {
    // empty
} else if (236) {
    // empty
} else if (237) {
    // empty
} else if (238) {
    // empty
} else if (239) {
    // empty
} else if (240) {
    // empty
} else if (241) {
    // empty
} else if (242) {
    // empty
} else if (243) {
    // empty
} else if (244) {
    // empty
} else if (245) {
    // empty
} else if (246) {
    // empty
} else if (247) {
    // empty
} else if (248) {
    // empty
} else if (249) {
    // empty
} else if (250) {
    // empty
} else if (251) {
    // empty
} else if (252) {
    // empty
} else if (253) {
    // empty
} else if (254) {
    // empty
} else if (255) {
    // empty
} else if (256) {
    // empty
} else if (257) {
    // empty
} else if (258) {
    // empty
} else if (259) {
    // empty
} else if (260) {
    // empty
} else if (261) {
    // empty
} else if (262) {
    // empty
} else if (263) {
    // empty
} else if (264) {
    // empty
} else if (265) {
    // empty
} else if (266) {
    // empty
} else if (267) {
    // empty
} else if (268) {
    // empty
} else if (269) {
    // empty
} else if (270) {
    // empty
} else if (271) {
    // empty
} else if (272) {
    // empty
} else if (273) {
    // empty
} else if (274) {
    // empty
} else if (275) {
    // empty
} else if (276) {
    // empty
} else if (277) {
    // empty
} else if (278) {
    // empty
} else if (279) {
    // empty
} else if (280) {
    // empty
} else if (281) {
    // empty
} else if (282) {
    // empty
} else if (283) {
    // empty
} else if (284) {
    // empty
} else if (285) {
    // empty
} else if (286) {
    // empty
} else if (287) {
    // empty
} else if (288) {
    // empty
} else if (289) {
    // empty
} else if (290) {
    // empty
} else if (291) {
    // empty
} else if (292) {
    // empty
} else if (293) {
    // empty
} else if (294) {
    // empty
} else if (295) {
    // empty
} else if (296) {
    // empty
} else if (297) {
    // empty
} else if (298) {
    // empty
} else if (299) {
    // empty
} else if (300) {
    // empty
} else if (301) {
    // empty
} else if (302) {
    // empty
} else if (303) {
    // empty
} else if (304) {
    // empty
} else if (305) {
    // empty
} else if (306) {
    // empty
} else if (307) {
    // empty
} else if (308) {
    // empty
} else if (309) {
    // empty
} else if (310) {
    // empty
} else if (311) {
    // empty
} else if (312) {
    // empty
} else if (313) {
    // empty
} else if (314) {
    // empty
} else if (315) {
    // empty
} else if (316) {
    // empty
} else if (317) {
    // empty
} else if (318) {
    // empty
} else if (319) {
    // empty
} else if (320) {
    // empty
} else if (321) {
    // empty
} else if (322) {
    // empty
} else if (323) {
    // empty
} else if (324) {
    // empty
} else if (325) {
    // empty
} else if (326) {
    // empty
} else if (327) {
    // empty
} else if (328) {
    // empty
} else if (329) {
    // empty
} else if (330) {
    // empty
} else if (331) {
    // empty
} else if (332) {
    // empty
} else if (333) {
    // empty
} else if (334) {
    // empty
} else if (335) {
    // empty
} else if (336) {
    // empty
} else if (337) {
    // empty
} else if (338) {
    // empty
} else if (339) {
    // empty
} else if (340) {
    // empty
} else if (341) {
    // empty
} else if (342) {
    // empty
} else if (343) {
    // empty
} else if (344) {
    // empty
} else if (345) {
    // empty
} else if (346) {
    // empty
} else if (347) {
    // empty
} else if (348) {
    // empty
} else if (349) {
    // empty
} else if (350) {
    // empty
} else if (351) {
    // empty
} else if (352) {
    // empty
} else if (353) {
    // empty
} else if (354) {
    // empty
} else if (355) {
    // empty
} else if (356) {
    // empty
} else if (357) {
    // empty
} else if (358) {
    // empty
} else if (359) {
    // empty
} else if (360) {
    // empty
} else if (361) {
    // empty
} else if (362) {
    // empty
} else if (363) {
    // empty
} else if (364) {
    // empty
} else if (365) {
    // empty
} else if (366) {
    // empty
} else if (367) {
    // empty
} else if (368) {
    // empty
} else if (369) {
    // empty
} else if (370) {
    // empty
} else if (371) {
    // empty
} else if (372) {
    // empty
} else if (373) {
    // empty
} else if (374) {
    // empty
} else if (375) {
    // empty
} else if (376) {
    // empty
} else if (377) {
    // empty
} else if (378) {
    // empty
} else if (379) {
    // empty
} else if (380) {
    // empty
} else if (381) {
    // empty
} else if (382) {
    // empty
} else if (383) {
    // empty
} else if (384) {
    // empty
} else if (385) {
    // empty
} else if (386) {
    // empty
} else if (387) {
    // empty
} else if (388) {
    // empty
} else if (389) {
    // empty
} else if (390) {
    // empty
} else if (391) {
    // empty
} else if (392) {
    // empty
} else if (393) {
    // empty
} else if (394) {
    // empty
} else if (395) {
    // empty
} else if (396) {
    // empty
} else if (397) {
    // empty
} else if (398) {
    // empty
} else if (399) {
    // empty
} else if (400) {
    // empty
} else if (401) {
    // empty
} else if (402) {
    // empty
} else if (403) {
    // empty
} else if (404) {
    // empty
} else if (405) {
    // empty
} else if (406) {
    // empty
} else if (407) {
    // empty
} else if (408) {
    // empty
} else if (409) {
    // empty
} else if (410) {
    // empty
} else if (411) {
    // empty
} else if (412) {
    // empty
} else if (413) {
    // empty
} else if (414) {
    // empty
} else if (415) {
    // empty
} else if (416) {
    // empty
} else if (417) {
    // empty
} else if (418) {
    // empty
} else if (419) {
    // empty
} else if (420) {
    // empty
} else if (421) {
    // empty
} else if (422) {
    // empty
} else if (423) {
    // empty
} else if (424) {
    // empty
} else if (425) {
    // empty
} else if (426) {
    // empty
} else if (427) {
    // empty
} else if (428) {
    // empty
} else if (429) {
    // empty
} else if (430) {
    // empty
} else if (431) {
    // empty
} else if (432) {
    // empty
} else if (433) {
    // empty
} else if (434) {
    // empty
} else if (435) {
    // empty
} else if (436) {
    // empty
} else if (437) {
    // empty
} else if (438) {
    // empty
} else if (439) {
    // empty
} else if (440) {
    // empty
} else if (441) {
    // empty
} else if (442) {
    // empty
} else if (443) {
    // empty
} else if (444) {
    // empty
} else if (445) {
    // empty
} else if (446) {
    // empty
} else if (447) {
    // empty
} else if (448) {
    // empty
} else if (449) {
    // empty
} else if (450) {
    // empty
} else if (451) {
    // empty
} else if (452) {
    // empty
} else if (453) {
    // empty
} else if (454) {
    // empty
} else if (455) {
    // empty
} else if (456) {
    // empty
} else if (457) {
    // empty
} else if (458) {
    // empty
} else if (459) {
    // empty
} else if (460) {
    // empty
} else if (461) {
    // empty
} else if (462) {
    // empty
} else if (463) {
    // empty
} else if (464) {
    // empty
} else if (465) {
    // empty
} else if (466) {
    // empty
} else if (467) {
    // empty
} else if (468) {
    // empty
} else if (469) {
    // empty
} else if (470) {
    // empty
} else if (471) {
    // empty
} else if (472) {
    // empty
} else if (473) {
    // empty
} else if (474) {
    // empty
} else if (475) {
    // empty
} else if (476) {
    // empty
} else if (477) {
    // empty
} else if (478) {
    // empty
} else if (479) {
    // empty
} else if (480) {
    // empty
} else if (481) {
    // empty
} else if (482) {
    // empty
} else if (483) {
    // empty
} else if (484) {
    // empty
} else if (485) {
    // empty
} else if (486) {
    // empty
} else if (487) {
    // empty
} else if (488) {
    // empty
} else if (489) {
    // empty
} else if (490) {
    // empty
} else if (491) {
    // empty
} else if (492) {
    // empty
} else if (493) {
    // empty
} else if (494) {
    // empty
} else if (495) {
    // empty
} else if (496) {
    // empty
} else if (497) {
    // empty
} else if (498) {
    // empty
} else if (499) {
    // empty
} else if (500) {
    // empty
} else {
    // empty
}

Both before and after yield the same result:

XHPAST Parse Error: memory exhausted on line 59
epriestley added a reviewer: epriestley.

haha, alright

I mean, that's bad, but I guess this is no worse.

This revision is now accepted and ready to land.Nov 30 2015, 8:14 PM
joshuaspence edited edge metadata.
  • Bump version
  • Add test case
This revision was automatically updated to reflect the committed changes.