lua-users home
lua-l archive

[Date Prev][Date Next][Thread Prev][Thread Next] [Date Index] [Thread Index]


This is not about aesthetics.
The sources use a consistent coding style and it is a right thing.

But this style is not friendly with some tools like debugger or code coverage analysor.
For example (from luaB_select):
    if (i < 0) i = n + i;
    else if (i > n) i = n;
could be rewritten :
    if (i < 0)
      i = n + i;
    else if (i > n)
      i = n;
This allows to set breakpoint on condition and on instruction,
and the code coverage analysis doesn't mix condition & instruction.

The attached script lgrep allows to find all occurences
    $ lgrep "if %b()%s+%l" *.c *.h

Now, a personal preference, please remove also the trailing spaces
    $ lgrep "%s$" *.c *.h

François

Note: You could find a full example of code coverage here
http://fperrad.github.com/lua-TestMore/cover_lua520/src/index.html

Attachment: lgrep
Description: Binary data