| Commit message (Collapse) | Author | Age | Files | Lines |
|
|
|
| |
this is handled by the necessary ParentPart check
|
|
|
|
|
|
|
| |
(saving extra null checks, etc.)
However, it looks like we should retain SP.ParentID since it's much easier to use that in places where another thread could change ParentPart to null.
Otherwise one has to clumsily put ParentPart in a reference, etc. to avoid a race.
|
|
|
|
| |
(part != null), rather than having unnecessary multiple checks
|
|
|
|
|
|
| |
ParentPart is still not null if the ParentID != 0
Another thread could come in and stand the avatar between those two instructions.
|
|
|
|
| |
the check for significant is carried out. Prevents a deadlock condition.
|
| |
|
|
|
|
| |
parameters. They were only used by the get/set and make code harder to refactor.
|
|
|
|
|
|
|
|
|
|
| |
banlines or freezing on the banline.
This involves
1) On forcible teleport, call m_scene.RequestTeleportLocation() rather than ScenePresence.Teleport() - only EntityTransferModule now should call SP.Teleport()
2) When avatar is being forcibly moved due to banlines, use a 'stop movement' tolerance of 0.2 to requested position rather than 1
This prevents the avatar sometimes being stuck to banlines until they teleport somewhere else.
This aims to fix some problems in http://opensimulator.org/mantis/view.php?id=5822
|
|
|
|
|
|
| |
potential bad update that places an object at the opposite side of the
origin sim for a moment before actually crossing it. Especially important in
grids like OSG where lag between sims is high.
|
| |
|
|
|
|
| |
concurrent uses of the same TCP connection, and even of the connections to the same server. So let's stop doing it. This patch makes movement much smoother when there are lots of neighbours.
|
|
|
|
| |
complexity of code analysis
|
| |
|
|
|
|
|
|
| |
adjustment commit.
Left in the method doc.
|
|
|
|
|
| |
monocov was a code coverage attempt 3 years ago which no longer works.
other removed targets have been commented out or unused for a very long time
|
|
|
|
| |
avies always land in 0,0
|
|
|
|
|
|
| |
Add configuration option - DEBUG to enable debugging methods. This is temporary for helping users testing teleport routing be able to report back the data with the test cases. We can remove when finished with this, or leave it if it proves to be useful.
Users: set DEBUG = true in OpenSim.ini to get more information from teleport routing. The default is false. It presently prints the TeleportFlags value.
|
|
|
|
| |
Switch to our TeleportFlags enum instead of LibOMV because we need to define a type for HG Logins. Also moved some debugging in ScenePresence into a function to make it simpler to enable/disable.
|
|
|
|
| |
Fix test to checking against bitfield instead of int
|
|
|
|
| |
Incoming HG owner/estate manager, etc. is routed according to the rules defined for teleports within the local grid. Left some commented debugging code inside so we can test other cases. Will remove when tings are settled in.
|
|
|
|
| |
Add some temporary debugging to the teleport routing to get a better view of what happens when HG jumps are made.
|
|
|
|
| |
Needed to breakout the ViaHGLogin check to it's own section. For some reason it would not factor in when combined with the other teleport flag types.
|
| |
|
| |
|
| |
|
|
|
|
| |
Route non-owner avatars according to land settings
|
|
|
|
|
|
| |
other than falling again.
Addresses http://opensimulator.org/mantis/view.php?id=5839
|
|
|
|
|
|
| |
problematic with bot testing.
Please uncomment if still needed.
|
| |
|
| |
|
| |
|
|
|
|
| |
CheckForBorderCrossing the right way.
|
|\ |
|
| |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
baked texture caching when crossing region boundaries.
Needs further investigation.
Revert "Stop sending the viewer its own AvatarAppearance packet."
This reverts commit 92039f295d7fe66bf1a09b29483f9057e395839e.
|
|/ |
|
|\ |
|
| |
| |
| |
| |
| | |
The viewer warns in the log if it receives this.
Stopping this doesn't appear to have adverse effects on viewer 1 or viewer 3 - the viewer gets its own appearance from body parts/clothes and self-baked textures.
|
| |
| |
| |
| |
| |
| |
| |
| | |
AvatarFactoryModule from AppearanceInfoModule so that it can be used in debug (inactive).
Further filters "debug packet <level>" to exclused [Request]ObjectPropertiesFamily if level is below 25.
Adjust some method doc
Minor changes to some logging messages.
|
| |
| |
| |
| | |
These were entirely unused.
|
|/
|
|
| |
a little bit.
|
|
|
|
|
|
| |
BEGIN] to [SCENE] because that's where the message happens.
Also changed the instantiation of a vector object to be done only once instead of every time we receive a position update.
|
|
|
|
|
|
| |
when we've already passed size information to the avatar at PhysicsScene.AddAvatar()
Eliminate some copypasta for height setting in OdeCharacter
|
|
|
|
|
|
| |
Stop hiding RemoveAvatar failure, add log messages when characters are removed through defects or re-added unexpectedly.
Add commented out log lines for future use.
Use automatic property for PhysicsActor for better code readability and simplicity
|
|
|
|
| |
IsColliding
|
|
|
|
| |
in favour of IClientAPI.ISceneAgent.IsChildAgent instead.
|
| |
|
|
|
|
|
| |
The only caller is the LLUDP stack and this has to validate the UDP circuit itself, so we know that it exists.
This allows us to eliminate another null check elsewhere and simplifies the method contract
|
| |
|
|
|
|
|
|
| |
asynchronously to reduce hold up in the IN UDP packet processing loop.
This is already being done for the initial object data send.
|
|
|
|
|
|
|
|
|
| |
it's live before sending other data.
This means that avatar/appearance data of other avatars and scene objects for a client will be sent after the ack rather than possibly before.
This may stop some avatars appearing grey on login.
This introduces a new OpenSim.Framework.ISceneAgent to accompany the existing OpenSim.Framework.ISceneObject and ISceneEntity
This allows IClientAPI to handle this as it can't reference OpenSim.Region.Framework.Interfaces
|