Opened 9 years ago

Last modified 3 years ago

#769 assigned task

Assertions

Reported by: Elly Owned by: elly
Milestone: Future Keywords: cleanup
Cc:

Description (last modified by takkaria)

~/s/angband/src$ find . -name '*.c' | xargs wc -l | tail -1

130707 total

~/s/angband/src$ grep -R assert * | wc -l

78

The ratio between these two numbers is "somewhat low", to quote a friend of mine. We should probably have on the order of 2-3 assertions per medium-sized function.

Change History (8)

comment:1 Changed 9 years ago by takkaria

  • Description modified (diff)

comment:2 Changed 8 years ago by magnate

  • Keywords cleanup added; code-quality removed

comment:3 Changed 7 years ago by takkaria

currently:

jjquark:src takkaria$ find . -name '*.c' | xargs wc -l | tail -1
  141147 total
jjquark:src takkaria$ grep -R assert * | wc -l
     215

That's 2.7x more in 2 years...

comment:4 Changed 7 years ago by magnate

  • Status changed from new to confirmed

comment:5 Changed 7 years ago by magnate

  • Type changed from bug to task

comment:6 Changed 4 years ago by takkaria

  • Description modified (diff)

Now 552. Slowly getting better at this...

comment:7 Changed 4 years ago by elly

  • Owner set to elly
  • Status changed from confirmed to assigned

comment:8 Changed 3 years ago by nckmccnnll

Now 133569 to 612

Note: See TracTickets for help on using tickets.