Angus Leeming <[EMAIL PROTECTED]> writes:

| Lars Gullik Bjønnes <[EMAIL PROTECTED]> writes:
| > This patch speeds up the scroll-down test by and odd second or two for
| > me.
| 
| Really?

A _very_ coarse test really. I might be mistaken. Howver it cannot
hurt.

| Wow! Did John Torjo's logging library ever make it into Boost? IIRC,
| using it would be equivalent to your change here.

No I don't think so. And in some ways our DebugStream is more usable.
(My plan was to adapt DebugStream to the logging library when it was
accepted into boost)
 
| >  void BufferView::Pimpl::update(Update::flags flags)
| >  {
| > +       // This is close to a hot-path.
| > +       if (lyxerr.debugging(Debug::DEBUG)) {
| >         lyxerr[Debug::DEBUG]
| 
| You don't need to call operator[] anymore...

So it would seem... but then DebugStream looses why it was called, and
we render it unable to output f.ex. "Debug:".

-- 
        Lgb

Reply via email to