> >virtual std::string checkIn(std::string const & msg) = 0; virtual
> >std::string checkOut() = 0; virtual std::string repoUpdate() = 0;
> >virtual std::string lockingToggle() = 0;
>
> I don't understand why these return strings. That's at least very
> unintuitive. Why don't we call Buffer::message(string) to show the
> status message.
>
>iirc its an ugly hack to postpone displaying of the messages.
>i have tried your proposal and anything directly written into
>message() get immediately overwritten by the next multiple
>messages on the top of the processed dispatch machinery.
>
This works perfectly for me:
string SVN::checkOut()
{
docstring log;
[..]
theLyXFunc().setMessage("SVN: " + log);
}
If it is a hack, it's a nicer one than returning log messages IMHO.
Vincent
