c stack overflow (moving ComboPointPlayerFrame)

[Cie]

Hi,

im moving the Rogue Combo Point Bar like this:

local function yOffset(Frame,y)
    local P = { Frame:GetPoint() }
    Frame:SetPoint(P[1], P[2], P[3], P[4],y)
end
 
hooksecurefunc(ComboPointPlayerFrame,"SetPoint",function(ComboPointPlayerFrame) yOffset(ComboPointPlayerFrame,91) end)

however i get the following error:

Message: C stack overflow
Time: 04/13/18 15:51:24
Count: 1
Stack: C stack overflow
[C]: ?
[C]: in function `SetPoint'
.lua:84: in function <...lua:1>
.lua:87: in function <...lua:6>
[C]: in function `SetPoint'
.lua:84: in function <...lua:1>
.lua:87: in function <...lua:6>
[C]: in function `SetPoint'
.lua:84: in function <...lua:1>
.lua:87: in function <...lua:6>
[C]: in function `SetPoint'
.lua:84: in function <...lua:1>
...
[C]: in function `SetPoint'
.lua:84: in function <...lua:1>
.lua:87: in function <...lua:6>
[C]: in function `SetPoint'
.lua:84: in function <...lua:1>
.lua:87: in function <...lua:6>
[C]: in function `SetPoint'
Interface\FrameXML\ComboFramePlayer.lua:35: in function `Setup'
Interface\FrameXML\ClassPowerBar.lua:56: in function `OnEvent'
Interface\FrameXML\ComboFramePlayer.lua:27: in function <Interface\FrameXML\ComboFramePlayer.lua:21>

its working fine though, but i cant fix it, whats the exact problem here?

[Kanegasi]

It’s recursive. You attached a function that calls SetPoint to SetPoint itself. Find another function to hook to.

[myrroddin]

Yep. You are using SetPoint on each pixel of the y axis (same would be true if you also did it on the x axis). Try setting the frame to movable and moving or sizing. Once the frame has stopped moving, then GetPoint for the new position and save that to your SVs.

[Kakjens]

Shouldn't this work:

local needs_to_setpoint = true
local function yOffset(Frame,y)
    if needs_to_setpoint
        needs_to_setpoint = false
        local P = { Frame:GetPoint() }
        Frame:SetPoint(P[1], P[2], P[3], P[4],y)
    end
    needs_to_setpoint = true
end
 
hooksecurefunc(ComboPointPlayerFrame,"SetPoint",function(ComboPointPlayerFrame) yOffset(ComboPointPlayerFrame,91) end)

?

[Cie]

i just anchored it to the playerframe , didnt worked the first time ive tried it , but it does now. well