谭明明吐吐舌头,也就她师傅这种猛人能说两天看完源代码了,她自己负责的模块当初都用了好几天才看明白。韩邵晨明明只负责高清图像处理模块,可现下这模样是要review所有模块包括系统流程的底层源码了,而且居然敢说他这么短的时间就能看完!
好吧,人家敢说也敢做。确认好操作之后,韩邵晨交代谭明明用测试机复现死机之前的所有操作步骤,打印出日志信息以供参考。自己则打开笔记本调出整个机顶盒的底层代码开始看了起来。
幸好对方的技术人员也很专业,操作也是按照他们的测试文档流程来的,复现步骤很容易,韩邵晨大致浏览了下就知道自己应该重点查看哪些操作相关的源代码。
还不到一个小时,韩邵晨突然抬头问谭明明,“最终发布之前每个模块代码做过联合review没?”
“好像有,但是因为催得太紧,大家可能都看得比较仓促。”很熟悉自己师傅说话风格的谭明明接着惊讶道,“师傅,你已经找到问题原因了吗?”
闻言,周围的人都看了过来。
“找到一个问题点,先修复重新测试。”韩邵晨已经在动手编译了,“我继续复查剩下的代码。”
“什么原因?”丁赤和郑硕辉几乎同时开口。
韩邵晨看了一眼郑硕辉,“源码漏洞,具体内容能不能透露需要项目经理同意。”
“切。出这么多问题还好意思说保密条例。”□□在一旁冷哼。
丁赤强笑着打圆场:“没关系,我先了解一下,后面再跟你们沟通。”说完就拉着韩邵晨带上笔记本找了个没人的角落。
谭明明和郭卫东默默的跟上。
韩邵晨点了点屏幕,“有人把线程退出的函数里的一句关键代码注释了,所有线程都没有退出,随着用户的操作线程不断产生,不断占用处理器和更多内存资源,即使在待机程序调用之后线程资源也不会被释放,机器运行的时间越长,占用的就越多,到最后系统必然会没反应。”至于这种错误能不能透露给客户他就不知道了。
“简直啪啪打脸。”谭明明接了句,“我的个天,这注释是谁码上去的,一点都不符合代码规范,还有这部分代码谁review的,扣他们奖金!”
即使不大懂软件,也能明白那两条斜线摆在一句代码前就会把这句代码的作用废除掉,不知道是谁出于什么目的有意还是无意的随手这么敲了两下,就让他们如此大动干戈。真是郁闷到顶点。
几个人面面相觑(除了说完话又开始埋头看代码的韩邵晨),丁赤叹口气,“就跟他们说我们线程处理的有点小问题,具体原因就别说了,不然简直让人家怀疑我们的专业性。”
“其实也不能全怪咱们的开发人员,”郭卫东说了句公道话,“我看了你们项目上每步流程,时间压缩得太厉害,压力之下有疏漏确实可以理解,谁知道会导致这么严重的问题。”
再说下去就是骂大领导了,谭明明缩缩脑袋,不敢参与他们的对话,低头问韩邵晨,“师傅,还有多久能编译完?”
“估计到半夜了,内核改动,上层代码也要一起重新编译。不过这个时间没多少人在用服务器,可能会快一点。”
“你真的还要把剩下的代码全看一遍吗?”谭明明小声问,“估计就是这个原因了,问题应该修复了吧。”
韩邵晨看了她一眼,“这个项目我只参与了图像处理和一些突发状况的解决,并没有浏览全部代码。”言下之意就是他不能保证其它地方没问题
“一般人类看一遍也不能保证啊。”谭明明偷偷吐槽。
“比不看好。”韩邵晨淡淡的道。
吐槽被听到的谭明明也没流露出害怕的表情,笑嘻嘻地去拿了台空闲的机顶盒过来等着刷新修复后的系统,顺便问了句,“那我还要继续操作打印log吗?”
“继续。”韩邵晨是真的没听懂谭明明想要偷懒的意思。
“哦。”谭明明闻言只好继续去对着测试机按遥控器去了。
接近午夜时分,对方的测试人员都走得差不多了,只剩下了组长郑硕辉还在对着文档里的测试用例按着机顶盒遥控器各种换台,M通讯的技术人员对加班到深夜已是习以为常,即使是在别人的地盘上也毫无影响,丁赤和郭卫东捧着笔记本在处理邮件,同样一点下班的意思也没有。
“啊,还没有下班?”门口响起来林宇的声音,“这么有干劲,问题有点眉目了吗?”正对着电视节目发呆的郑硕辉惊讶地差点跳起来,怎么大领导这么晚了还来视察?如果是产研部的也就罢了,市场部的经理这么密切关注新产品研发是要闹哪样?
“找出一点儿,已经修复了,新的系统刷在这两台机顶盒里,先测着,韩工他们那边还在排查可能会出问题的地方。”郑硕辉强打精神回答,“目前还没有再发现新的问题。”
“我记得之前你们说除了长时间运行会没反应以外,别的测试都已经过了?”林宇看了眼一边还在全神贯注看着笔记本屏幕的韩邵晨,虽然很想过去搭话,但还是要把正事说完。
“之前是这样。”郑硕辉挠挠头,“不过因为修复之后有可能又会带来新问题,理论上应该重新再测一遍。”
“怎么这个修改会涉及到别的操作吗?”林宇一挑眉。