10 Star 12 Fork 5

OpenHarmony / hiviewdfx_interfaces_innerkits_hilog

【讨论】hilog_cp.h中c++代码使用typedef struct HiLogLabel 定义结构体类型HiLogLabel 没有必要

任务
已完成
Gymee  Opened this issue

该问题是怎么引起的?

重现步骤

报错信息

c++代码中没有必要使用typedef struct XXX定义结构体类型,直接使用struct XXX就可以了
也可以直接引用类型XXX

#ifdef __cplusplus

namespace OHOS {
namespace HiviewDFX {
typedef struct HiLogLabel { // 没有必要使用typedef
    LogType type;
    unsigned int domain;
    const char *tag;
} HiLogLabel;

修改为

#ifdef __cplusplus

namespace OHOS {
namespace HiviewDFX {
struct HiLogLabel {
    LogType type;
    unsigned int domain;
    const char *tag;
}

另外建议:HiLogLabel的每个成员赋一个初值

如果有不对的地方,麻烦指出来

Attachments
5200794 stesen 1599180294 total 2 participants

Comments (3)

5200794 stesen 1599180294
stesen 2020-09-15 10:55 member

去掉typedef的建议是合理的,你看是你提个PR还是我这边提?
初值不应当赋,避免调用者滥用初值,对日志统计不利

Gymee 2020-09-15 11:23

去掉typedef的建议是合理的,你看是你提个PR还是我这边提?
初值不应当赋,避免调用者滥用初值,对日志统计不利

@stesen 我这边提一个PR吧

Sign in to comment

Assignees
Labels
Not set
Projects
Milestones
Branches
Planed to start
Not set
Planed to end
Not set
Top level
Priority
1
https://git.oschina.net/openharmony/hiviewdfx_interfaces_innerkits_hilog.git
git@git.oschina.net:openharmony/hiviewdfx_interfaces_innerkits_hilog.git
openharmony
hiviewdfx_interfaces_innerkits_hilog
hiviewdfx_interfaces_innerkits_hilog

Search