John Levon <[EMAIL PROTECTED]> writes:

| On Thu, Jul 25, 2002 at 11:32:02AM +0200, Lars Gullik Bjønnes wrote:
>
>> |  void Menubar::Pimpl::update()
>> |  {
>> | -  // FIXME: handle special stuff to be updated.
>> | - 
>> | -  ItemMap::const_iterator cit = items_.begin();
>> | -  ItemMap::const_iterator end = items_.end();
>> | +  size_t i;
>> 
>> Why is this funciton wide?
>
| wide ? Sorry, I don't know what you mean.

scope then...

>> 
>> | +  QMenuData const * bar(owner_->menuBar());
>> |  
>> | -  for (; cit != end; ++cit)
>> | -          updateItem(cit->second);
>> | +  for (i = 0; i < bar->count(); ++i) {
>> 
>> Just use: for (size_t i = 0; i < bar->count(); ++i) {
>
| Sure. This was just demo code, not a patch submission ...

even for demo code...
Other people learn by example, better keep the examples nice.

-- 
        Lgb

Reply via email to