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.

> 
> | +   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 ...

regards
john

-- 
"Hungarian notation is the tactical nuclear weapon of source code obfuscation
techniques." 
        - Roedy Green 

Reply via email to