willwolfram18 / 183lint Goto Github PK

Here is the source line:

        cout << "Choice is: " << choice << endl;

And the error message:

Line 122, Column 31: Incorrect spacing around < operator.

code:
if (!cin || !isValidDate(month, day, year)) {

But
if (! cin || ! isValidDate(month, day, year)) { //gives
Line 320, Column 9: Incorrect spacing around ! operator.
Line 320, Column 18: Incorrect spacing around the unary ! operator.

Any time I have an assert statement, it's being flagged as using a ternary operator.

For instance:

assert(!inSortedVector(nums, 5));

Gets me this feedback:

Line 25, Column 5: EECS 183 advises against the use of the ternary operator (e.g. int x = foo() ? 1 : 2).

I had the line of code

int num_batches = ceil(static_cast<double>(num_people)/PER_BATCH);

and I received the feedback

Line 46, Column 39: Incorrect spacing around < operator.

When submitting a file with this line of code:

return x >= -9 && x <= 9;

I received the message "Incorrect spacing around - operator."

Complaints have been made, stating it isn't clear which files are correct and which ones have errors. One design idea is grouping incorrect files in box, which can be further broken into individual files

Does not catch spacing errors on operators that are overloaded by classes. Example code that can create the bug:

string s = "Hello";
s = s+"goodbye";
// as well as
if (!cin || true)
{
}

Issue discovered by Professor Dorf in ticket #3

For the following code, it results in a style error:

   } else if (slash1 != '/' || slash2 != '/' ||

Error message:

Incorrect spacing around / operator.

The spacing shouldn't lead to a style error (it just splits a really long line on the extraction operator), but I'm getting the error in the title