/elec/propeller-clock

To get this branch, use:
bzr branch http://bzr.ed.am/elec/propeller-clock

« back to all changes in this revision

Viewing changes to src/TODO

  • Committer: Tim Marston
  • Date: 2012-05-18 18:29:50 UTC
  • Revision ID: tim@ed.am-20120518182950-t85bn9a21n72uzm8
text messages are now individually enabled and draw()n automatically

Show diffs side-by-side

added added

removed removed

src/TODO
1
 
* Automatically draw text.
2
 
 
3
 
  Text::draw() should be called from propeller-clock.cc, and it should itterate
4
 
  over active messages and call draw_message() on them (which does what the
5
 
  current draw() does).
6
 
  
7
 
  This requires that Text somehow knows which messages are active. Perhaps
8
 
  message_reset() could somehow be used to determine this?
9
 
  
10
 
  This is necessary so that the cludgy call to Text::post_draw() can be removed
11
 
  from propeller-clock.cc. The problem here is that if _do_draw_reset is set
12
 
  false inside what is currently Text::draw(), only the first drawn message is
13
 
  ever correctly reset!
14
 
  
15
 
  This would also make calls to Text::draw() unnecessary from mode draw()
16
 
  routines.
17
 
 
18
 
  At the same time, remove calls to Text::reset() from SwitcherMajorMode. This
19
 
  now be done at the start of reset_messages(), which should be renamed to
20
 
  set_up_messages() in all modes.
 
1
* Fix led_off() being removed.
 
2
 
 
3
  Call from Text::reset() instead? No. Might have to add it to all the
 
4
  activate()s and reset_messages()...
 
5
  
 
6
* Bug: _message_param is not getting set correctly for message 1 and up.