search

blitzxion / Tanks.js

Using Javascript and HTML5, watch the AI build up their army and battle each other!
12 stars 4 forks source link

I fixed issue #9. #29

Closed jmaygarden closed 12 years ago

jmaygarden commented 12 years ago

My last commit addresses the wraparound issues.

derekgates commented 12 years ago

Very difficult to read the diff due to what I assume to be space/tabs, but this is most impressive to see after just talking about it today! :D

jmaygarden commented 12 years ago

Yeah, sorry about the diff. My editor trims trailing whitespace. Do you guys want to use tabs or spaces for indentation? -Judge

On Fri, May 4, 2012 at 10:49 PM, Derek Gates reply@reply.github.com wrote:

Very difficult to read the diff due to what I assume to be space/tabs, but this is most impressive to see after just talking about it today! :D

Reply to this email directly or view it on GitHub: https://github.com/blitzxion/Tanks.js/pull/29#issuecomment-5524196

derekgates commented 12 years ago

I use tabs due to my Windows background. OSX loves spaces.

I'll let Richard chime in here as I don't care either way.

blitzxion commented 12 years ago

I vote tabs!

jmaygarden commented 12 years ago

OK, I'll use tabs.