Rockbox

Tasklist

FS#10379 - Fix lua coding style

Attached to Project: Rockbox
Opened by Daurnimator (daurnimator) - Wednesday, 24 June 2009, 15:45 GMT
Last edited by Maurus Cuelenaere (mcuelenaere) - Sunday, 28 June 2009, 14:55 GMT
Task Type Patches
Category Plugins
Status Closed
Assigned To No-one
Operating System All players
Severity Low
Priority Normal
Reported Version Daily build (which?)
Due in Version Undecided
Due Date Undecided
Percent Complete 100%
Votes 0
Private No

Details

Fixed lua coding style in helloworld and test_viewports:
optimised some code, made it more readable
This task depends upon

Closed by  Maurus Cuelenaere (mcuelenaere)
Sunday, 28 June 2009, 14:55 GMT
Reason for closing:  Accepted
Additional comments about closing:  In r21541.
Comment by Maurus Cuelenaere (mcuelenaere) - Wednesday, 24 June 2009, 17:52 GMT
Most of these changes are whitespace changes, which I'm not inclined to accept; see the Rockbox guidelines for more information: http://svn.rockbox.org/viewvc.cgi/trunk/docs/CONTRIBUTING?view=markup

I will probably commit some of the functional changes when I'll remove the current way to access files and replace it with the Lua IO library, but I'll need your real name for that (again, see the guidelines).
Comment by Daurnimator (daurnimator) - Thursday, 25 June 2009, 16:20 GMT
oh well,
You can do what you want, I just thought it was very unreadable.
Comment by Maurus Cuelenaere (mcuelenaere) - Thursday, 25 June 2009, 16:58 GMT
Well, I don't agree with you :)

Anyway, I will commit the functional changes; but as I said I need your real name for that.

Loading...