Skip to content

Commit

Permalink
Figured out the issue with problem.md in trouble
Browse files Browse the repository at this point in the history
Basically, a macro match was gobbling it up. Refactoring that part would
be substantial and is a goal, but it will take time. In the meantime, if
you have something like CAPS(_"dude") then change that to CAPS
(_"dude"). This is true even if CAPS does not match anything. That's the
problem.

Did find out that the parsing is not line based in compiling code
blocks.
  • Loading branch information
jostylr committed May 6, 2014
1 parent 5679ec8 commit 5a1d6e3
Show file tree
Hide file tree
Showing 4 changed files with 24 additions and 8 deletions.
6 changes: 5 additions & 1 deletion README.md
Original file line number Diff line number Diff line change
Expand Up @@ -110,7 +110,11 @@ syntaxes:
2. ``_`javascript code` `` One can execute arbitrary javascript code within
the backticks, but the parser limits what can be in there to one line.
3. `MACROS` all caps are for constants or macro functions that insert their
output in place of the caps.
output in place of the caps. Note that if you have `MACRO(_"something")`
then the current version does not parse `_"something"` as a code block.
This will hopefully get fixed along with being able to use code blocks in
commands. This applies even if `MACRO` does not match so it is a bug, not a
feature :( To fix this, put a space between `MACRO` and the parenthesis.

For both 1 and 3, if there is no match, then the text is unchanged. One can
have more than one underscore for 1 and 2; this delays the substitution until
Expand Down
6 changes: 5 additions & 1 deletion lp.md
Original file line number Diff line number Diff line change
Expand Up @@ -3300,7 +3300,11 @@ syntaxes:
2. ``_`javascript code` `` One can execute arbitrary javascript code within
the backticks, but the parser limits what can be in there to one line.
3. `MACROS` all caps are for constants or macro functions that insert their
output in place of the caps.
output in place of the caps. Note that if you have `MACRO(_"something")`
then the current version does not parse `_"something"` as a code block.
This will hopefully get fixed along with being able to use code blocks in
commands. This applies even if `MACRO` does not match so it is a bug, not a
feature :( To fix this, put a space between `MACRO` and the parenthesis.

For both 1 and 3, if there is no match, then the text is unchanged. One can
have more than one underscore for 1 and 2; this delays the substitution until
Expand Down
14 changes: 11 additions & 3 deletions trouble/problem.js
Original file line number Diff line number Diff line change
Expand Up @@ -14,11 +14,19 @@ var feedNew = new RSS({

});

var rssUpdate = new RSS({
var rssUpdate = new RSS ({
site_url : "http://mord.jostylr.com",
author : "Mord Drom of Drok",
managingEditor : "Janord Drom",
webMaster : "James Taylor",
language : "en",
categories : ["fantasy"],
pubDate : now.toString(),
ttl: '1440',
copyright : now.getFullYear() + " James Taylor"
title: "Mord Update",
description : "I am Mord. I serve my lord Kord with my greatsword. Misspeak I do.",
feed_url : "http://mord.jostylr.com/rssupdate.xml",
_"common"
feed_url : "http://mord.jostylr.com/rssupdate.xml",
});

var news, udates;
Expand Down
6 changes: 3 additions & 3 deletions trouble/problem.md
Original file line number Diff line number Diff line change
Expand Up @@ -19,11 +19,11 @@ This is for setting up the rss feeds. So we create two need feeds, one for new i
});


var rssUpdate = new RSS({
var rssUpdate = new RSS ({
_"common"
title: "Mord Update",
description : "I am Mord. I serve my lord Kord with my greatsword. Misspeak I do.",
feed_url : "http://mord.jostylr.com/rssupdate.xml",
_"common"
feed_url : "http://mord.jostylr.com/rssupdate.xml",
});

var news, udates;
Expand Down

0 comments on commit 5a1d6e3

Please sign in to comment.