Lars Gullik Bjønnes wrote:
Bernhard Roider <[EMAIL PROTECTED]> writes:

| @@ -150,9 +153,12 @@
|  void QTocDialog::on_moveDownPB_clicked()
|  {

same comment

Same comment to what?


| @@ -160,9 +166,12 @@
|  void QTocDialog::on_moveInPB_clicked()
|  {

same comment

ditto


| @@ -170,9 +179,12 @@
|  void QTocDialog::on_moveOutPB_clicked()
|  {

same comment

ditto


| @@ -217,7 +229,7 @@
|  {
|       QStringListModel * type_model = form_->typeModel();
|       if (type_model->stringList().isEmpty()) {
| -             enableButtons();
| +             enableButtons(false);

Does the enableButtons() have a default arg?
If that is our own function I think it should be removed so that have
to use true/false explictly.

Why?
enableButtons(true) is redundant information.

Abdel.

Reply via email to