Skip to main content
Respuesta rehecha.
Origen Enlace

Lo que te está ocurriendo es que estás llamando a la variable actions desde el scope y desde la definición de la función.

Cambios estéticos para facilitar la explicación

Es decirPermíteme poder hacer unos cuantos cambios estéticos que me permitan trabajar con tus datos. No recomiendo usar struct action siempre, has sobrescritosino hacer un actionstypedef de lo que era originalmentepermita usar esta estructura como un único tipo. La llamaremos, por esofacilidad, cuando has intentado acceder a la otra definición, no has podidoaction. También pondré los namespaces, porque ya no sabría a cuál te estás refiriendoque es una mala práctica de programación demasiado extendida usar using namespace std; en alguna cabecera.

Y por último alinearé los tipos de datos por un lado y las variables por otro para placer visual. Todo esto mejorará notablemente tu forma de trabajar leyendo código, que es realmente el trabajo que vas a realizar más tiempo.

Tu tipo de dato de origen pasará a ser:

another_struct_wprtypedef *my_function_create(action_wprstruct *actions,s_action
{
    std::string                   name       ;
    int                           type       ;
    std::vector < unsigned char *nActions)> data       ;
{    int                           value      ;
    //unsigned Instanciamoschar el vector de acciones para que lo rellene my_function()        source     ;
    vector<structunsigned action>char actions; // <- fíjate aquí, actions lo **REDEFINES**         destination;
} action;

El poner los actions; alineados ayudará muchísimo de cara a un futuro cuando tengas que editar todos los datos secuencialmente, ya que con un editor que permita copiar en vertical, tendrás gran parte del trabajo hecho.

Posteriormente veo que haces un typedef, pero es que podías haberlo hecho desde antes (si es código heredado, haz baby steps: introduce el typedef, compila, y comprueba quién llama al struct con los errores de esa líneacompilación, era una variablehaces los cambios uno por uno quitando struct y listo)

typedef struct s_action_wpr
{
    char *          name       ;
    int             type       ;
    key_wpr         auth_key   ;
    unsigned char * data       ;
    int             value      ;
    unsigned char   source     ;
    unsigned char   destination;
} action_wpr;

Por amor a los debugguers, no pongas el mismo nombre al typedef que al struct.

Habrás visto también que al colocar el struct de esta manera, queda el tipo de datos unívocamente definido. Esto nos ayudará en la siguiente fase, que consiste en aprender hacer TDD en C para la migración de los datos de origen a los datos de destino.

TDD en C para la migración de datos

Lo primero que tienes que testar es la migración en sí. No puedes picar un código sin pruebas, si lo haces, corres el riesgo de que acabes desarrollando Skynet y tu código tome conciencia de sí mismo.

Así que, para la migración, deberás crear una función que te permita introducir un action y devuelva un action_wpr *. De ahí te darás cuenta que tendrás que hacer una función para migrar cada parte que no se migre de forma directa.

Migración de action en action_wpr

Con un editor que permite la edición en vertical, despuéspuedes ver que los campos no son iguales, te sobra en action_wpr un campo (que nos has omitido, imagino, a propósito)

typedef struct s_action                        | typedef struct s_action_wpr
{                                              | {
    std::string                   name       ; |     char *          name       ;
    int                           type       ; |     int             type       ;
    std::vector < unsigned char > data       ; |     key_wpr         auth_key   ;
    int                           value      ; |     unsigned char * data       ;
    unsigned char                 source     ; |     int             value      ;
    unsigned char                 destination; |     unsigned char   source     ;
} action;                                      |     unsigned char   destination;
                                                 } action_wpr;

Así que voy a quitarlo a propósito, porque no hay forma de esahacer el traspaso fácilmente:

typedef struct s_action                        | typedef struct s_action_wpr
{                                              | {
    std::string                   name       ; |     char *          name       ;
    int                           type       ; |     int             type       ;
    std::vector < unsigned char > data       ; |     unsigned char * data       ;
    int                           value      ; |     int             value      ;
    unsigned char                 source     ; |     unsigned char   source     ;
    unsigned char                 destination; |     unsigned char   destination;
} action;                                      | } action_wpr;

Mucho mejor.

¿Qué vemos que podemos traspasar sin hacer cambios? Pues los unsigned char, los int.

Nota: ¿Has visto que haya puesto una sola línea seráde código? No, a esto se le llama pensar antes de actuar, te acostumbrarás cuando te pase mil veces, así que asume siempre que lo haces mal. Cuando seas capaz de cambiar tu forma de hacer las cosas habrás pasado al siguiente nivel. No es un ataque, es un consejo.

¿Qué datos necesitas para terminar la migración que aún no hemos hecho? vector<structstd::string y std::vector action>< unsigned char > hastaen, respectivamente, char * y unsigned char *

Migrar std::string en char *

¿Conoces las propiedades de std::string? ¿Y las de char *? Viendo tu código, asumo que, efectivamente, no.

  • No son la misma "cosa"
  • std::string no es una forma de guardar caracteres uno detrás de otro, guardas cadenas de caracteres. Está indicado para guardar trozos de texto, y puedes realizarle operaciones de manipuación de cadenas de caracteres de forma directa con sus operadores.
  • char * es un puntero a un "vector" de una dimensión de elementos de char, igual que cuando estudias en matemáticas los vectores, van unos detrás de otros. Alguno de ellos puede ser \0, 0xFF o incluso 0xDE 0xAD 0xBE 0xEF, que puede quedar como 0xDEADBEEF, una referencia bastante habiutal en variables hexadecimales por su significado en inglés (vaca muerta).

Entendiendo que no son la misma cosa, ahora tienes que hacer una función a la que le pases el finaltipo de dato de un lado y devuelva el del otro con el contenido de entrada.

Ejemplos de tests que vas a tener que hacer:

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

char * funcion_migracion (std::string);

void test_harness_01 (void)
{
    /** Datos con los que voy a probar **/
    char * dato_char;
    char comparador_base[] = "test de prueba 01";

    /** Ejecución de la prueba **/
    dato_char = funcion_migracion ( std::string ( comparador_base ) ); // Al hacer esto, haces la conversión explícita de `char[]` a `std::string`, aunque no haría falta porque `funcion_migracion` sólo acepta `std::string` y lo convertiría internamente por ti
    
    /** Comprobar resultados esperados **/
    if ( dato_char == NULL )
    {
        printf("dato_char no puede ser NULL\r\n");
        return;
    }
    if ( strcmp(dato_char, comparador_base) != 0 )
    {
        printf("no se obtiene el restulado esperado\r\n");
    }

    /** Limpieza de los resultados obtenidos **/
    if ( dato_char != NULL )
    {
        free ( dato_char );
    }
}

Ahora pones diferentes resultados copiando y pegando la función scopetest_harness_01 y modificando comparador_base para ir viendo que te sale lo que tiene que salir. Mientras no salgan las pruebas, quiere decir que tu código no está bien hecho.

¿Qué obtienes con esto? Aprendes a hacer pruebas, sabes exactamente qué quieres y siempre que ejecutes los tests, volverás a comprobar que no has roto nada en el camino. Como habrás imaginado, estas pruebas no se ejecutan en tu programa real, se ejecutan en un lugar separado de tu programa, al que vamos a llamar "espacio de desarrollo". Te ayuda a que, cuando tengas que hacer o romper algo, tengas presente qué rompes y qué no, y mejora tu forma de realizar código a largo plazo. Dado que parece que estás aprendiendo a programar, tener esta forma de realizar programas es una herramienta de lo más útil para tu día a día.

Sabiendo esto, ya puedes, como ejercicio, saber cómo migrar un std::vector de unsigned chars en un unsigned char *.

Migrar std::vector < unsigned char > en unsigned char *

¿Por qué no es tan fácil la migración y te has quedado atascado? Efectivamente, porque te has olvidado de un detallito sin importancia que los tests nos enseñarán llegado el momento.

Voy a plantearte el siguiente entorno de tests para que lo veas por ti mismo, y veas qué ocurre.

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

char * funcion_migracion_vector (std::vector < unsigned char >);

void test_harness_vector_01 (void)
{
    /** Datos con los que voy a probar **/
    char * dato_char;
    std::vector < unsigned char > comparador_base;

    comparador_base.push_back('\0');

    /** Ejecución de la prueba **/
    dato_char = funcion_migracion_vector ( comparador_base );
    
    /*
     * ¿Qué demonios tiene que ocurrir aquí?
     */
}

Acabas de descubrir que, por construcción, ¡un std::vector de unsigned chars permite insertar \0 en cualquier parte! ¿Y cómo podemos procesar los datos que hay detrás de \0? ¿Uso magia vudú?

Necesitarás, inevitablemente, un contador de todos los elementos de ese vector.

Y así es como se averiguan bugs de base que tienes en tu código y que desconocías. Igual que desconozco si eres profesional o no, sólo así salen estos errores, cuando haces pruebas (si las automatizas, directamente triunfas).

Una vez sabes cómo migrar los datos "difíciles" de migrar, hay que hacer el test de integración, que es migrar de una estructura a otra.

Migrar desde action hasta action_wpr

En TDD se usa, llegado a este punto, una cosa que llamaremos mocks, que permiten engañar al sistema para no hacer todas las llamadas internas (por ejemplo, engañar al sistema para que no llame a una base de datos, que es una operación lenta y costosa), como esto me huelo que va para wiki de comunidad, te lo pondré fácil y seguiremos con las pruebas sencillas, ahora tus estructuras pasarán, con el bug que mencioné antes, a esto:

typedef struct s_action                        | typedef struct s_action_wpr
{                                              | {
    std::string                   name       ; |     char *          name       ;
    int                           type       ; |     int             type       ;
    std::vector < unsigned char > data       ; |     unsigned char * data       ;
                                               |     unsigned int    n_data_elms;
    int                           value      ; |     int             value      ;
    unsigned char                 source     ; |     unsigned char   source     ;
    unsigned char                 destination; |     unsigned char   destination;
} action;                                      | } action_wpr;

Aquí tendrías el código con el que tendrías que hacer la prueba para la migración entre estructuras:

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

typedef struct s_action                       
{                                             
    std::string                   name       ;
    int                           type       ;
    std::vector < unsigned char > data       ;
                                              
    int                           value      ;
    unsigned char                 source     ;
    unsigned char                 destination;
} action;                                     

typedef struct s_action_wpr
{
    char *          name       ;
    int             type       ;
    unsigned char * data       ;
    unsigned int    n_data_elms;
    int             value      ;
    unsigned char   source     ;
    unsigned char   destination;
} action_wpr;



action_wpr * funcion_migracion_structs (action);

void test_harness_migration_structs_01 (void)
{
    /** Datos con los que voy a probar **/
    action_wpr * dato_char;
    action base;

    base.name("namestring prueba 01");
    base.type = 0xDEADBEEF;
    base.data.push_back('h');
    base.data.push_back('\0');
    base.data.push_back('l');
    base.data.push_back('a');
    base.data.push_back(' ');
    base.data.push_back('m');
    base.data.push_back('u');
    base.data.push_back('n');
    base.data.push_back('d');
    base.data.push_back('o');
    base.value = -35;
    base.source = 0x33;
    base.destination = 0xCC;

    /** Ejecución de la prueba **/
    dato_char = funcion_migracion_vector ( base );
    
    /*
     * ¿Qué demonios tiene que ocurrir aquí?
     */
}

Como has visto, he dejado sin completar. ¿Qué faltaría?

Pues, con esto, puedes hacer la migración entre elemento y elemento. Tu siguiente y último paso, ahora que ya sabes cómo se testa en escalera, directamente será testar cómo se migra un "vector" de elementos de un tipo a otro.

Qué has aprendido

Has aprendido con esto que uno no se pone a programar a lo loco. Si tienes que migrar una estructura, haz pruebas para que dicha migración (migración como sinónimo de traspaso, de traductor, de conversor) esté siempre controlada.

Hay montones de frameworks que funcionan estupendamente en C y C++ para este fin. Y dado que los compiladores de C++ admiten código en C (usando extern 'C'), no tendrías problemas de ningún tipo en usar un framework de testing en C++, por ejemplo.

Mi respuesta no te hace código por ti. Ni pretendo que lo haga. Lo que hago es que veas qué problemática tienes a la hora de realizar código y una posible solución a corto y medio plazo que te ayudará.

Lo que te está ocurriendo es que estás llamando a la variable actions desde el scope y desde la definición de la función.

Es decir, has sobrescrito actions de lo que era originalmente, por eso, cuando has intentado acceder a la otra definición, no has podido, porque ya no sabría a cuál te estás refiriendo.

another_struct_wpr *my_function_create(action_wpr *actions, unsigned char *nActions)
{
    // Instanciamos el vector de acciones para que lo rellene my_function()
    vector<struct action> actions; // <- fíjate aquí, actions lo **REDEFINES**

actions, antes de esa línea, era una variable de tipo action_wpr *, después de esa línea será un vector<struct action> hasta el final del scope.

Cambios estéticos para facilitar la explicación

Permíteme poder hacer unos cuantos cambios estéticos que me permitan trabajar con tus datos. No recomiendo usar struct action siempre, sino hacer un typedef que permita usar esta estructura como un único tipo. La llamaremos, por facilidad, action. También pondré los namespaces, ya que es una mala práctica de programación demasiado extendida usar using namespace std; en alguna cabecera.

Y por último alinearé los tipos de datos por un lado y las variables por otro para placer visual. Todo esto mejorará notablemente tu forma de trabajar leyendo código, que es realmente el trabajo que vas a realizar más tiempo.

Tu tipo de dato de origen pasará a ser:

typedef struct s_action
{
    std::string                   name       ;
    int                           type       ;
    std::vector < unsigned char > data       ;
    int                           value      ;
    unsigned char                 source     ;
    unsigned char                 destination;
} action;

El poner los ; alineados ayudará muchísimo de cara a un futuro cuando tengas que editar todos los datos secuencialmente, ya que con un editor que permita copiar en vertical, tendrás gran parte del trabajo hecho.

Posteriormente veo que haces un typedef, pero es que podías haberlo hecho desde antes (si es código heredado, haz baby steps: introduce el typedef, compila, y comprueba quién llama al struct con los errores de compilación, haces los cambios uno por uno quitando struct y listo)

typedef struct s_action_wpr
{
    char *          name       ;
    int             type       ;
    key_wpr         auth_key   ;
    unsigned char * data       ;
    int             value      ;
    unsigned char   source     ;
    unsigned char   destination;
} action_wpr;

Por amor a los debugguers, no pongas el mismo nombre al typedef que al struct.

Habrás visto también que al colocar el struct de esta manera, queda el tipo de datos unívocamente definido. Esto nos ayudará en la siguiente fase, que consiste en aprender hacer TDD en C para la migración de los datos de origen a los datos de destino.

TDD en C para la migración de datos

Lo primero que tienes que testar es la migración en sí. No puedes picar un código sin pruebas, si lo haces, corres el riesgo de que acabes desarrollando Skynet y tu código tome conciencia de sí mismo.

Así que, para la migración, deberás crear una función que te permita introducir un action y devuelva un action_wpr. De ahí te darás cuenta que tendrás que hacer una función para migrar cada parte que no se migre de forma directa.

Migración de action en action_wpr

Con un editor que permite la edición en vertical, puedes ver que los campos no son iguales, te sobra en action_wpr un campo (que nos has omitido, imagino, a propósito)

typedef struct s_action                        | typedef struct s_action_wpr
{                                              | {
    std::string                   name       ; |     char *          name       ;
    int                           type       ; |     int             type       ;
    std::vector < unsigned char > data       ; |     key_wpr         auth_key   ;
    int                           value      ; |     unsigned char * data       ;
    unsigned char                 source     ; |     int             value      ;
    unsigned char                 destination; |     unsigned char   source     ;
} action;                                      |     unsigned char   destination;
                                                 } action_wpr;

Así que voy a quitarlo a propósito, porque no hay forma de hacer el traspaso fácilmente:

typedef struct s_action                        | typedef struct s_action_wpr
{                                              | {
    std::string                   name       ; |     char *          name       ;
    int                           type       ; |     int             type       ;
    std::vector < unsigned char > data       ; |     unsigned char * data       ;
    int                           value      ; |     int             value      ;
    unsigned char                 source     ; |     unsigned char   source     ;
    unsigned char                 destination; |     unsigned char   destination;
} action;                                      | } action_wpr;

Mucho mejor.

¿Qué vemos que podemos traspasar sin hacer cambios? Pues los unsigned char, los int.

Nota: ¿Has visto que haya puesto una sola línea de código? No, a esto se le llama pensar antes de actuar, te acostumbrarás cuando te pase mil veces, así que asume siempre que lo haces mal. Cuando seas capaz de cambiar tu forma de hacer las cosas habrás pasado al siguiente nivel. No es un ataque, es un consejo.

¿Qué datos necesitas para terminar la migración que aún no hemos hecho? std::string y std::vector < unsigned char > en, respectivamente, char * y unsigned char *

Migrar std::string en char *

¿Conoces las propiedades de std::string? ¿Y las de char *? Viendo tu código, asumo que, efectivamente, no.

  • No son la misma "cosa"
  • std::string no es una forma de guardar caracteres uno detrás de otro, guardas cadenas de caracteres. Está indicado para guardar trozos de texto, y puedes realizarle operaciones de manipuación de cadenas de caracteres de forma directa con sus operadores.
  • char * es un puntero a un "vector" de una dimensión de elementos de char, igual que cuando estudias en matemáticas los vectores, van unos detrás de otros. Alguno de ellos puede ser \0, 0xFF o incluso 0xDE 0xAD 0xBE 0xEF, que puede quedar como 0xDEADBEEF, una referencia bastante habiutal en variables hexadecimales por su significado en inglés (vaca muerta).

Entendiendo que no son la misma cosa, ahora tienes que hacer una función a la que le pases el tipo de dato de un lado y devuelva el del otro con el contenido de entrada.

Ejemplos de tests que vas a tener que hacer:

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

char * funcion_migracion (std::string);

void test_harness_01 (void)
{
    /** Datos con los que voy a probar **/
    char * dato_char;
    char comparador_base[] = "test de prueba 01";

    /** Ejecución de la prueba **/
    dato_char = funcion_migracion ( std::string ( comparador_base ) ); // Al hacer esto, haces la conversión explícita de `char[]` a `std::string`, aunque no haría falta porque `funcion_migracion` sólo acepta `std::string` y lo convertiría internamente por ti
    
    /** Comprobar resultados esperados **/
    if ( dato_char == NULL )
    {
        printf("dato_char no puede ser NULL\r\n");
        return;
    }
    if ( strcmp(dato_char, comparador_base) != 0 )
    {
        printf("no se obtiene el restulado esperado\r\n");
    }

    /** Limpieza de los resultados obtenidos **/
    if ( dato_char != NULL )
    {
        free ( dato_char );
    }
}

Ahora pones diferentes resultados copiando y pegando la función test_harness_01 y modificando comparador_base para ir viendo que te sale lo que tiene que salir. Mientras no salgan las pruebas, quiere decir que tu código no está bien hecho.

¿Qué obtienes con esto? Aprendes a hacer pruebas, sabes exactamente qué quieres y siempre que ejecutes los tests, volverás a comprobar que no has roto nada en el camino. Como habrás imaginado, estas pruebas no se ejecutan en tu programa real, se ejecutan en un lugar separado de tu programa, al que vamos a llamar "espacio de desarrollo". Te ayuda a que, cuando tengas que hacer o romper algo, tengas presente qué rompes y qué no, y mejora tu forma de realizar código a largo plazo. Dado que parece que estás aprendiendo a programar, tener esta forma de realizar programas es una herramienta de lo más útil para tu día a día.

Sabiendo esto, ya puedes, como ejercicio, saber cómo migrar un std::vector de unsigned chars en un unsigned char *.

Migrar std::vector < unsigned char > en unsigned char *

¿Por qué no es tan fácil la migración y te has quedado atascado? Efectivamente, porque te has olvidado de un detallito sin importancia que los tests nos enseñarán llegado el momento.

Voy a plantearte el siguiente entorno de tests para que lo veas por ti mismo, y veas qué ocurre.

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

char * funcion_migracion_vector (std::vector < unsigned char >);

void test_harness_vector_01 (void)
{
    /** Datos con los que voy a probar **/
    char * dato_char;
    std::vector < unsigned char > comparador_base;

    comparador_base.push_back('\0');

    /** Ejecución de la prueba **/
    dato_char = funcion_migracion_vector ( comparador_base );
    
    /*
     * ¿Qué demonios tiene que ocurrir aquí?
     */
}

Acabas de descubrir que, por construcción, ¡un std::vector de unsigned chars permite insertar \0 en cualquier parte! ¿Y cómo podemos procesar los datos que hay detrás de \0? ¿Uso magia vudú?

Necesitarás, inevitablemente, un contador de todos los elementos de ese vector.

Y así es como se averiguan bugs de base que tienes en tu código y que desconocías. Igual que desconozco si eres profesional o no, sólo así salen estos errores, cuando haces pruebas (si las automatizas, directamente triunfas).

Una vez sabes cómo migrar los datos "difíciles" de migrar, hay que hacer el test de integración, que es migrar de una estructura a otra.

Migrar desde action hasta action_wpr

En TDD se usa, llegado a este punto, una cosa que llamaremos mocks, que permiten engañar al sistema para no hacer todas las llamadas internas (por ejemplo, engañar al sistema para que no llame a una base de datos, que es una operación lenta y costosa), como esto me huelo que va para wiki de comunidad, te lo pondré fácil y seguiremos con las pruebas sencillas, ahora tus estructuras pasarán, con el bug que mencioné antes, a esto:

typedef struct s_action                        | typedef struct s_action_wpr
{                                              | {
    std::string                   name       ; |     char *          name       ;
    int                           type       ; |     int             type       ;
    std::vector < unsigned char > data       ; |     unsigned char * data       ;
                                               |     unsigned int    n_data_elms;
    int                           value      ; |     int             value      ;
    unsigned char                 source     ; |     unsigned char   source     ;
    unsigned char                 destination; |     unsigned char   destination;
} action;                                      | } action_wpr;

Aquí tendrías el código con el que tendrías que hacer la prueba para la migración entre estructuras:

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

typedef struct s_action                       
{                                             
    std::string                   name       ;
    int                           type       ;
    std::vector < unsigned char > data       ;
                                              
    int                           value      ;
    unsigned char                 source     ;
    unsigned char                 destination;
} action;                                     

typedef struct s_action_wpr
{
    char *          name       ;
    int             type       ;
    unsigned char * data       ;
    unsigned int    n_data_elms;
    int             value      ;
    unsigned char   source     ;
    unsigned char   destination;
} action_wpr;



action_wpr * funcion_migracion_structs (action);

void test_harness_migration_structs_01 (void)
{
    /** Datos con los que voy a probar **/
    action_wpr * dato_char;
    action base;

    base.name("namestring prueba 01");
    base.type = 0xDEADBEEF;
    base.data.push_back('h');
    base.data.push_back('\0');
    base.data.push_back('l');
    base.data.push_back('a');
    base.data.push_back(' ');
    base.data.push_back('m');
    base.data.push_back('u');
    base.data.push_back('n');
    base.data.push_back('d');
    base.data.push_back('o');
    base.value = -35;
    base.source = 0x33;
    base.destination = 0xCC;

    /** Ejecución de la prueba **/
    dato_char = funcion_migracion_vector ( base );
    
    /*
     * ¿Qué demonios tiene que ocurrir aquí?
     */
}

Como has visto, he dejado sin completar. ¿Qué faltaría?

Pues, con esto, puedes hacer la migración entre elemento y elemento. Tu siguiente y último paso, ahora que ya sabes cómo se testa en escalera, directamente será testar cómo se migra un "vector" de elementos de un tipo a otro.

Qué has aprendido

Has aprendido con esto que uno no se pone a programar a lo loco. Si tienes que migrar una estructura, haz pruebas para que dicha migración (migración como sinónimo de traspaso, de traductor, de conversor) esté siempre controlada.

Hay montones de frameworks que funcionan estupendamente en C y C++ para este fin. Y dado que los compiladores de C++ admiten código en C (usando extern 'C'), no tendrías problemas de ningún tipo en usar un framework de testing en C++, por ejemplo.

Mi respuesta no te hace código por ti. Ni pretendo que lo haga. Lo que hago es que veas qué problemática tienes a la hora de realizar código y una posible solución a corto y medio plazo que te ayudará.

Origen Enlace

Lo que te está ocurriendo es que estás llamando a la variable actions desde el scope y desde la definición de la función.

Es decir, has sobrescrito actions de lo que era originalmente, por eso, cuando has intentado acceder a la otra definición, no has podido, porque ya no sabría a cuál te estás refiriendo.

another_struct_wpr *my_function_create(action_wpr *actions, unsigned char *nActions)
{
    // Instanciamos el vector de acciones para que lo rellene my_function()
    vector<struct action> actions; // <- fíjate aquí, actions lo **REDEFINES**

actions, antes de esa línea, era una variable de tipo action_wpr *, después de esa línea será un vector<struct action> hasta el final del scope.