Re: Newbies don't learn C++

Leigh Johnston wrote:


If you want to define this term like that, so be it. But then, who cares
if a program leaks? After all, the memory is released on shutdown[0]. Yes,
these "leaks" can be detected automatically and there are easy ways to
avoid them, basically discipline in low-level resource management will do,
but they don't say anything about the runtime behaviour of the program.

Instead, the IMHO important property of a resource leak is that resources
are allocated and not used any more but also not released. In that sense,
I'd say you need at least two stages in the program, one that allocates an
object and a second that neither releases nor uses it. Further, I would
say that some steps are then usually repeated in a loop, causing an
increasing memory consumption with each repetition.

Example:

   int main() {
     vector<char> f = read_cfg_file();
     settings s = parse_cfg(f);
     // (1)
     run_program(s);
   }

At point (1), "f" becomes unused but remains allocated and thus
constitutes a memory leak according to my definition. It is paired with a
proper deallocation on program termination, so it isn't one in your
definition. I find mine more useful when talking about a program.

Even more so when considering a server process:

   int main() {
     list<connection> clients;
     try {
       while(true) {
         connection c = accept_connection();
         if(c)
           clients.push_back(c);
         service_connections(clients);
       }
     } catch(...) {
     }
   }

This program will not terminate on its own but continue to allocate new
connections until it runs out of memory because it fails to clean up or
otherwise limit the number of connections. It will then release existing
connections, so allocations and deallocations are paired.