A contract accesses its own variable using scoped qualification.

Using more-complicated-than-necessary variable access makes code harder to read and invites hard-to-find bugs. For example accidently calling a function with external visibility when internal is appropriate will cost more gas and also produce incorrect results during initialization.

Recommendation: use direct variable access when possible. e.g. Deadline.getDeadline() -> _deadline.

References:

• erasure-protocol/contracts/modules/Deadline.sol

  Line 35 in 4a3d98c

  if (Deadline.getDeadline() > now)