stan-dev/stan

Clean up code: `.str().length()`

开放

#2,577 创建于 2018年7月10日

在 GitHub 查看贡献者指南
 (13 条评论) (0 个反应) (1 位负责人)C++ (387 个派生)batch import
code cleanupgood first issue

仓库指标

星标
 (2,761 个星标)
PR 合并指标
 (30 天内没有已合并 PR)

描述

Summary:

There are many instances of code that looks like:

if (msg.str().length() > 0)
  logger.info(msg);

We should either:

  • have the info() method ignore empty messages in general
  • wrap the call in a function

Additional Information:

Originally from this PR comment: https://github.com/stan-dev/stan/pull/2570#discussion_r201151286

Current Version:

v2.17.1

贡献者指南

研究方向
在代码库中识别所有 `if (msg.str().length() > 0) logger.info(msg);` 模式的实例。评估两种方法：(1) 修改 info() 方法以忽略空消息，或 (2) 将模式包装在辅助函数中。一致地实现所选解决方案并相应更新测试。
技术栈
cpp
领域
backend
议题类型
重构
难度面向新贡献者的预计实现难度，1 表示很小改动，5 表示专家级工作。
2
预计时间有经验贡献者完成调查、实现、测试并准备 pull request 的粗略时间范围。
1-2 天
活动状态议题当前的可参与程度：新鲜、活跃、陈旧、阻塞或等待维护者输入。
活跃
清晰度议题是否清楚说明期望改动、验收标准和下一步。
清晰
前置要求
C++Git
新手友好度1-100 的估计分数，表示该议题对首次贡献者的友好程度。
60