jeffreykegler / yahc

Yet Another Hoon Compiler
MIT License
0 stars 1 forks source link

WUTBAR, WUTPAM anomalies #19

Closed jeffreykegler closed 5 years ago

jeffreykegler commented 5 years ago

wutbarpam.lint.txt I implemented both split and joined 0-running hoons, and will introduce a stub that will allow for a minimum length of split and a maximum length of joined 0-running hoons. I'll follow up with that back in issue #13 .

Here I think this are all candidates for anomaly.suppressions, with the possible exception of jael, where you might say a pseudo-joined style is being used. But it would be the only case for WUTBAR/WUTPAM in the whole arvo/ corpus, suggesting it's a error and not a pattern.

@ohAitch -- let me know what you think.

ohAitch commented 5 years ago

Yup, the outdented blank comments should certainly be outdented all the way to the running rune, and the zuse newline is good.

As for pseudo-joined style, it looks like some of clay is doing something similar? But yeah just outdenting them seems fine.

On Friday, 1 March 2019, Jeffrey Kegler notifications@github.com wrote:

wutbarpam.lint.txt https://github.com/jeffreykegler/yahc/files/2921614/wutbarpam.lint.txt I implemented both split and joined 0-running hoons, and will introduce a stub that will allow for a minimum length of split and a maximum length of joined 0-running hoons. I'll follow up with that back in issue #13 https://github.com/jeffreykegler/yahc/issues/13 .

Here I think this are all candidates for anomaly.suppressions, with the possible exception of jael, where you might say a pseudo-joined style is being used. But it would be the only case for WUTBAR/WUTPAM in the whole arvo/ corpus, suggesting it's a error and not a pattern.

@ohAitch https://github.com/ohAitch -- let me know what you think.

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/jeffreykegler/yahc/issues/19, or mute the thread https://github.com/notifications/unsubscribe-auth/ABxXhj889bTtR6H5me7evZeZAQvUTrEHks5vSdTrgaJpZM4baFnk .

jeffreykegler commented 5 years ago

Added suppressions to anomaly.suppressionsin commit 0666382. Marking as "ready to be closed".

jeffreykegler commented 5 years ago

Closed as per the above.