WIP whitespace and indentation

[erights]

trafficstars

DO NOT MERGE: Based on an ancient master

An experiment in whitespace layout for grammar definition. Only approve if you like it better,

[erights]

Assumes reading at 100 columns.

On Sun, Nov 17, 2019, 6:17 PM Mark S. Miller [email protected] wrote:

An experiment in whitespace layout for grammar definition. Only approve if you like it better,

You can view, comment on, or merge this pull request online at:

https://github.com/agoric-labs/jessica/pull/21 Commit Summary

• whitespace

File Changes

• M lib/quasi-jessie.js https://github.com/agoric-labs/jessica/pull/21/files#diff-0 (188)
• M lib/quasi-json.js https://github.com/agoric-labs/jessica/pull/21/files#diff-1 (52)
• M lib/quasi-justin.js https://github.com/agoric-labs/jessica/pull/21/files#diff-2 (56)

Patch Links:

• https://github.com/agoric-labs/jessica/pull/21.patch
• https://github.com/agoric-labs/jessica/pull/21.diff

— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub https://github.com/agoric-labs/jessica/pull/21?email_source=notifications&email_token=AACC3TH3WVQRMCPHD5DHTK3QUH3KJA5CNFSM4JONEBNKYY3PNVWWK3TUL52HS4DFUVEXG43VMWVGG33NNVSW45C7NFSM4HZ5B2EQ, or unsubscribe https://github.com/notifications/unsubscribe-auth/AACC3TCWOJAKTUT4MKDFVN3QUH3KJANCNFSM4JONEBNA .

[michaelfig]

Thanks, I do like the sentiment of the change.

Unfortunately, the lib/*.js files are generated by ./build.bat from the lib/*.js.ts files, and because of the type annotations in the *.js.ts files, your changes don't apply cleanly to them.

Let's revisit this when we have time to format the grammars in the *.js.ts files better.

[erights]

Let's revisit this when we have time to format the grammars in the *.js.ts files better.

Converted to Draft