| Commit message (Collapse) | Author | Files | Lines |
|
Signed-off-by: Mattias Andrée <maandree@kth.se>
|
|
Signed-off-by: Mattias Andrée <maandree@kth.se>
|
|
Signed-off-by: Mattias Andrée <maandree@kth.se>
|
|
Signed-off-by: Mattias Andrée <maandree@kth.se>
|
|
Signed-off-by: Mattias Andrée <maandree@kth.se>
|
|
The test suite simply runs makel in turn on all Makefiles in tests/,
and compares makel's exit codes with the expected exit codes that are
written in the Makefiles themselves.
This system could be improved by comparing warning classes instead,
and using text identifiers instead of numerical codes which might be subject to change.
Revision 2 of this patch:
Make target to run the test suite is now called 'check' to avoid confusion.
|
|
The checks for line continuation starting on an empty line, ending on an empty line,
and ending on EOF were broken because of logic errors in said checks, as well as
an accidental negation in is_line_blank().
|
|
Signed-off-by: Mattias Andrée <maandree@kth.se>
|
|
Signed-off-by: Mattias Andrée <maandree@kth.se>
|
|
Signed-off-by: Mattias Andrée <maandree@kth.se>
|
|
Signed-off-by: Mattias Andrée <maandree@kth.se>
|
|
Suggested by Laslo Hunhold, "Makel" is German for
defect, deficiency, impurity, or imperfection (both
singular and plural); a flaw.
This change is to avoid confusion between
make(1) and mk(1)
Signed-off-by: Mattias Andrée <maandree@kth.se>
|
|
imo a good linter should be concise and helpful. printing argv0 doesn't
add much value but instead adds noise to the output.
i've left printerrorf untouched however, since it calls exit.
|
|
Signed-off-by: Mattias Andrée <maandree@kth.se>
|
|
Signed-off-by: Mattias Andrée <maandree@kth.se>
|
|
Signed-off-by: Mattias Andrée <maandree@kth.se>
|
|
Signed-off-by: Mattias Andrée <maandree@kth.se>
|
|
Signed-off-by: Mattias Andrée <maandree@kth.se>
|
|
Signed-off-by: Mattias Andrée <maandree@kth.se>
|
|
Signed-off-by: Mattias Andrée <maandree@kth.se>
|
|
currently if open_default_makefile() finds a standard makefile it will
print info message but then warn about the same file being "additional."
[nrk mklint master]% ./mklint
./mklint: [info] found standard makefile to use: Makefile (-wmakefile)
./mklint: [warn] found additional standard makefile: Makefile (-wextra-makefile)
this makes it so that it doesn't produce warning about the the makefile
that it already found as "additional."
also declares the default_makefiles pointer as const for good measures.
|
|
Signed-off-by: Mattias Andrée <maandree@kth.se>
|
|
Signed-off-by: Mattias Andrée <maandree@kth.se>
|