There is really no reason at all to mess up the current sorting
of the attributes just by adding a new attribute. git-svn-id: file:///srv/svn/repos/haiku/haiku/trunk@40379 a95241bf-73f2-0310-859d-f6bbb57e9c96
This commit is contained in:
parent
c8cbb2763f
commit
a416707fb3
@ -43,20 +43,6 @@ const uint32 kMsgAlignmentChosen = 'alnc';
|
|||||||
const uint32 kMsgAccept = 'acpt';
|
const uint32 kMsgAccept = 'acpt';
|
||||||
|
|
||||||
|
|
||||||
static int
|
|
||||||
compare_attributes(const void* _a, const void* _b)
|
|
||||||
{
|
|
||||||
AttributeItem* a = *(AttributeItem **)_a;
|
|
||||||
AttributeItem* b = *(AttributeItem **)_b;
|
|
||||||
|
|
||||||
int compare = strcasecmp(a->PublicName(), b->PublicName());
|
|
||||||
if (compare != 0)
|
|
||||||
return compare;
|
|
||||||
|
|
||||||
return strcmp(a->Name(), b->Name());
|
|
||||||
}
|
|
||||||
|
|
||||||
|
|
||||||
static bool
|
static bool
|
||||||
compare_display_as(const char* a, const char* b)
|
compare_display_as(const char* a, const char* b)
|
||||||
{
|
{
|
||||||
@ -444,7 +430,6 @@ AttributeWindow::MessageReceived(BMessage* message)
|
|||||||
BList list;
|
BList list;
|
||||||
|
|
||||||
const char* newAttribute = fAttributeControl->Text();
|
const char* newAttribute = fAttributeControl->Text();
|
||||||
list.AddItem(_NewItemFromCurrent());
|
|
||||||
|
|
||||||
const char* attribute;
|
const char* attribute;
|
||||||
for (int32 i = 0; attributes.FindString("attr:name", i,
|
for (int32 i = 0; attributes.FindString("attr:name", i,
|
||||||
@ -460,7 +445,7 @@ AttributeWindow::MessageReceived(BMessage* message)
|
|||||||
list.AddItem(item);
|
list.AddItem(item);
|
||||||
}
|
}
|
||||||
|
|
||||||
list.SortItems(compare_attributes);
|
list.AddItem(_NewItemFromCurrent());
|
||||||
|
|
||||||
// Copy them to a new message (their memory is still part of the
|
// Copy them to a new message (their memory is still part of the
|
||||||
// original BMessage)
|
// original BMessage)
|
||||||
@ -483,9 +468,10 @@ AttributeWindow::MessageReceived(BMessage* message)
|
|||||||
status = fMimeType.SetAttrInfo(&newAttributes);
|
status = fMimeType.SetAttrInfo(&newAttributes);
|
||||||
}
|
}
|
||||||
|
|
||||||
if (status != B_OK)
|
if (status != B_OK) {
|
||||||
error_alert(B_TRANSLATE("Could not change attributes"),
|
error_alert(B_TRANSLATE("Could not change attributes"),
|
||||||
status);
|
status);
|
||||||
|
}
|
||||||
|
|
||||||
PostMessage(B_QUIT_REQUESTED);
|
PostMessage(B_QUIT_REQUESTED);
|
||||||
break;
|
break;
|
||||||
|
Loading…
Reference in New Issue
Block a user