Small doubt in B2DPlugin

Previous Topic Next Topic
 
classic Classic list List threaded Threaded
1 message Options
Reply | Threaded
Open this post in threaded view
|

Small doubt in B2DPlugin

Nicolas Cellier
 
Hi,
in stepToFirstWideBezierIn: bezier at: yValue
I see this;

    xDir := ((self bezierUpdateDataOf: bezier) at: GBUpdateDX).
    xDir = 0 ifTrue:[((self bezierUpdateDataOf: bezier) at: GBUpdateDDX)].
    xDir >= 0 ifTrue:[xDir := 1] ifFalse:[xDir := -1].

I wonder if the intention of 2nd line is not

    xDir = 0 ifTrue:[xDir := ((self bezierUpdateDataOf: bezier) at: GBUpdateDDX)].

because otherwise the statement has no effect
(it would warn -Wunused-value)