rgheck schreef:
Vincent van Ravesteijn - TNW wrote:
* Why don't we need the case_insensitive_ private member? I ask this
because we do have the sort_ private member. There might be a reason why
the following is needed in updateContents():
"sortCB->setChecked(sort_);". Otherwise we could remove the sort_
member.

I think this was there for restoring previous behavior if the dialog was destroyed. But it seems to work fine if I comment out this line, so maybe it can be removed.

rh

Yes, it stems from the old days where a new Dialog was instantiated everytime and thus the checkbox had to be set afterwards. It is not necessary anymore now.. so I'll remove it.

Vincent

Reply via email to