Aren/haiku
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

Fix incorrect use of _SendMonitorUpdate

Browse Source 
Icon size is only a boolean here (true for a large icon, false
otherwise). Using B_LARGE_ICON directly is not appropriate.

As found by Murai Takashi, but there was a second case in the same file
which I also fixed.

ticket : #12955
This commit is contained in:
Adrien Destugues 2016-10-09 21:25:53 +02:00
parent fe2557b6eb
commit 069c569704
1 changed files with 7 additions and 5 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

12
src/kits/storage/mime/Database.cpp
View File

@ -1114,9 +1114,10 @@ Database::DeleteIcon(const char *type, icon_size which)
{
const char *attr = which == B_MINI_ICON ? kMiniIconAttr : kLargeIconAttr;
status_t status = fLocation->DeleteAttribute(type, attr);
if (status == B_OK)
_SendMonitorUpdate(B_ICON_CHANGED, type, which, B_META_MIME_DELETED);
else if (status == B_ENTRY_NOT_FOUND)
if (status == B_OK) {
_SendMonitorUpdate(B_ICON_CHANGED, type, (which == B_LARGE_ICON),
B_META_MIME_DELETED);
} else if (status == B_ENTRY_NOT_FOUND)
status = B_OK;
return status;
@ -1135,10 +1136,11 @@ Database::DeleteIcon(const char *type, icon_size which)
status_t
Database::DeleteIcon(const char *type)
{
// TODO: extra notification for vector icon (uses B_LARGE_ICON now)
// TODO: extra notification for vector icon (for now we notify a "large"
// icon)
status_t status = fLocation->DeleteAttribute(type, kIconAttr);
if (status == B_OK) {
_SendMonitorUpdate(B_ICON_CHANGED, type, B_LARGE_ICON,
_SendMonitorUpdate(B_ICON_CHANGED, type, true,
B_META_MIME_DELETED);
} else if (status == B_ENTRY_NOT_FOUND)
status = B_OK;