Skip to content

Commit 3cbafd0

Browse files
committed
Add braces to avoid dangling-else ambiguity
The C compiler does not care about indentation...
1 parent 11fe20a commit 3cbafd0

File tree

1 file changed

+20
-0
lines changed

1 file changed

+20
-0
lines changed

src/ansi-c/c_typecheck_type.cpp

Lines changed: 20 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -133,42 +133,56 @@ void c_typecheck_baset::typecheck_type(typet &type)
133133
typet result;
134134

135135
if(gcc_attr_mode == "__QI__") // 8 bits
136+
{
136137
if(is_signed)
137138
result=signed_char_type();
138139
else
139140
result=unsigned_char_type();
141+
}
140142
else if(gcc_attr_mode == "__byte__") // 8 bits
143+
{
141144
if(is_signed)
142145
result=signed_char_type();
143146
else
144147
result=unsigned_char_type();
148+
}
145149
else if(gcc_attr_mode == "__HI__") // 16 bits
150+
{
146151
if(is_signed)
147152
result=signed_short_int_type();
148153
else
149154
result=unsigned_short_int_type();
155+
}
150156
else if(gcc_attr_mode == "__SI__") // 32 bits
157+
{
151158
if(is_signed)
152159
result=signed_int_type();
153160
else
154161
result=unsigned_int_type();
162+
}
155163
else if(gcc_attr_mode == "__word__") // long int, we think
164+
{
156165
if(is_signed)
157166
result=signed_long_int_type();
158167
else
159168
result=unsigned_long_int_type();
169+
}
160170
else if(gcc_attr_mode == "__pointer__") // size_t/ssize_t, we think
171+
{
161172
if(is_signed)
162173
result=signed_size_type();
163174
else
164175
result=size_type();
176+
}
165177
else if(gcc_attr_mode == "__DI__") // 64 bits
166178
{
167179
if(config.ansi_c.long_int_width==64)
180+
{
168181
if(is_signed)
169182
result=signed_long_int_type();
170183
else
171184
result=unsigned_long_int_type();
185+
}
172186
else
173187
{
174188
assert(config.ansi_c.long_long_int_width==64);
@@ -180,11 +194,14 @@ void c_typecheck_baset::typecheck_type(typet &type)
180194
}
181195
}
182196
else if(gcc_attr_mode == "__TI__") // 128 bits
197+
{
183198
if(is_signed)
184199
result=gcc_signed_int128_type();
185200
else
186201
result=gcc_unsigned_int128_type();
202+
}
187203
else if(gcc_attr_mode == "__V2SI__") // vector of 2 ints, deprecated
204+
{
188205
if(is_signed)
189206
result=vector_typet(
190207
signed_int_type(),
@@ -193,7 +210,9 @@ void c_typecheck_baset::typecheck_type(typet &type)
193210
result=vector_typet(
194211
unsigned_int_type(),
195212
from_integer(2, size_type()));
213+
}
196214
else if(gcc_attr_mode == "__V4SI__") // vector of 4 ints, deprecated
215+
{
197216
if(is_signed)
198217
result=vector_typet(
199218
signed_int_type(),
@@ -202,6 +221,7 @@ void c_typecheck_baset::typecheck_type(typet &type)
202221
result=vector_typet(
203222
unsigned_int_type(),
204223
from_integer(4, size_type()));
224+
}
205225
else // give up, just use subtype
206226
result=type.subtype();
207227

0 commit comments

Comments
 (0)