进展:Pegasus的自动化编译测试

进展:Pegasus的自动化编译测试Pegasus以往只会在版本发布之前,进行少数几个编译器版本(如gcc4.8, gcc5.4)的测试。但众所周知,C++世界里,不同版本的编译器所产出的编译错误大有不同,有时gcc5.4.0能够顺利编译通过,但在clang9下却报出一连串errors。如果碰巧发版前,我们使用的编译器忽略了这些语法错误,那就留下了隐患,严重的甚至会影响数据一致性。- zauto_write_lock(app_lock());+ zauto_write_lock l(ap
摘要由CSDN通过智能技术生成

Pegasus以往只会在版本发布之前,进行少数几个编译器版本(如gcc4.8, gcc5.4)的测试。但众所周知,C++世界里,不同版本的编译器所产出的编译错误大有不同,有时gcc5.4.0能够顺利编译通过,但在clang9下却报出一连串errors。

如果碰巧发版前,我们使用的编译器忽略了这些语法错误,那就留下了隐患,严重的甚至会影响数据一致性。

-    zauto_write_lock(app_lock());
+    zauto_write_lock l(app_lock());

(这是一个在gcc5能顺利通过的语法错误)

同时作为Code Reviewer,单靠人眼很难仔细去检查每一行代码的细节,除非每个PR都pull下来用C++ IDE看一遍。那么有一种解决办法,就是让每个PR都跑全版本编译器测试

我们就是这么做的:

strategy:
  fail-fast: false
  matrix:
    
  • 0
    点赞
  • 0
    收藏
    觉得还不错? 一键收藏
  • 打赏
    打赏
  • 0
    评论

“相关推荐”对你有帮助么?

  • 非常没帮助
  • 没帮助
  • 一般
  • 有帮助
  • 非常有帮助
提交
评论
添加红包

请填写红包祝福语或标题

红包个数最小为10个

红包金额最低5元

当前余额3.43前往充值 >
需支付:10.00
成就一亿技术人!
领取后你会自动成为博主和红包主的粉丝 规则
hope_wisdom
发出的红包

打赏作者

Tao Wu

你的鼓励将是我创作的最大动力

¥1 ¥2 ¥4 ¥6 ¥10 ¥20
扫码支付:¥1
获取中
扫码支付

您的余额不足,请更换扫码支付或充值

打赏作者

实付
使用余额支付
点击重新获取
扫码支付
钱包余额 0

抵扣说明:

1.余额是钱包充值的虚拟货币,按照1:1的比例进行支付金额的抵扣。
2.余额无法直接购买下载,可以购买VIP、付费专栏及课程。

余额充值