Snippet Spinning Attachment Tool

[wraithseeker]

 method SetEndEffect takes boolean b returns nothing
        set .endeffect = b
    endmethod

method SetRange takes real range returns nothing
        set .range = range
    endmethod

Why do you keep them here?

 //Hook new instance to rotating instances
    call d.Start()

Couldn't you have just done your coding inside the Add function itself?

 

[kingkingyyk3]

Why do you keep them here?
Just some options make it more freedom.

Couldn't you have just done your coding inside the Add function itself?
To uniform the step.

 

[wraithseeker]

Why do you keep them here?
Just some options make it more freedom.

Couldn't you have just done your coding inside the Add function itself?
To uniform the step.

They are the same as setting .range = to range, no point if you are not doing anything else.

 

[Jesus4Lyf]

Wraithseeker, it's good programming practise to do such things since they inline anyway. Later, you can make it do things when such values are set without changing the interface.

Alternatively in JassHelper, you use method operators. But why tell him to change it?