Todo List

Member Dry::AnimatedModel::OnWorldBoundingBoxUpdate () override

If it's a skinned attachment that does not cover the whole body, it will have unnecessarily large bounds

Member Dry::AnimatedModel::UpdateBoneBoundingBox ()

The sphere radius should be multiplied with bone scale

Member Dry::Animation::CreateTrack (const String &name)

When tracks / keyframes are created dynamically, memory use is not updated

Member Dry::Camera::GetFrustumSize (Vector3 &near, Vector3 &far) const

Necessary? Explain this

Member Dry::CollisionGeometryDataCache

Remove duplicate declaration

Member Dry::Connection::ProcessExistingNode (Node *node, NodeReplicationState &nodeState)

Searching for the component is a potential CPU hotspot. It should be cached

Member Dry::Connection::ProcessSceneUpdate (int msgID, MemoryBuffer &msg)

On mobile devices processing this message may potentially cause a crash if it attempts to load new GPU resources while the application is minimized

Member Dry::Constraint::OnMarkedDirty (Node *node) override

This does not catch the connected body node's scale changing

Member Dry::Context::~Context () override

Context should not need to know about subsystems

Member Dry::CrowdAgent::OnMarkedDirty (Node *node) override

Handle node transform being dirtied.

Member Dry::DecalSet::AddDecal (Drawable *target, const Vector3 &worldPosition, const Quaternion &worldRotation, float size, float aspectRatio, float depth, const Vector2 &topLeftUV, const Vector2 &bottomRightUV, float timeToLive=0.0f, float normalCutoff=0.1f, unsigned subGeometry=M_MAX_UNSIGNED)

target transform is not right if adding a decal to StaticModelGroup

Member Dry::DecalSet::OnWorldBoundingBoxUpdate () override

The sphere radius should be multiplied with bone scale

Member Dry::EventReceiverGroup::EndSendEvent ()

Could be optimized by erase-swap, but this keeps the receiver order

Member Dry::File::Read (void *dest, unsigned size) override

Handle errors

Member Dry::FileSystem::ScanDirInternal (Vector< String > &result, String path, const String &startPath, const String &filter, unsigned flags, bool recursive) const

Filename may be unnormalized Unicode on Mac OS X. Re-normalize as necessary

Member Dry::Graphics::Clear (ClearTargetFlags flags, const Color &color=Color(0.0f, 0.0f, 0.0f, 0.0f), float depth=1.0f, unsigned stencil=0)

Any user-set scissor test will be lost

Member Dry::HttpRequest::ThreadFunction () override

SSL mode will not actually work unless Civetweb's SSL mode is initialized with an external SSL DLL

Member Dry::Image::Resize (int width, int height)

Reducing image size does not sample all needed pixels

Member Dry::Input::AddScreenJoystick (XMLFile *layoutFile=nullptr, XMLFile *styleFile=nullptr)

After a real joystick has been plugged in 1073741824 times, the ranges will overlap

Axis emulation for screen joystick is not fully supported yet.

Member Dry::Input::ResetState ()

Check if resetting joystick state on input focus loss is even necessary

Member Dry::JSONValue::GetVariantMap () const

Ideally this should allow any strings, but for now the convention is that the keys need to be hexadecimal StringHashes

Member Dry::NavigationMesh::BuildTile (Vector< NavigationGeometryInfo > &geometryList, int x, int z)

Assignment of flags from navigation areas?

Member Dry::NavigationMesh::CollectGeometries (Vector< NavigationGeometryInfo > &geometryList, Node *node, HashSet< Node *> &processedNodes, bool recursive)

Evaluate whether should handle other types. Now StaticModel & TerrainPatch are supported, others skipped

Member Dry::ParticleEffect2D::Clone (const String &cloneName=String::EMPTY) const

Zero if source was created programmatically

Member Dry::ParticleEffect::Clone (const String &cloneName=String::EMPTY) const

Zero if source was created programmatically

Member Dry::Renderer::DrawDebugGeometry (bool depthTest)

Because debug geometry is per-scene, if two cameras show views of the same area, occlusion is not shown correctly

Member Dry::Renderer::GetScreenBuffer (int width, int height, unsigned format, int multiSample, bool autoResolve, bool cubemap, bool filtered, bool srgb, unsigned persistentKey=0)

Mipmaps disabled for now. Allow to request mipmapped buffer?

Member Dry::Renderer::GetShadowMap (Light *light, Camera *camera, unsigned viewWidth, unsigned viewHeight)

Allow to specify maximum shadow maps per resolution, as smaller shadow maps take less memory

Member Dry::Renderer::UpdateQueuedViewport (unsigned index)

May result in incorrect debug geometry culling if the same scene is drawn from multiple viewports

Member Dry::Scene::UpdateAsyncLoading ()

Works poorly in scenes where one root-level child node contains all content

Member Dry::Spline::BezierInterpolation (const Vector< Variant > &knots, float t) const

Do not allocate a new vector each time

Member Dry::Spline::GetPoint (float f) const

Do not allocate a new vector each time

Member Dry::Sprite::IsWithinScissor (const IntRect &currentScissor) override

Implement properly, for now just checks visibility flag

Member Dry::Terrain::GetHeight (const Vector3 &worldPosition) const

This assumes that the terrain scene node is upright

Member Dry::UIElement::SortChildren ()

Order is not stable when children have same priorities

Member Dry::ValueAnimationInfo::GetEventFrames (float beginTime, float endTime, PODVector< const VAnimEventFrame *> &eventFrames)

This can miss an event if the deltatime is exactly the animation's length

Member Dry::View::PrepareInstancingBuffer ()

If rendering the same view several times back-to-back, would not need to refill the buffer