太常见了，我上家公司，你不这么写，基本review不过。

【 在 z16166 的大作中提到: 】
: 搞两个const来声明一个变量的代码，很少见啊。
: 虽然说法没错，但是太啰嗦了，代码里到处是这种双const的话，扎眼
--
FROM 103.19.214.*

本老前几天刚搜了一下代码库，没有一个地方这么用的。
不过code review不就是那么回事，一般这种要求本老都是直接ACK的，有来回扯皮的时间下一个patch都写好了。
【 在 freyoneby 的大作中提到: 】
: 一定要函数参数加个const，比如conf（const char＊ path）改成conf（const char＊ const path），我说参数的值改不了，没必要加，他说能加const地方都要加，想问一下参数要么进栈要么进寄存器，const是怎么作用到寄存器上的？
: - 来自 水木社区APP v3.5.7
--
FROM 43.254.66.*

如果想反驳，可以用代码风格来说事儿。
比如说之前的旧代码都是这个风格，为了保持风格统一，建议不改。
如果要改，可以再另起一个修改，把整个项目全部都改一遍。

【 在 slowaction 的大作中提到: 】
: reviewer的意思就是你别给这个path重新赋值
: 不是什么大问题
: 对方提不提都不是问题，
: 如果提了，你非要说他说的不对，那更没必要
: 说什么要复用这个变量那就是搞笑了
--
FROM 112.97.61.*

要不要给 path 赋值，看代码就看出来了呀。
那反过来看，对方写了什么代码，是否都加了 const，
大家总得一致吧。
有规范就按规范来，
没规范要么我学你要么你学我，
你自己不这么干偏让我这么干，我是否可以合理推测你就是在有意针对我呢？
如果你是有意针对我，那就要看看行政序列，
如果我找别人 review 也可以，那我肯定就不找你了呀。
如果你不同意这事儿就不算完，那就看你是不是我领导了。

【 在 slowaction 的大作中提到: 】
: 这个理由不够充分
: 对方的出发点是不要给path赋值
: 这是个功能性的建议
: 而不是类似命名规则 缩进之类的风格问题
--
FROM 112.97.61.*

关键我好奇他自己是不是真的这么写。
严以律人宽以待己就不好了。
他要真这么写，如果他行政序列比较靠前，follow 他也没错呀。
关键他真这么写吗？
还有，他是否对任何人不分场合都会提这个意见？

【 在 webhost 的大作中提到: 】
: 某些老外就是吃饱了撑的，技术不咋地，整天揪着鸡毛蒜皮的小事要求别人一定要像他一样写法。
--
FROM 112.97.61.*