how to covert string array to char**

Question

I want to send the values manually here

void processArgs(int argc, char** argv);

if I sending like this

char* cwd[] = {"./comDaemon", "--loggg=pluginFramework:debug"};

parser->processArgs(2, cwd);

compiler showing warning as

warning: ISO C++ forbids converting a string constant to ‘char*’ [-Wwrite-strings]

 char* cwd[] = {"./comDaemon", "--loggg=pluginFramework:debug"};

You want const char* cwd[]

Borgleader
– Borgleader

2019-01-30 01:20:12 +00:00
Commented Jan 30, 2019 at 1:20 — Borgleader
– Borgleader, Commented Jan 30, 2019 at 1:20
And then you want const char **argc) in processArgs().

user207421
– user207421

2019-01-30 01:36:55 +00:00
Commented Jan 30, 2019 at 1:36 — user207421
– user207421, Commented Jan 30, 2019 at 1:36

Chris Dodd · Accepted Answer · 2019-01-30 02:08:07Z

6

Others have noted that the problem is you're trying to pass string literals (which are const) to a function that takes a non-const char ** argument. If what you want is to create non-const strings that you can pass to your non-const arg function, you need explicit char arrays (which you can initialize with string literals):

char arg0[] = "./comDaemon";
char arg1[] = "--loggg=pluginFramework:debug";
char *cwd[] = { arg0, arg1 };

you could even do this all on one line:

char arg0[] = "./comDaemon", arg1[] = "--loggg=pluginFramework:debug", *cwd[] = { arg0, arg1 };

answered Jan 30, 2019 at 2:08

Chris Dodd

127k14 gold badges150 silver badges243 bronze badges

Sign up to request clarification or add additional context in comments.

1 Comment

Kunal Puri Over a year ago

That's really nice answer.

Govind Parmar · Accepted Answer · 2019-01-30 02:07:52Z

4

If the function you're passing cwd to expects char ** argument, instead of const char **, here is one way:

    char *cwd[] = { const_cast<char *>("value1"), const_cast<char *>("value2") };

edited Jan 30, 2019 at 2:07

answered Jan 30, 2019 at 1:21

Govind Parmar

21.7k7 gold badges57 silver badges91 bronze badges

12 Comments

Kunal Puri Over a year ago

Please consider user207421's comment. Otherwise, option 1 will give compilation error.

Govind Parmar Over a year ago

@KunalPuri Edited my post, see my new disclaimer near option 2

Kunal Puri Over a year ago

I am more concerned about option 1. Because that is much better than option 2. Also, OP is supposed to change the argument type for option 1 not for option 2. I think that disclaimer should be part of option 1.

Govind Parmar Over a year ago

@KunalPuri Not sure what you're saying

Kunal Puri Over a year ago

void processArgs(int argc, char** argv); This is the declaration of function used by OP. So, What I want to say is that for option 1, you have to change it to void processArgs(int argc, const char** argv);. For option 2, I think void processArgs(int argc, char** argv); will work just fine as far as compilation is concerned.

|

Collectives™ on Stack Overflow

how to covert string array to char**

2 Answers 2

1 Comment

12 Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

2 Answers 2

1 Comment

12 Comments

Your Answer

Sign up or log in

Post as a guest

Related