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
F18733575: D17772.id.diff
Tue, Sep 30, 10:37 PM
F18646453: D17772.id42735.diff
Fri, Sep 19, 10:03 AM
F18646444: D17772.id42753.diff
Fri, Sep 19, 9:57 AM
F18646362: D17772.id.diff
Fri, Sep 19, 9:51 AM
F18583439: D17772.diff
Sep 11 2025, 10:25 AM
F18220464: D17772.id42753.diff
Aug 19 2025, 2:49 PM
F18220463: D17772.id42735.diff
Aug 19 2025, 2:49 PM
F18220461: D17772.id.diff
Aug 19 2025, 2:49 PM
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.