if/else, try/catch blocks
Devan Goodwin
dgoodwin at rm-rf.ca
Mon Mar 22 22:46:21 UTC 2010
2
On Mon, Mar 22, 2010 at 6:11 PM, Justin Harris <jharris at redhat.com> wrote:
>
> ----- "Jesus M. Rodriguez" <jesusr at redhat.com> wrote:
>
>> -----BEGIN PGP SIGNED MESSAGE-----
>> Hash: SHA1
>>
>> It was brought up today that some folks prefer:
>>
>> if (true) {
>> // some code
>> } else if (some_expressions) {
>> // some code
>> } else {
>> // some code
>> }
>>
>> to the current format of
>> if (true) {
>> // some code
>> }
>> else if (some_expressions) {
>> // some code
>> }
>> else {
>> // some code
>> }
>>
>> Same applies to try/catch/finally blocks:
>>
>> try {
>> // some code
>> } catch (Exception e) {
>> // log or rethrow
>> } finally {
>> // final cleanup
>> }
>>
>> as opposed to the current format:
>>
>> try {
>> // some code
>> }
>> catch (Exception e) {
>> // log or rethrow
>> }
>> finally {
>> // final cleanup
>> }
>>
>> I've grown partial to the latter (new lined version, but was used to
>> the
>> former in a past job). So I'm putting this up for a vote, majority
>> wins.
>> Only one vote per person.
>>
>> 1) same line
>> 2) new line
>
> Vote for 1
>
>>
>> jesus
>>
>> - --
>> jesus m. rodriguez | jesusr at redhat.com
>> principal software engineer | irc: zeus
>> red hat systems management | 919.754.4413 (w)
>> rhce # 805008586930012 | 919.623.0080 (c)
>> +---------------------------------------------+
>> | "Those who cannot remember the past |
>> | are condemned to repeat it." |
>> | -- George Santayana |
>> +---------------------------------------------+
>>
>> -----BEGIN PGP SIGNATURE-----
>> Version: GnuPG v1.4.9 (GNU/Linux)
>> Comment: Using GnuPG with Fedora - http://enigmail.mozdev.org/
>>
>> iEYEARECAAYFAkun3KIACgkQvJZ57YntiYPtBQCeI+IfmeKwmjS/PqksdZTWrDly
>> S/MAn0CAWjOlKkAhmdqXO22yHvcFv3nL
>> =dVIQ
>> -----END PGP SIGNATURE-----
>> _______________________________________________
>> candlepin mailing list
>> candlepin at lists.fedorahosted.org
>> https://fedorahosted.org/mailman/listinfo/candlepin
> _______________________________________________
> candlepin mailing list
> candlepin at lists.fedorahosted.org
> https://fedorahosted.org/mailman/listinfo/candlepin
>
--
Devan Goodwin <dgoodwin at rm-rf.ca>
http://rm-rf.ca
More information about the candlepin
mailing list