GivingThanks

Summary

GivingThanks contract declares field address public owner; which is set in the constructor to msg.sender but it is not used.

Impact

Depending on the intent of the authors this can have different impact. At the very least this a waste of gas and brings confusion when trying to understand the logic of the code.
If the authors had the idea to use the field as to restrict some of the functionality of the contract to the owner then this has high impact as the field is not used.

Tools Used

Manual review

Recommendations

Either remove the field if it is not going to be used or use it.