closing GPSDClient

[JordanMcManus]

In the client.cpp source file for gpsd_client we see this:

void stop() {
      // gpsmm doesn't have a close method? OK ...
    }

The documentation for libgpsmm reads

libgpsmm is a mere wrapper over libgps. Method names are the same as
the analogue C functions. For a detailed description of the functions
please read libgps(3). open() must be called after class constructor
and before any other method (open() is not inside the constructor since
it may fail, however constructors have no return value). The analogue
of the C function gps_close() is in the destructor.

Do you think that simply delete the gps pointer can solve this or is too naive?

void stop() {
      delete gps;
    }

[danthony06]

I'd have to look a little more closely, but I think I'd prefer to make it a smart pointer so that it automatically goes out of scope and is destroyed when main class is destroyed.