dlang-community / Pegged

A Parsing Expression Grammar (PEG) module, using the D programming language.
534 stars 66 forks source link

Added generating D grammar, plus fix to NestingBlockComment rule #77

Closed callumenator closed 12 years ago

callumenator commented 12 years ago

I did regenerate the dparser from the grammar, but when I try to compile it I get a stack overflow error. So I haven't included the generated parser, just the grammar with the fix.

PhilippeSigaud commented 12 years ago

Thanks Cal!

Github seems to have trouble right now (lots server errors). I'll merge it a bit later.

PhilippeSigaud commented 12 years ago

Still broken. Ooh, bad day for github.

Btw, I finally coerced github into letting me look at the file you're adding. The module name should be pegged.examples.d (you call it c). There is a recent thread on the newsgroup saying to avoid that. What about calling it pegged.examples.dgrammar.d or somesuch?

callumenator commented 12 years ago

Oops I copied the C grammar file, forgot to change the name. Pushed the fix.

On Mon, Oct 29, 2012 at 1:24 PM, Philippe Sigaud notifications@github.comwrote:

Still broken. Ooh, bad day for github.

Btw, I finally coerced github into letting me look at the file you're adding. The module name should be pegged.examples.d (you call it c). There is a recent thread on the newsgroup saying to avoid that. What about calling it pegged.examples.dgrammar.d or somesuch?

— Reply to this email directly or view it on GitHubhttps://github.com/PhilippeSigaud/Pegged/pull/77#issuecomment-9886044.

PhilippeSigaud commented 12 years ago

OK, cool.

Github still don't let me merge :( Too bad. Any problem on your side?

callumenator commented 12 years ago

Naw it's running ok from here (in the US).

On Mon, Oct 29, 2012 at 1:36 PM, Philippe Sigaud notifications@github.comwrote:

OK, cool.

Github still don't let me merge :( Too bad. Any problem on your side?

— Reply to this email directly or view it on GitHubhttps://github.com/PhilippeSigaud/Pegged/pull/77#issuecomment-9886395.

PhilippeSigaud commented 12 years ago

Still crashing here. Sending mail from my mail client works, but not typing comments on the github site, nor merging. (test) OK, more generally, nothing on the site works. Transatlantic cable problem, maybe?

As an aside, do you know how I can give you merging rights?

callumenator commented 12 years ago

Must be the Frankenstorm.

I think under admin, there is a collaborators section, which gives someone else read/write permissions.

PhilippeSigaud commented 12 years ago

Yes, but the 'add' button is grayed.

callumenator commented 12 years ago

Even when you type a name in the box?

PhilippeSigaud commented 12 years ago

Ah, everything seems to be back in order. Here we go!