Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Reformat and synchronize Nock 5K spec #28

Merged
merged 1 commit into from Sep 28, 2013
Merged

Reformat and synchronize Nock 5K spec #28

merged 1 commit into from Sep 28, 2013

Conversation

ghost
Copy link

@ghost ghost commented Sep 28, 2013

This makes minor formatting changes to the Nock 5K spec: it (1) removes some trailing whitespace characters and (2) reorders the reduction lines so that the rules for each operation are contiguous. (The "never terminate" rules were once moved to the end, which broke the reduction order; in fixing this, some operations were kept contiguous while others weren't. This makes it consistent.)

This also updates some copies of the spec: in the spec directories under hub and urb, and also in the comments of two copies of hoon.hoon. There are copies of the Nock spec in the aug, jupiter, and lib directories, but I understood these to be historical versions, so I didn't touch them. The Nock tutorial (doc/book/1-nock.markdown) also incorporates an out-of-date copy of the Nock spec, but I'll leave it to the author to eventually update that.

Since we're very interested in the gzipped spec size, I'll mention that the 5K spec in this pull request gzips to 371 bytes.

(@abliss, sorry about the t-shirts.)

This commit makes minor formatting changes to the Nock 5K spec: it (1)
removes some trailing whitespace characters and (2) reorders the
reduction lines so that the rules for each operation are contiguous.

This commit also updates some copies of the spec: in the spec
directories under hub and urb, and also in the comments of two copies of
hoon.hoon.

At this point, the 5K spec gzips to 371 bytes.
@abliss
Copy link
Contributor

abliss commented Sep 28, 2013

Oh, I already wrote off the T-shirts with the obsolete 2ea195 version of Nock5K. But now the 092154 version has been incorporated into a mural across the side of the bike shed outside my nuclear reactor!

cgyarvin added a commit that referenced this pull request Sep 28, 2013
Reformat and synchronize Nock 5K spec
@cgyarvin cgyarvin merged commit 78276ed into urbit:master Sep 28, 2013
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants