edscott <[EMAIL PROTECTED]> writes:

| Changelog: file support/filetools.C (DestroyTmpDir (string const &
| tmpdir, bool Allfiles)) 
| Allow LyX to exit if it cannot delete a temporary directory that
| does not exist

Why is this the correct place to fix this? Shouldn't this be fixed in
the function that calls DestroyTmpdir?

| The second bug occurs when you have a main file with bunch of input
| file frames 
| and are constantly scrolling up and down with the wheel. It so
| happens that depending on the 
| cursor position, LyX will open the "update_inset" dialog. This should never
| happen on button4 or button5, and there is a line to that effect (if
| (button >= 2) return;) 
| But when you move the wheel fast, a button release event may be
| generated with button==0. If 
| the cursor is a top an input frame, bingo: you open the
| "update_inset" dialog. 
| This bug is easy to repeat, and the input files don't even have to
| exist (only 
| the input frames in the main lyx file).
| 
| patch 2:
| 
| Changelog: file  BufferView_pimpl.C (void 
|BufferView::Pimpl::workAreaButtonRelease(int x, int y,
|                                             unsigned int button))
| This lets LyX avoid action when it has a button release from button 0.
| This event occurs randomly by using rapid motion of mouse wheel on
| linux 2.2.16 and Xfree86 

Is this really a lyx problem? Or a XFree86/mouse driver problem?

| 
| ------------------------------------------------------
| --- BufferView_pimpl.C.patch1   Fri Jul  7 13:00:36 2000
| +++ BufferView_pimpl.C  Fri Jul  7 17:16:09 2000
| @@ -732,6 +732,7 @@
|                                               unsigned int button)
|  {
|         if (buffer_ == 0 || screen == 0) return;
| +       if (button <= 0) return; // rapid wheel movement (button4+5)
| at times yields button==0 

the button is usigned so < 0 is always false

if (button == 0) would in any case be sufficient.

        Lgb

Reply via email to