Welcome to the Lounge

/* Enter your C++ code here (you can delete this line) */


    #include <stdio.h>
    #include <string.h>

    struct X {
        char *p;
        X() { p = new char[100]; strcpy( p, "hello world" ); }
        X( const X & x )
            { p = new char[ strlen(x.p) + 1 ]; strcpy( p, x.p ); }
        X &operator=( const X & x )
           { delete[] p;  p = new char[ strlen(x.p) + 1 ];
             strcpy( p, x.p ); return *this; }
       ~X() { delete[] p; }
       };

   void g( X &x1, X &x2 ) { x1 = x2; }

   int main()
       {
       X a;
       g( a, a );
       printf( "%s\n", a.p );
       return 0;
       }

FlexeLint for C/C++ (Unix) Vers. 9.00h, Copyright Gimpel Software 1985-2011
--- Module: diy.cpp (C++)
     1  /* Enter your C++ code here (you can delete this line) */
     2  
     3  
     4      #include <stdio.h>
     5      #include <string.h>
     6  
     7      struct X {
     8          char *p;
     9          X() { p = new char[100]; strcpy( p, "hello world" ); }
    10          X( const X & x )
    11              { p = new char[ strlen(x.p) + 1 ]; strcpy( p, x.p ); }
    12          X &operator=( const X & x )
    13             { delete[] p;  p = new char[ strlen(x.p) + 1 ];
    14               strcpy( p, x.p ); return *this; }
    15         ~X() { delete[] p; }
                                                     _
    14               strcpy( p, x.p ); return *this; }
diy.cpp  14  Warning 1529:  Symbol 'X::operator=(const X &)' not first checking for assignment to this
    16         };
    17  
                                             _
    18     void g( X &x1, X &x2 ) { x1 = x2; }
diy.cpp  18  Info 1764:  Reference parameter 'x2' (line 18) could be declared const ref
    19  
    20     int main()
    21         {
    22         X a;
    23         g( a, a );
    24         printf( "%s\n", a.p );
    25         return 0;
                       _
    26         }

diy.cpp  26  Info 783:  Line does not end with new-line

--- Global Wrap-up

Info 1714:  Member function 'X::X(const X &)' (line 10, file diy.cpp) not referenced

_beauw_ wrote:

Configurability can mitigate a flaw's negative impact, but it cannot eliminate it. Beyond the cost of writing the code for the configurability, there is the cost imposed on the end users, who must re-configure around unreasonable defaults.

Steve Mayfield wrote:

so size doesn't really matter on cell phones...