Rockbox

  • Status Closed
  • Percent Complete
    100%
  • Task Type Patches
  • Category Plugins
  • Assigned To No-one
  • Operating System All players
  • Severity Low
  • Priority Very Low
  • Reported Version Daily build (which?)
  • Due in Version Undecided
  • Due Date Undecided
  • Votes
  • Private
Attached to Project: Rockbox
Opened by daurnimator - 2009-06-24
Last edited by mcuelenaere - 2009-06-28

FS#10379 - Fix lua coding style

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

   patch.diff (12.8 KiB)
Closed by  mcuelenaere
2009-06-28 14:55
Reason for closing:  Accepted
Additional comments about closing:   Warning: Undefined array key "typography" in /home/rockbox/flyspray/plugins/dokuwiki/inc/parserutils.php on line 371 Warning: Undefined array key "camelcase" in /home/rockbox/flyspray/plugins/dokuwiki/inc/parserutils.php on line 407

In r21541.

mcuelenaere commented on 2009-06-24 17:52

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).

daurnimator commented on 2009-06-25 16:20

oh well,
You can do what you want, I just thought it was very unreadable.

mcuelenaere commented on 2009-06-25 16:58

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...

Available keyboard shortcuts

Tasklist

Task Details

Task Editing