A、代码规范。包括代码注释规范,代码风格:如缩进、换行、空行等实用,命名规范等,log输出是否符合要求
B、代码逻辑及技术实现。从技术实现层面给出专业的评审包括代码修改是否符合架构、解决方式是否合理、是否存在潜在的内存泄露、数组越界、空指针异常、,是否可能导致CTS fail等其他技术漏洞
C、提交模板规范。如issue description、test requirement、modify method等信息是否填写清晰是否符合规范要求
D、代码verify 时的自测内容。是否符合规范要求的数量,是否有按照规范填写自测用例及结果,自测用例是否覆盖较全面的场景
E、代码修改在手机上的实际运行效果确认
F、代码是否编译验证通过