Martin Vermeer wrote:

> On Sun, Mar 21, 2004 at 01:27:21PM -0800, Kayvan A. Sylvan spake
> thusly:
>  
>> I have a lyx document with a branch called "titlepage" (for when I
>> want to turn a custom titlepage off or on).
>> 
>> When LyX reads it, I see:
>> 
>>     no text in updateScrollbar
>>     Buffer::Buffer()
>>     LColor::getFromLyXName: Unknown color "titlepage"

>     251 LColor::color LColor::getFromLyXName(string const & lyxname)
> const
>     252 {
>     253     string const lcname = ascii_lowercase(lyxname);
>     254     if (pimpl_->transform.find(lcname) ==
>     pimpl_->transform.end()) {
>     255         lyxerr << "LColor::getFromLyXName: Unknown color \""
>     256                << lyxname << '"' << endl;
>     257         addColor(static_cast<color>(pimpl_->infotab.size()),
>     lcname);
>     258     }
>     259
>     260     return
>     static_cast<LColor::color>(pimpl_->transform[lcname]); 261 }
> 
> As you see, this is only a warning that the colour isn't yet in the
> registry, and then it is added.
> 
> Should this be made more clearly a warning or info message? Like
> "... Color \"" << lyxname << '"' added to color table" << endl ?

Yes. That is exactly what the code is doing, after all. Moreover, I 
think that the warning should be output in debug mode only.

        lyxerr[your debug flavour] << ...;

-- 
Angus

Reply via email to