Page MenuHomePhabricator

In Remarkup, prevent "%%%" on a line from self-terminating the literal block
ClosedPublic

Authored by epriestley on Apr 23 2017, 1:01 PM.
Tags
None
Referenced Files
F18855614: D17772.id42753.diff
Sat, Nov 1, 5:54 AM
F18822911: D17772.id42735.diff
Thu, Oct 23, 9:14 AM
F18807852: D17772.diff
Sun, Oct 19, 3:31 AM
F18804152: D17772.id42753.diff
Sat, Oct 18, 6:22 AM
F18733575: D17772.id.diff
Sep 30 2025, 10:37 PM
F18646453: D17772.id42735.diff
Sep 19 2025, 10:03 AM
F18646444: D17772.id42753.diff
Sep 19 2025, 9:57 AM
F18646362: D17772.id.diff
Sep 19 2025, 9:51 AM
Subscribers
None

Details

Summary

Fixes T12621. Currently, %%% on a line by itself can both start and terminate a literal block. This is silly and presumably not what anyone ever intends.

Instead, if %%% started a block, don't let it terminate the block.

Test Plan
  • Added a failing test.
  • Made it pass.

Screen Shot 2017-04-23 at 5.56.18 AM.png (489×440 px, 28 KB)

Diff Detail

Repository
rPHU libphutil
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

This revision is now accepted and ready to land.Apr 23 2017, 6:20 PM
This revision was automatically updated to reflect the committed changes.