diff options
| author | Ran Benita <ran234@gmail.com> | 2019-03-24 14:05:17 +0200 |
|---|---|---|
| committer | GitHub <noreply@github.com> | 2019-03-24 14:05:17 +0200 |
| commit | db7e79e78df07723dd07c8d00283244092dd8816 (patch) | |
| tree | d74f596f07c318137264dc44448d7a9109b31179 /src | |
| parent | 83cfbf8840722bb56112b90a682ebc3e063edcea (diff) | |
| parent | 75d1110c22746bf22c88bbecb01935ae98fc3c03 (diff) | |
| download | xorg-lib-libxkbcommon-db7e79e78df07723dd07c8d00283244092dd8816.tar.gz | |
Merge pull request #95 from Hi-Angel/fix-lgtm-warnings
Fix some of LGTM warnings
Diffstat (limited to 'src')
| -rw-r--r-- | src/xkbcomp/symbols.c | 3 |
1 files changed, 3 insertions, 0 deletions
diff --git a/src/xkbcomp/symbols.c b/src/xkbcomp/symbols.c index 9b05ec9..f693bae 100644 --- a/src/xkbcomp/symbols.c +++ b/src/xkbcomp/symbols.c @@ -227,6 +227,7 @@ MergeGroups(SymbolsInfo *info, GroupInfo *into, GroupInfo *from, bool clobber, /* First find the type of the merged group. */ if (into->type != from->type) { if (from->type == XKB_ATOM_NONE) { + /* it's empty for consistency with other comparisons */ } else if (into->type == XKB_ATOM_NONE) { into->type = from->type; @@ -269,6 +270,7 @@ MergeGroups(SymbolsInfo *info, GroupInfo *into, GroupInfo *from, bool clobber, struct xkb_level *fromLevel = &darray_item(from->levels, i); if (fromLevel->action.type == ACTION_TYPE_NONE) { + /* it's empty for consistency with other comparisons */ } else if (intoLevel->action.type == ACTION_TYPE_NONE) { intoLevel->action = fromLevel->action; @@ -290,6 +292,7 @@ MergeGroups(SymbolsInfo *info, GroupInfo *into, GroupInfo *from, bool clobber, } if (fromLevel->num_syms == 0) { + /* it's empty for consistency with other comparisons */ } else if (intoLevel->num_syms == 0) { intoLevel->num_syms = fromLevel->num_syms; |