Fix order in jvm options in `java -jar` sample #13

Merged
merged 1 commit into from May 31, 2014

Projects

None yet

2 participants

@ardumont

Hello,

I just fixed the order in the sample as suggested.

Just out of curiosity, is there a reason not to factorize the common behavior between the private functions read-system-env and read-system-props?

Cheers,

@weavejester
Owner

No particular reason, except that it wouldn't save any lines of code.

@weavejester weavejester merged commit 0c6711f into weavejester:master May 31, 2014
@ardumont ardumont deleted the ardumont:fix-sample-java-jar branch May 31, 2014
@ardumont

No particular reason, except that it wouldn't save any lines of code.

Ok.
I thought more in terms of potential subtle bugs (you know, the insidious ones when you are not aware that you are not at your best, if it ever happens to you that is) than lines of code :D

By the way, thanks for the multiple clojure libraries.

Cheers,

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment