Jquery Function definition in a Carousel Script

Question

I have this script for a Carousel Images with LOOP

$(document).ready(function() {

//rotation speed and timer
var speed = 5000;
var run = setInterval(rotate(), speed);   

//grab the width and calculate left value
var item_width = $('#slides li').outerWidth(); 
var left_value = item_width * (-1); 

//move the last item before first item, just in case user click prev button
$('#slides li:first').before($('#slides li:last'));

//set the default item to the correct position 
$('#slides ul').css({'left' : left_value});

//if user clicked on next button
function rotate() {
    //get the right position
        var left_indent = parseInt($('#slides ul').css('left')) - item_width;

        //slide the item
        $('#slides ul').animate({'left' : left_indent}, 3000, function () {

            //move the first item and put it as last item
            $('#slides li:last').after($('#slides li:first'));                  

            //set the default item to correct position
            $('#slides ul').css({'left' : left_value});

        });

        //cancel the link behavior
        return false;
}       

});

But I receive this javascript error in firebug:

useless setInterval call (missing quotes around argument?) [Interrompi per questo errore] var run = setInterval(rotate(), speed);

I thing it's an error of rotate function definition!

Answer 1

This means the you should write:

 var run = setInterval(rotate, speed);   

instead of

 var run = setInterval(rotate(), speed);   

because you need to pass the reference of a function to setInterval, what you are passing is the return value of the function rotate();