search

TechnionYP5779 / team4

29 stars 2 forks source link

eliminate all @SuppressWarnings #117

Closed yossigil closed 5 years ago

yossigil commented 5 years ago

From @anny234 on November 14, 2018 10:28

from all code that you wrote. do not add @SuppressWarnings unless there is no other choice

Copied from original issue: TechnionYP5779/team3#89

yossigil commented 5 years ago

This issue was moved to TechnionYP5779/team1#88

yossigil commented 5 years ago

This issue was moved to TechnionYP5779/team2#72

ron4548 commented 5 years ago

I think all of us should work on this since there are many @SuppressWarnings in the code.

ron4548 commented 5 years ago

Done. Removed about 60 instances of @SuppressWarnings. The others couldn't be removed since there was no other choice. Took me 2.75 hours.