-
Notifications
You must be signed in to change notification settings - Fork 279
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
No style given for try..catch #141
Comments
2nd style should be preferred. |
@arrdem is completely right. Guess we should add a rule about this. |
Oh no, the style I've seen/am used to is: (try (f)
(catch E e
(g)) That's the Emacs default! It's also similar to the way it's done with It would be great to get this settled/codified. |
@MicahElliott Well, both are standard Emacs indentations - the first layout is wider, the second is narrower (and I believe way more common). The second layout has the distinct advantage that it highlights better the special nature of |
I guess I'd argue that a My point was probably more to the notion that if you do a one-line try, you should indent the So my vote would be to not capture a prescription here for the style guide. It is an interesting case, though, where we could capture that there are a couple acceptable ways to do it, so as to avoid people changing existing code that's one or the other. |
If I remember correctly, when I filed this issue I was thinking of a full-sized nested |
Two styles I have seen:
The text was updated successfully, but these errors were encountered: