shop3 icon indicating copy to clipboard operation
shop3 copied to clipboard

Check on style warning in test

Open rpgoldman opened this issue 5 years ago • 2 comments

Running the tests on SBCL, I see this in the transcript:

 Running test METHOD-TESTS ..
WARNING: Condition STYLE-WARNING was signalled.

not yet sure what causes this, and the test still passes.

rpgoldman avatar Apr 11 '19 18:04 rpgoldman

Also there are style warnings in TEST-FLAW-DETECTION, I believe.

rpgoldman avatar Apr 11 '19 20:04 rpgoldman

Style warnings, are just warnings. Not errors.

For example in the source code are lambda lists with &optional and &key. Example: (sexp &optional (stream t) &key (canonical nil)). There are other examples. The compiler tells the developer of a programming construct that represents what is thought to be bad programming style.

These kind of lambda lists are thought to be error-prone for programmers. They may forget to provide the optional argument when providing the keyword args. That's why there is a style warning.

SBCL also warns of uses of deprecated functions. In the code there is a call to sb-debug:backtrace. The compiler recommends to use sb-debug:print-backtrace instead.

lispm avatar Apr 12 '19 06:04 lispm