Angus Leeming <[EMAIL PROTECTED]> writes:

| On Saturday 31 March 2001 11:34, Lars Gullik Bjønnes wrote:
| > Angus Leeming <[EMAIL PROTECTED]> writes:
| > 
| > | I've implemented a controller-view split for the TOC popup. The 
| functionality 
| > | is that of the original xforms code before Lars introduced us to the new 
| > | buffer labelling scheme.
| > 
| > ?? So it does not work?
| 
| Sure it does. Try it out.
| 
| > If you have hardcoded any of the lists I'll be really mad... because
| > that was the whole idea behind the "rewrite".
| 
| Uh oh! Well, the lists are hardcoded in the controller as an interim 
| solution: look in ControlToc.C. This means that the GUIs don't have anything 
| hardcoded. You'll note that I have some questions that you are better able to 
| answer than I. I seem to remember that you said you'd "be there" when such 
| questions came your way. So, consider the questions asked!


vector<string> const ControlToc::getTypes() const
{
        vector<string> types;
 
        Buffer::Lists const tmp = lv_.view()->buffer()->getLists();
 
        Buffer::Lists::const_iterator cit = tmp.begin();
        Buffer::Lists::const_iterator end = tmp.end();
 
        for (; cit != end; ++cit) {
                types.push_back(cit->first.c_str());
                                          ^^^^^^^^^^
                                This looks strange. Just use cit->first.
        }
 
        return types;
}


I do not get why getType() is needed.

The different float types can be found in the FloatList, and the
specific parameters for that float is in Floating. However that actual
(LaTeX) command to produce these floats are not provided, and I do not
see why you should need those.
 
-- 
        Lgb

Reply via email to