Commit message (Collapse) | Author | Age | Files | Lines | |
---|---|---|---|---|---|
* | Rename UpdatePrimRotation() to UpdatePrimGroupRotation() since this is what ↵ | Justin Clark-Casey (justincc) | 2011-07-19 | 2 | -7/+11 |
| | | | | it actually does and is more consistent with other method names. | ||||
* | Stop undo of just the root prim position in the linkset from shifting the ↵ | Justin Clark-Casey (justincc) | 2011-07-19 | 3 | -9/+15 |
| | | | | | | whole linkset. However, what happens now is that undo just doesn't do anything when the root prim is selected on its own. This requires more code than just fiddling with undo states. | ||||
* | fix undo when resizing of non-root individual prims in a linkset | Justin Clark-Casey (justincc) | 2011-07-19 | 2 | -4/+6 |
| | | | | undo resize, rotation and position still needs fixing when only editing root prim of a linkset | ||||
* | Fix undo for resizing linksets | Justin Clark-Casey (justincc) | 2011-07-19 | 3 | -29/+60 |
| | | | | | | This involves implementing a boolean in UndoState to signal whether the undo needs to be done for an entire group/linkset or just a single prim Resizing individual components of linksets is still dodgy. Resizing still has to be down twice, since for some reason the client is sending two multiobjectupdate packets on every resize except the very first. This applies to single prims and linksets. Need to look into this. | ||||
* | remove undo state storage in a few places where it's pointless | Justin Clark-Casey (justincc) | 2011-07-19 | 2 | -5/+1 |
| | | | | no functional effect - existing bugs still remain | ||||
* | Make various tweaks to undo code in an effort to get things working better. | Justin Clark-Casey (justincc) | 2011-07-18 | 5 | -53/+193 |
| | | | | | | | Undo rotation and position appear to be working. Resizing a single prim appears to be working, though the undo has to be done twice. Resizing a group of prims still does not work properly - possibly because in the UndoState we don't store a knowledge of when we're resizing a whole group rather than individual prims. This needs to be addressed. | ||||
* | Use a standard generic system stack for the undo/redo stacks instead of our ↵ | Justin Clark-Casey (justincc) | 2011-07-18 | 2 | -2/+16 |
| | | | | | | | own homebrew. system stack also uses an array, so no performance penalty. Also exposes undo count and adds a test assertion for correct undo count after resize | ||||
* | refactor: group all the undo/redo code in SOP into one place for easier code ↵ | Justin Clark-Casey (justincc) | 2011-07-18 | 1 | -59/+62 |
| | | | | reading | ||||
* | Replace ifs in SOG.GroupResize() with Math.Min() | Justin Clark-Casey (justincc) | 2011-07-16 | 3 | -14/+25 |
| | | | | | Also fiddle a bit with undo. This is not currently working properly, though to be fair it also didn't appear to work in 0.7.1.1 either (at least for resize). Will get some more attention soon. | ||||
* | remove now unncessary parts of SceneObjectGroup.Resize() | Justin Clark-Casey (justincc) | 2011-07-16 | 1 | -21/+5 |
| | |||||
* | refactor: replace scale limiting code with more elegant Math.Min calls | Justin Clark-Casey (justincc) | 2011-07-16 | 1 | -12/+6 |
| | |||||
* | refactor: Push all part resize code down into SceneObjectPart.Resize() | Justin Clark-Casey (justincc) | 2011-07-16 | 4 | -56/+32 |
| | |||||
* | add test for resizing one part in a group | Justin Clark-Casey (justincc) | 2011-07-16 | 2 | -1/+36 |
| | |||||
* | refactor: make SceneObjectGroup.GroupScale() a property rather than a mehod | Justin Clark-Casey (justincc) | 2011-07-16 | 1 | -29/+32 |
| | |||||
* | minor: add a log warning if a sculpt/mesh async asset request returns no data | Justin Clark-Casey (justincc) | 2011-07-16 | 1 | -0/+4 |
| | |||||
* | refactor: remove pointless sender != null tests, etc, in AssetReceived, ↵ | Justin Clark-Casey (justincc) | 2011-07-16 | 1 | -6/+9 |
| | | | | since the method called always belongs to the object that generated the request | ||||
* | eliminate unused and redundant SceneObjectGroup.AssetReceived() | Justin Clark-Casey (justincc) | 2011-07-16 | 1 | -17/+0 |
| | |||||
* | fix build break from last commit | Justin Clark-Casey (justincc) | 2011-07-16 | 1 | -1/+1 |
| | |||||
* | Eliminate the pointless textured id argument to SculptTextureCallback | Justin Clark-Casey (justincc) | 2011-07-16 | 1 | -15/+6 |
| | |||||
* | remove the need to supply SceneObjectGroup.GroupResize() with a localId. | Justin Clark-Casey (justincc) | 2011-07-16 | 3 | -132/+133 |
| | | | | This is utterly pointless scene we already know which sog we're dealing with. | ||||
* | Add very basic test for resizing a scene object with one prim | Justin Clark-Casey (justincc) | 2011-07-16 | 2 | -1/+69 |
| | |||||
* | If resized shape is a mesh/sculpt, leave it to the mesh asset callback to ↵ | Justin Clark-Casey (justincc) | 2011-07-16 | 1 | -2/+8 |
| | | | | | | trigger the physics actor taint. In the last commit, the fix was made by updating all the child prim physics actors with the new size rather than just the root part. | ||||
* | Fix physics proxy regeneration when a mesh with more than one submesh is resized | Justin Clark-Casey (justincc) | 2011-07-16 | 2 | -1/+20 |
| | | | | Addresses http://opensimulator.org/mantis/view.php?id=5584 | ||||
* | Fix some local id issues in physics glue | root | 2011-07-15 | 2 | -14/+22 |
| | |||||
* | fix duplication of physical objects for physics engines that care about the ↵ | Robert Adams | 2011-07-15 | 1 | -6/+6 |
| | | | | initial value of localID | ||||
* | When a mesh object is added to a scene, delay adding the physics actor until ↵ | Justin Clark-Casey (justincc) | 2011-07-12 | 3 | -4/+10 |
| | | | | | | the sculpt data has been added to the shape (possibly via an async asset service request) This prevents spurious 'no asset data' for meshes added on startup. | ||||
* | Properly regenerate physics proxy when a mesh is resized. | Justin Clark-Casey (justincc) | 2011-07-11 | 1 | -0/+8 |
| | | | | This is done in SOP.Resize(). More common code from callers needs to be refactored into this method to reduce confusing copy/pasting | ||||
* | refactor: push the part of SceneObjectGroup.CheckSculptAndLoad() that ↵ | Justin Clark-Casey (justincc) | 2011-07-11 | 2 | -17/+49 |
| | | | | actually deals with the part into a SceneObjectPart.CheckSculptAndLoad() method | ||||
* | When a sculpt/mesh texture is received by a part on a callback request, ↵ | Justin Clark-Casey (justincc) | 2011-07-11 | 3 | -9/+23 |
| | | | | | | | don't do the unnecessary work of copying the base shape. Just setting the new base shape is enough to reinsert the sculpt data and set the taint. Also cleans up a few more left-in debugging messages. | ||||
* | refactor: Make arguments to SceneGraph.UpdatePrimFlags() more readable | Justin Clark-Casey (justincc) | 2011-07-11 | 1 | -6/+11 |
| | |||||
* | refactor: Make arguments for SceneObjectGroup.UpdatePrimFlags() more readable | Justin Clark-Casey (justincc) | 2011-07-11 | 1 | -6/+8 |
| | |||||
* | minor: remove mono compiler warning | Justin Clark-Casey (justincc) | 2011-07-11 | 1 | -1/+1 |
| | |||||
* | refactor: very minor space insertion | Justin Clark-Casey (justincc) | 2011-07-11 | 1 | -1/+1 |
| | |||||
* | refactor: make argument to SOP.UpdatePrimFlags() more readable | Justin Clark-Casey (justincc) | 2011-07-11 | 1 | -14/+19 |
| | |||||
* | Rename SceneSetupHelpers.AddRootAgent to AddClient() to better represent its ↵ | Justin Clark-Casey (justincc) | 2011-07-09 | 6 | -7/+7 |
| | | | | effects and return object | ||||
* | minor: code tidy and inserted log lines for future use. | Justin Clark-Casey (justincc) | 2011-07-09 | 2 | -1/+2 |
| | | | | | | | | Unable to get to the bottom of why resizing a mesh fails to properly reset the physics proxy, when toggling phantom does After a mesh is generated, the existing sculptdata is set to zero in PrimitiveBaseShape to save memory When phantom is toggled, the sculptdata is regenerated before remeshing. But on resize, the sculptdata is not regenerated. So clearly, resetting sculptdata is possible, but haven't quite been able to pin down how this is being done when phantom is toggled. | ||||
* | refactor: rename bool returning GetAgentInventoryItem() to ↵ | Justin Clark-Casey (justincc) | 2011-07-08 | 1 | -1/+9 |
| | | | | CanGetAgentInventoryItem() to improve code readability | ||||
* | minor: commented out log lines for future use and very small code tidy | Justin Clark-Casey (justincc) | 2011-07-08 | 1 | -0/+4 |
| | |||||
* | Create a very basic initial test which just creates an 'npc' and tests that ↵ | Justin Clark-Casey (justincc) | 2011-07-02 | 1 | -0/+40 |
| | | | | the scene presence exists | ||||
* | fix build break I just introduced | Justin Clark-Casey (justincc) | 2011-07-01 | 1 | -2/+1 |
| | |||||
* | Add an async inventory details sender to respond to FetchInventory packets. | Justin Clark-Casey (justincc) | 2011-07-01 | 4 | -27/+164 |
| | | | | | | | If a user with a very large inventory right-clicks on their "My Inventory" folder, viewer 1 code will send a massive number of Fetchinventory requests. Even though each is handled asynchronously via a pool thread, the sheer frequency of requests overwhelms the pool and freezes inbound packet handling. This change makes the first Fetchinventory thread also handle subsequent requests, freeing up the other threads. Further efficiencies could be made by handling all the items in a particular FetchInventory request together, rather than separately. | ||||
* | Don't follow inventory links of links. | Justin Clark-Casey (justincc) | 2011-06-29 | 1 | -1/+4 |
| | | | | | This is to avoid problems with corrupt inventories where an inventory link target points back at the source's folder No viewer has been observed to set these up as of yet. If this ever happens, we will need a more sophisticated solution to track sent folders within the recursion | ||||
* | refactor: simplify redundant double containing folder check | Justin Clark-Casey (justincc) | 2011-06-29 | 1 | -2/+4 |
| | |||||
* | If an inventory link target is in the same folder as the source, then don't ↵ | Justin Clark-Casey (justincc) | 2011-06-29 | 1 | -1/+6 |
| | | | | | | recursively request that folder. Currently, this should never actually happen but certainly best to handle this case | ||||
* | Hack around with the NPC module to get osNpcCreate() partially working again. | Justin Clark-Casey (justincc) | 2011-06-29 | 3 | -6/+6 |
| | | | | | | This now creates an avatar but appearance is always cloudy. Move doesn't work. Really, creating an NPC should only involve a ScenePresence rather than doing anything with IClientAPI, since an NPC has no viewer to communicate with! | ||||
* | Implement the latest mesh mechanism so that rezzing the uploaded mesh now ↵ | Justin Clark-Casey (justincc) | 2011-06-24 | 1 | -3/+6 |
| | | | | | | | works again. Many thanks to the aurora project for pioneering this. This code is almost certainly not bug free, but it does at least appear to handle simple meshes (except when the viewer crashes - but it is beta!). | ||||
* | Add localID to physical object creation functions. | Mic Bowman | 2011-06-15 | 3 | -4/+4 |
| | |||||
* | Add the PhysActor to the correct SOP when duplicating a physical | Mic Bowman | 2011-06-15 | 1 | -4/+9 |
| | | | | prim. Thanks, MisterBlue | ||||
* | When serializing objects, stop accidentally using the green text colour ↵ | Justin Clark-Casey (justincc) | 2011-06-10 | 1 | -1/+1 |
| | | | | | | value for alpha This addresses http://opensimulator.org/mantis/view.php?id=5111 | ||||
* | minor: Add some commented out destructor logging messages for potential ↵ | Justin Clark-Casey (justincc) | 2011-06-10 | 2 | -0/+12 |
| | | | | | | future use. At the moment, client and scene objects are being garbage collected as expected, at least in simple scenarios. |