[prev in list] [next in list] [prev in thread] [next in thread] 

List:       kde-core-devel
Subject:    Re: A patch for kde-3.0.2 + qt-3.0.5
From:       Simon Hausmann <hausmann () kde ! org>
Date:       2002-08-02 8:24:40
[Download RAW message or body]

On Fri, Aug 02, 2002 at 10:11:56AM +0200, Laurent Montel wrote:
> What do you think about my patch.

Two minor comments:

1) +    text.remove(pos, QString("\n").length() );

   Why not just text.remove( pos, 1 ); ? :)

2) +  resize( 530, 230 );

   That one scares me :) . Why is it necessary to hardcode the size
   of the dialog?

Simon
[prev in list] [next in list] [prev in thread] [next in thread]