【吐槽】从loongson 源码静态检测来看，代码有多脏
如题，使用的工具是开源cppcheck
检测的目标代码是pmon。一大堆警告与错误提示，另外从代码看肯定也没有做过代码覆盖类的测试。有不少断头代码。
上图：
检测的目标代码是pmon。一大堆警告与错误提示，另外从代码看肯定也没有做过代码覆盖类的测试。有不少断头代码。
上图：
没有找到相关结果
已邀请:
6 个回复
water
赞同来自: dawoo
xen0n
赞同来自: dawoo
dawoo
赞同来自: zzz19760225
编码风格在sf.net上已经有小程序了。astyle
water
赞同来自: zzz19760225
https://www.cnblogs.com/jiangxinnju/p/4908575.html
批量格式化
for /R %f in (*.cpp;*.cs;) do D:\OpenSource\AStyle_3.0.1_windows\AStyle\bin\AStyle.exe --style=ansi "%f"
water
赞同来自:
void ls_pcie_mem_fixup(struct pci_config_data *pdata)
+{
+ unsigned int dev;
+ unsigned int val;
- /*DC*/
- dev = _pci_make_tag(0, 6, 0);
+ dev = _pci_make_tag(pdata->bus, pdata->dev, pdata->func);
val = _pci_conf_read32(dev, 0x00);
- if ( val != 0xffffffff) // device on the slot
- _pci_conf_write8(dev, 0x3c, 92);
+ /* device on the slot */
+ if ( val != 0xffffffff){
+ if(pdata->type == PCI_DEV){
+ /*write bar*/
+ _pci_conf_write32(dev, 0x10, pdata->start);
+ }else{
+ /*write memory base and memory limit*/
+ val = ((pdata->start >> 16)&0xfff0)|(pdata->end&0xfff00000);
+ _pci_conf_write32(dev, 0x20, val);
+ _pci_conf_write32(dev, 0x24, val);
+ }
+ }
+}
看这个最新提交的代码
感觉 对指针操作 没有先校验再使用的习惯
flygoatfree
赞同来自: