Memory leak when using the double-click feature

Ask for help and post your question on how to use XnView Classic.

Moderators: helmut, XnTriq, xnview

Post Reply
User avatar
Olivier_G
XnThusiast
Posts: 1423
Joined: Thu Dec 23, 2004 7:17 pm
Location: Paris, France
Contact:

Memory leak when using the double-click feature

Post by Olivier_G »

Hi,

I have just noticed that memory of images (current, and eventually of ahead/previous cache) is not released when going back from 'View' (normal or full screen) to browsing with the double-click feature.

A :bug: ?

Added: To be precise, it is never released (aka: a leak), even when viewing other images...

Olivier
PS: I use XnView 1.74 on Windows XP+SP2
User avatar
xnview
Author of XnView
Posts: 46238
Joined: Mon Oct 13, 2003 7:31 am
Location: France
Contact:

Re: Memory leak when using the double-click feature

Post by xnview »

Olivier_G wrote:I have just noticed that memory of images (current, and eventually of ahead/previous cache) is not released when going back from 'View' (normal or full screen) to browsing with the double-click feature.

A :bug: ?

Added: To be precise, it is never released (aka: a leak), even when viewing other images...
It's normal, because view is not closed. You can take a look at the menu "Window".
Pierre.
User avatar
helmut
Posts: 8704
Joined: Sun Oct 12, 2003 6:47 pm
Location: Frankfurt, Germany

Re: Memory leak when using the double-click feature

Post by helmut »

xnview wrote:It's normal, because view is not closed. You can take a look at the menu "Window".
Think like Olivier and other people I find this behaviour surprising. :o I expected the current image to be closed when double-clicking on it.

Sure enough the option in Options > View " says "Double-click to open Browser", not "Double-click to close view", so the current behaviour is correct, but I wonder whether this is really useful and what people need and expect.
User avatar
xnview
Author of XnView
Posts: 46238
Joined: Mon Oct 13, 2003 7:31 am
Location: France
Contact:

Re: Memory leak when using the double-click feature

Post by xnview »

helmut wrote:
xnview wrote:It's normal, because view is not closed. You can take a look at the menu "Window".
Think like Olivier and other people I find this behaviour surprising. :o I expected the current image to be closed when double-clicking on it.
Sure enough the option in Options > View " says "Double-click to open Browser", not "Double-click to close view", so the current behaviour is correct, but I wonder whether this is really useful and what people need and expect.
So what is better. When "Only one view" + "double click to open ..." must close all view before to return to the browser?
Pierre.
User avatar
helmut
Posts: 8704
Joined: Sun Oct 12, 2003 6:47 pm
Location: Frankfurt, Germany

Re: Memory leak when using the double-click feature

Post by helmut »

xnview wrote:...
So what is better. When "Only one view" + "double click to open ..." must close all view before to return to the browser?
I would expect that the current view (the one you double-click onto) is closed and the browser is opened. All the other views should not be affected.
User avatar
Olivier_G
XnThusiast
Posts: 1423
Joined: Thu Dec 23, 2004 7:17 pm
Location: Paris, France
Contact:

Re: Memory leak when using the double-click feature

Post by Olivier_G »

xnview wrote:It's normal, because view is not closed. You can take a look at the menu "Window".
Doh! :shock:
Right! I didn't even think about it due to the "Only one view" feature...
helmut wrote:
xnview wrote:So what is better. When "Only one view" + "double click to open ..." must close all view before to return to the browser?
I would expect that the current view (the one you double-click onto) is closed and the browser is opened. All the other views should not be affected.
This one looks like a winner... :D

Olivier
Dreamer
XnThusiast
Posts: 4608
Joined: Sun Jul 25, 2004 9:08 pm

Re: Memory leak when using the double-click feature

Post by Dreamer »

Olivier_G wrote:
helmut wrote:
xnview wrote:So what is better. When "Only one view" + "double click to open ..." must close all view before to return to the browser?
I would expect that the current view (the one you double-click onto) is closed and the browser is opened. All the other views should not be affected.
This one looks like a winner... :D
Another vote for that.

Also "Only one view" + "double click to open ..." options shoud be turned on by default, because (probably) most users expect it...
Post Reply