From: Mark H. <ma...@li...> - 2003-05-18 16:44:04
|
On Sat, 17, May, 2003 at 03:59:38PM +0200, Guillaume Laurent spoke thus.. > > OK. That makes a lot more sense. I've done it. I've only provided a > > const_iterator method though because I don't think we want people > > playing with the data directly, they should be using the accessor > > functions. > > Makes sense, though I'm not too keen on the "dual" way to access the data, > either through interators or through index-based methods. Are you using the > iterator somewhere else ? If not, it might be better to simply do away with > it. I think we should have a const_iterator method of accessing it. The reason is that we will (in at least two places) want to show the user a list of the available colours: |COLOUR1| name |COLOUR2| name etc. and it'll be easier to populate this using an iterator. The actual access functions will be more useful when drawing the segments on the canvas when we can get the relevent colour just using a single call. Mark -- Mark Hymers <markh at linuxfromscratch dot org> "Well, the thing about a black hole - it's main distinguishing feature - is it's black. And the thing about space, your basic space colour is black. So how are you supposed to see them?" Holly, Red Dwarf Series III - Marooned |