If the "if" block is more than one line long, braces should be used, even
if it doesn't make a semantical difference. I can't believe I've written that code. git-svn-id: file:///srv/svn/repos/haiku/haiku/trunk@16162 a95241bf-73f2-0310-859d-f6bbb57e9c96
This commit is contained in:
parent
7e46f8319e
commit
1ba1f4be4a
@ -2094,7 +2094,7 @@ BRoster::translate_type(const char *mimeType, BMimeType *appMeta,
|
||||
error = type.SetTo(mimeType);
|
||||
// get the preferred app
|
||||
char appSignature[B_MIME_TYPE_LENGTH];
|
||||
if (error == B_OK)
|
||||
if (error == B_OK) {
|
||||
if (type.IsInstalled()) {
|
||||
BMimeType superType;
|
||||
if (type.GetPreferredApp(appSignature) != B_OK
|
||||
@ -2109,6 +2109,7 @@ BRoster::translate_type(const char *mimeType, BMimeType *appMeta,
|
||||
// The type is not installed. We assume it is an app signature.
|
||||
strcpy(appSignature, mimeType);
|
||||
}
|
||||
}
|
||||
if (error == B_OK)
|
||||
error = appMeta->SetTo(appSignature);
|
||||
// check, whether the signature is installed and has an app hint
|
||||
|
Loading…
x
Reference in New Issue
Block a user