恩,又一款开源软件的修改 :-)
GPicview是Linux下的看图软件,其类似于Windows自带的看图软件,其基于GTK.在Linux查看图片文件十分方便,并且体积小巧,暂用资源很少,我十分喜欢.
但是,用过一段时间后,发现在滚动鼠标滚轮时,它老是自动跳到下一张图片了,而我实际上是想通过滚轮上下移动图片的.而且,我按左右方向键时,它也是切换图片,我则希望它是左右移动图片.唉,就这点,用得我始终不爽,怎么办呢?改呗!这也是开源软件的一个原则--允许用户自由修改代码,以满足用户需求.
首先,下载源代码,在Sourceforge上有代码,选择最新的版本0.2.2.需要说明的是,0.2.2版新增了幻灯片播放功能,而且全屏看图时隐藏了工具栏,并且占满了整个屏幕,感觉特别好.
解压源码包,在src目录下浏览了一下代码,发现代码很少,而且命名也很规范,一看就能知道是干什么用的.我要改的是按键和鼠标事件对应的响应方法,所以,迅速锁定文件main-win.c,主窗口嘛,一般都是负责显示和外部操作功能的.
前段时间刚刚了解了一些GTK方面的知识,于是直接定位到滚轮事件scroll-event的响应函数on_scroll_event上.一眼便看出,作者在鼠标滚轮操作中,都是进行图片的切换,那我只需要将图片切换改成图片移动就可以了.
但是,图片是如何移动的呢?经过分析发现,图片的移动操作的实现应该在鼠标移动事件motion-notify-event的响应函数中,于是,找到该响应函数on_mouse_move.浏览了一下代码,知道在这个函数中,首先是获取但前鼠标位置,然后,与上次移动后的鼠标位置求差值,最后,再将图片按差值进行移动.那我就直接独立出一个图片移动函数static void move_image(MainWin* mw, int dx, int dy),再将on_mouse_move中dx,dy求值以下的代码剪切到move_image中,作为实现,当然,还需要修改"mw->drag_old_x = cur_x;"为"mw->drag_old_x -= dx;", "mw->drag_old_y = cur_y;"修改为"mw->drag_old_y -= dy;".再在on_mouse_move中增加函数"move_image(mw, dx, dy);"即可.
然后,再将on_scroll_event中的所有"on_next( NULL, mw );"改为"move_image(mw, 0, MOVE_SPACE);", "on_prev( NULL, mw );"改为"move_image(mw, 0, -MOVE_SPACE);",这样,鼠标滚轮操作就修改完成了.其中MOVE_SPACE为每次滚动时图片的移动距离,必须大于4,否则,图片将不移动,看move_image就知道了,我设置为10,这样不会显得太慢.
下面还需要修改左右方向键的响应事件,对应的响应函数为on_key_press_event,并将"case GDK_rightarrow:"中的"on_prev( NULL, mw );"改为"move_image(mw, -MOVE_SPACE, 0);", "on_next( NULL, mw );"改为"move_image(mw, MOVE_SPACE, 0);",同理,在"case GDK_leftarrow:"中按一样的方式替换即可.
恩,改造完成了,终于可以滚动滚轮和左右方向键移动图片了.
编译代码并运行了一下,发现在浏览正常尺寸下的图片(尺寸比窗口大)时,移动鼠标和滚轮图片都不动.但是,在放大图片时,滚轮和方向键的图片移动都能正常工作.
经过分析,这应该是图片在尺寸缩放时出现了BUG,于是,锁定文件image-view.c,阅读代码后,发现在paint函数中对缩放图片进行了绘制操作,作者对缩放比例为1时,没有对图片进行操作,于是,我先把这个if判断给注释掉,让其对缩放比例为1的图片也进行一次缩放操作,虽然效率会受到影响,但是应该能够工作.
再编译修改后的源码,并打开了一张图片,放大到正常尺寸(快捷键g),滚动滚轮和左右方向键,结果让我十分高兴,图像能够正常移动了,哈哈!
但是,目前,我还不清楚为什么缩放比例为1的判断过程会出现问题,暂时,我也不打算深究了,先用着,效率影响应该也不大.不过,有知道的童鞋一定要告诉我哟! :-)
注: 附件中为本次修改后的补丁