[Create mexfunction from C code] warning: implicit declaration of function

Question

shdotcom shdotcom 2019-9-19

0
链接

此问题的直接链接

https://ww2.mathworks.cn/matlabcentral/answers/481175-create-mexfunction-from-c-code-warning-implicit-declaration-of-function

评论： shdotcom shdotcom 2019-9-19

why I got this warning message "warning: implicit declaration of function 'myfunction' " when I build the mex function?

#include <mex.h>
#include <matrix.h>
#include<stdio.h>
void mexFunction(int nlhs, mxArray *plhs[], int nrhs, const mxArray *prhs[])
{
    int argc = 0;
    char **argv;
    int i, result;
    argc = nrhs;
    argv = (char **) mxCalloc( argc, sizeof(char *) );
    for (i = 0; i < nrhs; i++){
        if( !mxIsChar( prhs[i] ) ){
        mexErrMsgTxt("Input must be of type char.");
        return;
        }
        argv[i] = mxArrayToString( prhs[i] );
    }
    result = myfunction( argc, argv );
    for( i=argc-1; i<=0; i-- )
        mxFree( argv[i] );
    
    mxFree( argv );
    if( result )
        mexErrMsgTxt("main function causes an error");
}
float square ( float x )  
{
    float p ;
    p = x * x ;
    return ( p ) ;
}
int myfunction(int argc, char *argv[]){
    float m, n ;
    m=5;
    n = square(m);                      
    printf ( "\nSquare of the given number %f is %f",m,n );
}

>> mex myfunction.c
Building with 'MinGW64 Compiler (C)'.
D:\TestMexFunction\myfunction.c: In function 'mexFunction':
D:\TestMexFunction\myfunction.c:21:14: warning: implicit declaration of function 'myfunction' [-Wimplicit-function-declaration]
     result = myfunction( argc, argv );
              ^~~~~~~~~~

I am using Matlab 64bit and the code works if I remove the square() function

0 个评论
显示 -2更早的评论隐藏 -2更早的评论

请先登录，再进行评论。

请先登录，再回答此问题。

Answer 1

James Tursa 2019-9-19

0
链接

此回答的直接链接

https://ww2.mathworks.cn/matlabcentral/answers/481175-create-mexfunction-from-c-code-warning-implicit-declaration-of-function#answer_392667

编辑：James Tursa 2019-9-19

在 MATLAB Online 中打开

You call myfunction( ) before the compiler has seen a prototype or definition of myfunction( ). Either move your square( ) and myfunction( ) code above the mexFunction( ) code, or put prototypes at the top of your code. E.g., put these lines at the front:

float square ( float x );
int myfunction(int argc, char *argv[]);

When you call a function before the compiler knows about it, the compiler assumes the function returns an int and that the argument types are exactly as listed. I.e., you get no argument type promotion and the function return type might be wrong. This can lead to crashes, or worse a wrong result without you knowing anything went wrong. C lets you get away with this and will compile the code anyway ... C++ will not let you get away with this and will not compile the code.

Bottom line: Always make sure you have function prototypes or definitions appearing in the code before they are used.

Side Note: On earlier versions of MATLAB, the mxArrayToString allocations were not put on the gargabe collection list, so your current code will leak memory on those versions if you run into an error condition.

2 个评论
显示无隐藏无

shdotcom shdotcom 2019-9-19

Thank you very much

shdotcom shdotcom 2019-9-19

"Side Note: On earlier versions of MATLAB, the mxArrayToString allocations were not put on the gargabe collection list, so your current code will leak memory on those versions if you run into an error condition."

Do you mean that I have to change mxArrayToString?

请先登录，再进行评论。