John Spray <[EMAIL PROTECTED]> writes:

|  * \file GToc.C
| void GToc::updateType()
| {
|       changing_views_ = true;
|       string const targettype =
|               toc::getType(controller().params().getCmdName());
>
|       typestore_->clear();
|       vector<string> types = controller().getTypes();
|       vector<string>::iterator it = types.begin();
|       vector<string>::iterator end = types.end();
|       for(;it != end; ++it) {
|               Gtk::TreeModel::iterator row = typestore_->append();
|               (*row)[listCol_] = *it;
|               if (*it == targettype)
|                       typecombo_->set_active(row);
|       }
>
|       // Because tiny empty ComboBoxes just look silly
|       typecombo_->set_size_request(
|               typestore_->children().empty() ? 130 : -1, -1);

or even simpler

   ...set_size_request(types.empty() ? 130 : -1, -1);

is the (-1, -1) call to set_size_request really needed if you don't
have any items in the list?

if (types.empty())
   ...set_size_request(130, _1);

perhaps even in front of the loop?

-- 
        Lgb

Reply via email to