| Commit message (Collapse) | Author | Age | Files | Lines |
|
|
|
| |
This is not used for anything - appearances are always properties of objects with ids (ScenePresence, AgentCircuitData) and just has the potential to get out of sync when the appearance is cloned.
|
|
|
|
|
|
| |
Had to stop using AvatarService for now since it doesn't store baked texture IDs (which is why this was failing).
Also failing because cloning appearance was also cloning the AvatarApperance.Owner field, which we weren't then changing.
Extended TestCreate() to check this.
|
|
|
|
| |
Fix build break.
|
|
|
|
|
|
| |
the Name property instead.
This is equivalent since the prim 'name' is never changed. In fact, this propery is never used for prims
|
| |
|
| |
|
| |
|
|
|
|
|
|
| |
or shape changed, also remove the OdeScene.actor_name_map entry pointing to the phys actor
This is to stop a small memory leak over time when prims are deleted or phantom-toggled
|
| |
|
|
|
|
| |
to the summary, it is being called from within Simulate() lock (OdeLock)
|
|
|
|
|
|
|
|
| |
with new prim_geom key, as the old one becomes invalid.
This resolves http://opensimulator.org/mantis/view.php?id=5603 where changing size or shape would stop collision_start being fired in a running script.
In both this and existing code we are not removing old actors from actor_name_map when the existing prim_geom is removed, which leads to a small memory leak over time.
This needs to be fixed.
|
| |
|
| |
|
|
|
|
| |
that we don't have to look at it if it's not relevant
|
|
|
|
| |
retrieved via GetDataBlock() then create a new stack instead of throwing an exception
|
|
|
|
|
|
| |
it's initially used.
This may improve memory usage for regions using mesh and sculpts, though I suspect that it doesn't address the current memory leak.
|
|
|
|
|
|
| |
is no longer required and it prevents removal of the _mesh field (which is only used temporarily)
If passing in a mesh becomes important again in the future then this can be reinstated.
|
| |
|
|
|
|
| |
this with the pre-existing _mesh field
|
| |
|
|
|
|
|
|
| |
some code reference the private var and other the public var without any functionality difference.
Add some method doc to IsPhysical
|
| |
|
| |
|
| |
|
|
|
|
| |
to reflect OpenSim standards.
|
| |
|
|
|
|
|
|
| |
it's own method.
fix build break.
|
| |
|
|
|
|
| |
into a separate method, in order to make the code more readable.
|
|
|
|
| |
specific stuff to return more information.
|
|
|
|
| |
this command is invoked.
|
|
|
|
|
|
|
| |
default Ruth height.
I was persuaded to do this because simulators on osgrid will persist in inserting bad values for an unknown length of time, even after the original simulator bug which was inserting bad values is out in an osgrid distro
This code can be removed at some point in the future, though I think there is an argument for having services police these values in open grids.
|
| |
|
|
|
|
|
|
|
|
|
|
| |
continuing wtih the invocation.
This is to avoid bugs where the locale is not manually set on the thread and bad data values get sent to the database or over the wire.
Lots of code does this manually but as we've seen, a subtle change can hit code which has forgotton to do this.
Since en_US show be used throughout the server at present, setting it at FireAndForget seems reasonable.
Arguably, it would be better to do this where data is sent, but doing it here is much easier.
All the manual BeginInvokes() remaining in the code should probably call FireAndForget instead.
|
|
|
|
|
|
|
| |
En_US so that a different culture doesn't save values with commas as decimal points, etc.
This will hopefully stop giants stalking the grid.
See http://opensimulator.org/mantis/view.php?id=5614
|
|
|
|
|
|
| |
data fails.
This is to address http://opensimulator.org/mantis/view.php?id=5612
|
|
|
|
| |
cleared on every OdeScene.Simulate() even if it was already empty.
|
|
|
|
| |
loop for consistency and readability.
|
|
|
|
|
|
| |
NINJA joints stuff into its own method.
Now if ninja joints isn't active (which is the default) don't have to wade through a lot of massively indented irrelevant code.
|
|
|
|
|
|
| |
out ifs and locks
This is to make it more readable.
|
|
|
|
|
|
|
|
| |
region name exactly matches (not near matches)
This is to prevent situations where the first name returned by GridService.GetRegionsByName is not one that exactly matches the given region name, even when there is an exact match later on in the list.
Only the above two functions call this teleport method (the map uses a different routine) so this seems safe to change.
Addresses http://opensimulator.org/mantis/view.php?id=5606
|
| |
|
| |
|
|
|
|
|
|
|
|
|
|
| |
authorization service rather than from the account.
This is to accomodate situations where the authorization service is being used by the hypergrid, where visitors have no user account.
See http://opensimulator.org/mantis/view.php?id=5517, this code is somewhat adapted/cleaned up from Michelle's patch
I'm a little ambivalent about this since visitors could put anything in firstname/lastname so it's not much of an auth measure.
It's up to the auth service to decide which data it actually uses.
Possibly we should be passing through other info such as agent circuit ip
|
| |
|
|
|
|
|
|
|
|
|
|
| |
scripts non-modifiable (but still copyable, etc).
Users should not be given the impression that they can modify these items.
This still does not solve the issue where library items cannot be dragged into prims or user inventory any time after they are initially seen.
Curiously, manually copying and pasting still appears to work.
On the surface, this seems to have something to do with library item caching on the client, since deleting the cache allows drag to work again once
Not sure what the exact problem is.
|
|
|
|
|
|
|
| |
they use the defaults instead.
Some items had completely wrong permissions - this is easier than correcting them all.
The ability to set permissions in xml is retained since there are use cases for this (e.g. to create no-mod library scripts)
|
|
|
|
|
|
|
|
| |
item xml files - always use PermissionMask.All instead (which was the existing default)."
There actually are uses for this. I will correct the perms instead since some entries appear to be wrong.
This reverts commit 667b54f5a2a04fa5a2859397868d270eab3913f1.
|
|
|
|
|
|
| |
files - always use PermissionMask.All instead (which was the existing default).
Library items always need the same permissions, so it doesn't make sense to load them from the xml files. This just opens the door to permissions mistakes.
|
|
|
|
|
| |
Fixed this by inspecting Shape.SculptEntry at various places instead of Shape.SculptType. Sculpties actually have a SculptType of Cylinder - only true mesh is SculptType.Mesh
This addresses http://opensimulator.org/mantis/view.php?id=5595
|