CommonState's walk and run states do not loop properly

CommonState's AddWalkStates and AddRunStates have an issue where in the "walk" and "run" states respectively don't loop properly, eventually causing a desync with the intended timeevents.

table.insert(states, State{
        name = "walk",
        tags = { "moving", "canrotate" },

        onenter = function(inst)
			if fns ~= nil and fns.walkonenter ~= nil then
				fns.walkonenter(inst)
			end
            inst.components.locomotor:WalkForward()
            inst.AnimState:PlayAnimation(get_loco_anim(inst, anims ~= nil and anims.walk or nil, "walk_loop"), true)
            inst.sg:SetTimeout(inst.AnimState:GetCurrentAnimationLength())
        end,

        timeline = timelines ~= nil and timelines.walktimeline or nil,

		onupdate = fns ~= nil and fns.walkonupdate or nil,

		onexit = fns ~= nil and fns.walkonexit or nil,

        ontimeout = walkontimeout,
    })

The main culprit seems to be the method of looping the states. Which is the Timeout.

inst.sg:SetTimeout(inst.AnimState:GetCurrentAnimationLength())

The state attempts to convert the animation length into an accurate timeout to properly loop the state, however due to either an error on GetCurrentAnimationLength or executing Timeout, there will be a very small delay causing the state and animations to desync. It's not noticeable on most things in game since the delay is so short it takes awhile for the desync to be noticeable, but it is an issue regardless.

--------------

While I don't know the technical reasons for choosing this method or if fixing it will cause issues elsewhere, I know of other methods of state-looping where the timings stay synced. A common one I use for custom mobs and playable pets is setting a "animqueueover"/"animover" event to replay the state.

events =
        {
            EventHandler("animover", function(inst) inst.sg:GoToState("idle") end ),    
        },