Why am I getting an "int-conversion" warning when I assign a value to a string array member in a struct initializer?

Question

So I have this code that I've boiled down so its only purpose is to show the problem I'm having:

typedef struct _TEST {char strarr[3][20];} TEST;

int main(int argc, char *argv[]) {
    char strarr[3][20] = {"some", "test", "strings"};
    TEST test = {.strarr=strarr};

    return 0;
}

When I try to compile it I get this warning:

$ gcc test.c -o test
test.c: In function ‘main’:
test.c:6:26: warning: initialization makes integer from pointer without a
cast [-Wint-conversion]
     TEST test = {.strarr=strarr};
                          ^
test.c:6:26: note: (near initialization for ‘test.strarr[0][0]’)

I've tried googling the error message, but the results I get are all people who are using pointers incorrectly. I don't think I'm using pointers incorrectly. I don't think I'm really even using pointers in this code, but I'm fairly new to C and there's still a lot of things that I don't really understand (how they work or why they're so unintuitive). Especially with strings.

I've also tried googling various phrases along the lines of "assigning a string array in a struct initializer," but none of the results I've found are relevant to my problem.

I did find something maybe relevant when I searched "near initialization for." While it's not specifically relevant to my problem, I did find some useful info: They got the warning because they were initializing the variable incorrectly. But I don't know what I'm doing incorrectly.

Answer 1

Because strarr is an array you can't write .strarr = strarr. Because an array is not assignable. So you could do:

test_s test = {
    .strarr = {{strarr[0][0],
                strarr[0][1] /* to strarr[0][x], x is the end of the string '\0' */},
               {0 /* same for strarr[1] */},
               {0 /* strarr[2] */}}};

but it's really painful. So you can use memcpy() instead:

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

typedef struct
{
    char strarr[3][20];
} test_s;

int main(void)
{
    char strarr[3][20] = {"some", "test", "strings"};

    test_s test;
    memcpy(test.strarr, strarr, sizeof test.strarr);

    for (size_t i = 0; i < sizeof strarr / sizeof *strarr; i++) {
        printf("%s\n", test.strarr[i]);
    }
}

---

Or use a pointer of pointer. So test will be valid only if strarr is valid too:

#include <stddef.h>
#include <stdio.h>

typedef struct
{
    char **strarr;
} test_s;

int main(void)
{
    char *strarr[] = {"some", "test", "strings"};

    test_s test = {.strarr = strarr};

    for (size_t i = 0; i < sizeof strarr / sizeof *strarr; i++) {
        printf("%s\n", test.strarr[i]);
    }
}

---

Or again use a pointer of pointer but allocate the memory so test can live without strarr:

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

typedef struct
{
    char **strarr;
} test_s;

int main(void)
{
    char *strarr[] = {"some", "test", "strings"};
    size_t const size_strarr = sizeof strarr / sizeof *strarr;

    test_s test = {.strarr = malloc(sizeof *test.strarr * size_strarr)};
    if (test.strarr == NULL) {
        return 1;
    }

    for (size_t i = 0; i < size_strarr; i++) {
        test.strarr[i] = malloc(sizeof *test.strarr[i] * (strlen(strarr[i]) + 1));
        if (test.strarr[i] == NULL) {
            return 1;
        }
        strcpy(test.strarr[i], strarr[i]);

        printf("%s\n", test.strarr[i]);
    }
}

---

More info about array.