-
Notifications
You must be signed in to change notification settings - Fork 54
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
#459 Filters on object inventory #2508
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Might be a good idea to add a reset when user hides/reopens build tools or changes selection.
ead1d0c
to
7e8bc8b
Compare
indra/newview/llinventoryfilter.cpp
Outdated
@@ -47,7 +47,7 @@ | |||
// linden library includes | |||
#include "llclipboard.h" | |||
#include "lltrans.h" | |||
|
|||
#pragma optimize("", off) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
this fails mac build
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Fixed, thanks!
7e8bc8b
to
a3d37ca
Compare
a3d37ca
to
166c782
Compare
166c782
to
da46027
Compare
std::shared_ptr<LLMoveInv> move_inv(new LLMoveInv); | ||
move_inv->mObjectID = object_id; | ||
move_inv->mCategoryID = category_id; | ||
move_inv->mCallback = callback; | ||
move_inv->mUserData = user_data; | ||
|
||
for ( ; it != end; ++it) | ||
for (LLPointer<LLInventoryObject> obj : inventory_objects) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
this should have been a reference
return false; | ||
|
||
if (!isAgentAvatarValid()) | ||
return false; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Why does this need to check isAgentAvatarValid()?
The filter editor looks like follows