Thread: what is not good in this function?

Code:

void set(int newprice=-1,char newname[]={"NONAME"},int newqty=-1)
        {
            if(price!=-1)
            price=newprice;
            if(strcmp(newname,"NONAME")!=0)
            strcpy(name,newname);
            if(qty!=-1)
            qty=newqty;
        }

in the above code .... what is not a good habit or illegal
i get a warning like this......

[Warning] extended initializer lists only available with -std=c++11 or -std=gnu++11 [enabled by default]

Code:

{"NONAME"}

is an array of strings containing a single string, AFAIK (unless using C++11 initializer lists).
It should just be "NONAME".

What is name, a global variable?
And why are you using C strings?
Unless there is some restriction, you'll find std::string s easier to use. (for example, you can assign instead of strcpy).

this is a class member function and the undeclared variables are declared in the class(pvt)