>>>>> "Juergen" == Juergen Spitzmueller <[EMAIL PROTECTED]> writes:

Juergen> Jean-Marc Lasgouttes wrote:
>> Actually, now that I have applied it, I see it does not look so
>> good

Juergen> I'm always happy to learn...

:)

>> A few things:

>> fl_set_choice_text(class_->choice_doc_fonts, params.fonts.c_str());
>> 
>> This one belongs to class_update

Juergen> Yes, sorry. This one went in by accident.

It was easy to miss.

>> As I already wrote, fl_set_choice_text would be better here.

Juergen> OK. Changed (though it was not my code ;-)).

Yes, I know.

Juergen> I do not understand this, sorry.

Forget about all this. Let's get it to work first :)

Juergen> Please have a look at the following code part. Does this get
Juergen> closer to your plan?

Yes. You can indeed remove the two fl_set_choice_text in class_update.

And does it work? This is the important question, after all :)

JMarc

Reply via email to