Page MenuHomePhabricator

Don't set `YYINITDEPTH`
ClosedPublic

Authored by joshuaspence on Nov 20 2015, 6:28 AM.
Tags
None
Referenced Files
Unknown Object (File)
Wed, Nov 27, 1:30 PM
Unknown Object (File)
Wed, Nov 27, 1:29 PM
Unknown Object (File)
Wed, Nov 27, 1:21 PM
Unknown Object (File)
Wed, Nov 27, 1:02 PM
Unknown Object (File)
Wed, Nov 27, 4:44 AM
Unknown Object (File)
Tue, Nov 26, 5:16 AM
Unknown Object (File)
Sat, Nov 23, 3:24 PM
Unknown Object (File)
Tue, Nov 19, 9:32 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
Tests Passed
Build Status
Buildable 9190
Build 10868: Run Core Tests
Build 10867: 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.