So I've been slowly converting my SWR to classes and adding the appropriate constructors and destructors. Well, I got to the shuttle system and began converting it over.

The shuttle system consists of two classes - one being the shuttle_data class and the other being the stop_data class. shuttle_data has a stop_data *current pointer.
So the problem I'm having, and I'm just not sure how to tackle it is the best way to convert this to use the std::list.

                                        shuttle->current = shuttle->current->next;

Also, just to double check with those who are smarter than I on this. Is this the proper way to say if the current stop is the last element in the std::list move to the front?

                                /* Move to next spot */
                                if (shuttle->current == shuttle->stops.back( )) 
                                {
                                        shuttle->current = shuttle->stops.front( );
                                        shuttle->current_number = 1;
                                }

You could consider turning "current" into an iterator instead of a pointer:

struct SHUTTLE_DATA
{
  std::list< STOP_DATA* >::iterator current; // or whatever your "stop" class is called
};

Initialize to first stop:

shuttle->current = shuttle->stops.begin();

Next stop:

shuttle->current++;

Find out if you're at the end of the list:

if( shuttle->current == shuttle->stops.end() )

In the last example you're actually one element past the last one, rather than at it, as per normal iterator behaviour.

As for your last question could you rewrite it? I don't understand what you're asking. Perhaps a word is missing?

I suddenly understood the question. Just change "front" and "back" with "begin" and "end" in your example.

I actually really like the first option better. It looks much cleaner.

Only thing I'm not sure about is checking if that iterator is null or not.

Basically, if I change it to be an iterator how do you make this work:

                        if (shuttle->current == NULL)

Edit - I forgot to make it

 if(*shuttle->current == NULL)

It worked after that.