On Sunday 14 June 2009 13:02:04 Oscar Miras wrote: > Ok, now it works ok.
OK, I think you can check it in the branch https://geeqie.svn.sourceforge.net/svnroot/geeqie/branches/calendar This should make it easier for anybody to test the code. > > I wonder if there's a way to not to have to stop pan full screen window > mode; before going to pan full screen image mode. > I have tried to go to full screen image mode from full screen window mode ; > but finally I get a 'Segmentation fault' ; because of not calling > fullscreen_stop ; it does not return properly to the previous state of the > program ( or almost I think something like that is happening). > > At least, isn't a way to tell Geeqie not to refresh pw->imd? If so, I > almost could avoid image blinking as a result of returning to pan view > normal mode and then going to full screen image mode. Maybe it needs some switching of parameters of existing windows in fullscreen.c, but I'd have to look into details. > > PD: I use "diff -uN pan-view.old pan-view.c > pan-view.patch" to make the > patch, but I suspect you are using antoher sentence. Wich is it , please ? > I attach original files too because I don't know if i'm making the patch > correctly. Just use "svn diff" > > PD2: I tried to doxygen pan_fullscreen_toggle_image ; tell me what do you > think about it. I would leave this to others because I don't have that many experiences with doxygen ;) Vladimir > > 2009/6/11 Vladimir Nadvornik <nadvor...@suse.cz> > > > Hi, > > > > On Thursday 11 June 2009 12:02:18 Oscar Miras wrote: > > > Hi, > > > > > > Ok. In fact, I never used g_new ... I realised when I watch again code, > > > sorry :S. > > > Finally I coded a version where everything works ok. > > > > > > But there's a problem, i'm creating a dependency from fullscreen -> > > > pan-type. Before it never existed. > > > Now in fullscreen_start function I'm invoking like if (!pw->image_fd) > > > . I'll show you important code lines: > > > > I wrote "if (fs->normal_imd)", not "if (!pw->image_fd)" ;-) > > See the attached patch. > > Fullscreen does not have to be dependent on pan view, of course. > > > > With this patch you can call > > > > pw->fs_image = fullscreen_start(pw->window, NULL, > > pan_fullscreen_stop_func, pw); > > > > and then setup the image with > > > > image_change_fd(pw->fs_image->imd, fd, image_zoom_get_default(NULL)); > > > > and similar functions explicitly from pan view. > > > > Vladimir > > > > > > > > > > ------------------------------------------------------------------------- > >----- Crystal Reports - New Free Runtime and 30 Day Trial > > Check out the new simplified licensing option that enables unlimited > > royalty-free distribution of the report engine for externally facing > > server and web deployment. > > http://p.sf.net/sfu/businessobjects > > _______________________________________________ > > Geeqie-devel mailing list > > Geeqie-devel@lists.sourceforge.net > > https://lists.sourceforge.net/lists/listinfo/geeqie-devel ------------------------------------------------------------------------------ Crystal Reports - New Free Runtime and 30 Day Trial Check out the new simplified licensing option that enables unlimited royalty-free distribution of the report engine for externally facing server and web deployment. http://p.sf.net/sfu/businessobjects _______________________________________________ Geeqie-devel mailing list Geeqie-devel@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/geeqie-devel
