i2p/haiku-beta6
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Fix BMessage::GetInfo() return value check

Browse Source 
Since B_OK is 0, the change doesn't affect the semantics.
This commit is contained in:
Ingo Weinhold
2013-04-16 17:08:45 +02:00
parent 88f22c9768
commit 4210ed011a
2 changed files with 4 additions and 4 deletions
Download Patch File Download Diff File Expand all files Collapse all files
src/kits/package/DaemonClient.cpp
+1 -1
View File
@@ -107,7 +107,7 @@ BDaemonClient::_ExtractPackageInfoSet(const BMessage& message,
// get the number of items // get the number of items
type_code type; type_code type;
int32 count; int32 count;
if (message.GetInfo(field, &type, &count)) { if (message.GetInfo(field, &type, &count) != B_OK) {
// the field is missing // the field is missing
return B_OK; return B_OK;
} }
src/kits/package/PackageInfo.cpp
+3 -3
View File
@@ -2033,7 +2033,7 @@ BPackageInfo::_ExtractStringList(BMessage* archive, const char* field,
// get the number of items // get the number of items
type_code type; type_code type;
int32 count; int32 count;
if (archive->GetInfo(field, &type, &count)) { if (archive->GetInfo(field, &type, &count) != B_OK) {
// the field is missing // the field is missing
return B_OK; return B_OK;
} }
@@ -2071,7 +2071,7 @@ BPackageInfo::_ExtractResolvables(BMessage* archive, const char* field,
// get the number of items // get the number of items
type_code type; type_code type;
int32 count; int32 count;
if (archive->GetInfo(nameField, &type, &count)) { if (archive->GetInfo(nameField, &type, &count) != B_OK) {
// the field is missing // the field is missing
return B_OK; return B_OK;
} }
@@ -2130,7 +2130,7 @@ BPackageInfo::_ExtractResolvableExpressions(BMessage* archive,
// get the number of items // get the number of items
type_code type; type_code type;
int32 count; int32 count;
if (archive->GetInfo(nameField, &type, &count)) { if (archive->GetInfo(nameField, &type, &count) != B_OK) {
// the field is missing // the field is missing
return B_OK; return B_OK;
} }