• 社区首页
  • 版块
dde-dock的一个编码bug?
体验心得55个浏览 ·3个评论
电梯直达到第
花心胡萝卜
deepin^4
发表于2020-10-23 08:04
楼主
在对dde-dock魔改的过程中, 发现,
frame/item/components/previewcontainer.cpp 中的 setWindowInfos 函数

  1. if (m_snapshots.isEmpty())
  2.         emit requestCancelPreviewWindow();
  3.         emit requestHidePopup();
Copy the Code



从代码格式上来说, 两个emit应该是属于 if 语句的

但是if并没有大括号, 所以从代码上来说, 第二个emit无论if条件是不是成立都会被执行.

那么,,,, 这到底是一个编码上的bug呢, 还是本来就应该无论什么时候都执行呢....

这个我没深入研究, 我加上大括号, 和现在的dock表现也没什么区别... 或者说有 我没看出来...

https://bbs.deepin.org/user/48640 让开发研究一下?

回复收藏只看作者
所有回复
juteman深度攻城组
发表于2020-10-23 09:40
沙发
花心胡萝卜
deepin^4
发表于2020-10-23 09:44
板凳

看来我得记住你俩, 你俩是开发组的!
owen1759
deepin^1
发表于2020-10-23 09:58
地板
没看代码,从方法名来看,应该是“取消预览窗口”和“隐藏popup”。从这段逻辑看应该是属于if里面的。
如果确实如此,那么BUG症状将会是“无论snapshot是否是空的,popup都被隐藏不显示”。
那么如果你观察不到区别,可能是本来就没有显示popup
至于这个popup指的是什么东西,我不太清楚,不知道是指预览窗口的显示窗口名的那个tips,还是指预览窗口的那个气泡
发布帖子

光荣榜

  • 周榜
  • 月榜
  • 总榜