search

Tencent / flare

Flare是广泛投产于腾讯广告后台的现代化C++开发框架，包含了基础库、RPC、各种客户端等。主要特点为易用性强、长尾延迟低。
Other
1.33k stars 200 forks source link

FLARE_CHECK 等字符过多，可以简化 #23

Closed pedia closed 3 years ago

pedia commented 3 years ago

首先要说代码质量相当不错，功能也相当充分，完全可以作为基础库了。

提个小建议

FLARE_LOG_INFO FLARE_LOG_WARNING FLARE_CHECK FLARE_LIKELY FLARE_DCHECK FLARE_LOG_WARNING_EVERY_SECOND

等宏名字过于冗长，可以大气一些直接改为 FLOG_INFO, FCHECK 等

namespace flare 可能会用得比较多，如果有更简洁的方案，当采用之

0x804d8000 commented 3 years ago

现在改名的可行性不太大，内部代码用的比较多了。

当初确实会担心有点长，不过用久了之后感觉也还好，可能是现在的ide的代码补全做的比较好。

有必要的话考虑自己#define一次？

pedia commented 3 years ago

OK, 只是建议而已