InsetWrap question

[Angus Leeming]

Dekel / LaTeX guru:

what's going on here? This suggests that 
WrapParams::placement and InsetWrap::Placement_ 
should be chars not strings.

Am I reading this wrong, or should I make the change?

Angus

------------------------------------------------
void FormWrap::apply()
{
        controller().params().pageWidth =
                LyXLength(getLengthFromWidgets(dialog_->input_width,
                                               dialog_->choice_width_units));

        if (fl_get_button(dialog_->radio_left))
                controller().params().placement = "l";
        else if (fl_get_button(dialog_->radio_right))
                controller().params().placement = "r";
        else if (fl_get_button(dialog_->radio_outer))
                controller().params().placement = "p";
        else
                controller().params().placement.erase();
}