Error invalid storage class for function

• 
• Forum
• General Programming Boards
• C Programming
• Invalid storage class for function

1. 10-15-2012

   
   

   #1

   

   
   Registered User
   

   ---

   Invalid storage class for function
   

   Hey again,
   So this time I am trying to make a function to call twice instead of repeating the chunk of code twice. When I was not using the function the code worked fine so it seems like I set up the function wrong. Here is what I have when the ‘invalid storage class’ error occurs:

   Code:

   static void getAndPrint (char mainFile[], FILE *search);
   
   void getAndPrint (char mainFile[], FILE *search)
   {
       char data1[256];
       char data2[256];
       char line[256];
       char term[12] = "FindMe";
       char dest[] = "/home/user/Desktop/folder/";
       strcat (dest, mainFile);
       FILE *data;
       data = fopen (dest, "w");
   
       if (search != NULL)
       {
            while (fgets (line, sizeof(line), search)
            {
                  if ( strstr (line, term) != NULL)
                  {
                       fgets (data1 -2, sizeof(data1), search);
                       fprintf (data, "%sn", data1);
                       fgets (data2 -2, sizeof(data2), search);
                       fprintf (data, "%sn", data2);
                  }
            }
       }
     fclose(search);
   }

   Thank you for your time.
   Dominic

   ---

2. 10-15-2012

   
   

   #2

   

   
   C++ Witch
   

   

   ---

   On which line does your compiler report that error and what is the exact error message?

   My guess would be that the problem has something to do with you forward declaring getAndPrint to be static, but as I don’t normally forward declare my static functions, I don’t dare to say that that is the problem without some investigation of my own.

   Last edited by laserlight; 10-15-2012 at 10:14 AM.

   Originally Posted by Bjarne Stroustrup (2000-10-14)

   I get maybe two dozen requests for help with some sort of programming or design problem every day. Most have more sense than to send me hundreds of lines of code. If they do, I ask them to find the smallest example that exhibits the problem and send me that. Mostly, they then find the error themselves. «Finding the smallest program that demonstrates the error» is a powerful debugging tool.

   Look up a C++ Reference and learn How To Ask Questions The Smart Way

   ---

3. 10-15-2012

   
   

   #3

   

   
   and the hat of int overfl
   

   

   ---

   > fgets (data1 -2, sizeof(data1), search);
   What are you hoping to achieve with this out of bound pointer to your buffer?

   ---

4. 10-15-2012

   
   

   #4

   

   
   Registered User
   

   ---

   From the example code, it would be on line 3 after I have made the prototype. The error is «invalid storage class for function ‘getAndPrint’ «. If I dont declare the protoype static I get errors: «colficting types for ‘getAndPrint», «previous implicit declaration of ‘getAndPrint’ was here», «static declaration of ‘getAndPrint’ follows non-static declaratation».

   ---

5. 10-15-2012

   
   

   #5

   

   
   — — — — — — — —
   

   

   ---

   Do you just need to include the word «static» in front of your function definition (and not just in the prototype)?

   Other problems:

   Since dest is defined like this:
   char dest[] = «/home/user/Desktop/folder/»;
   it does not have the storage to concatenate more chars into:
   strcat (dest, mainFile);
   Try this instead:
   char dest[100] = «/home/user/Desktop/folder/»;
   and use strncat to prevent possible buffer overflow
   strncat (dest, mainFile, sizeof dest — strlen(dest) — 1);

   And in your fgets, why are you subtracting 2 from data1 and data2? That’s undefined.

   The cost of software maintenance increases with the square of the programmer’s creativity. — Robert D. Bliss

   ---

6. 10-15-2012

   
   

   #6

   

   
   C++ Witch
   

   

   ---

   Also, you seem to be confused between the search and data FILE pointers.

   Originally Posted by InicDominus

   From the example code, it would be on line 3 after I have made the prototype. The error is «invalid storage class for function ‘getAndPrint’ «. If I dont declare the protoype static I get errors: «colficting types for ‘getAndPrint», «previous implicit declaration of ‘getAndPrint’ was here», «static declaration of ‘getAndPrint’ follows non-static declaratation».

   It might be good for you to reduce your program to the smallest and simplest program that you expect to compile but which demonstrates this problem, and post it. As it stands, it is possible to take your example code, fix an unrelated syntax error, then have a compilable (though incorrect) program.

   Originally Posted by Bjarne Stroustrup (2000-10-14)

   I get maybe two dozen requests for help with some sort of programming or design problem every day. Most have more sense than to send me hundreds of lines of code. If they do, I ask them to find the smallest example that exhibits the problem and send me that. Mostly, they then find the error themselves. «Finding the smallest program that demonstrates the error» is a powerful debugging tool.

   Look up a C++ Reference and learn How To Ask Questions The Smart Way

   ---

7. 10-15-2012

   
   

   #7

   

   
   Registered User
   

   ---

   Salem, it gathers a whole line of data from a file after finding a keyword. I know it may be sloppy as I have never used C before but it works fine.

   ---

8. 10-15-2012

   
   

   #8

   

   
   C++ Witch
   

   

   ---

   Originally Posted by InicDominus

   I know it may be sloppy as I have never used C before but it works fine.

   It may appear to work fine, yet still be horribly wrong. One of the effects of undefined behaviour is that the code works perfectly until the moment that you really need it to work perfectly (e.g., when your program is being graded or when it is being demonstrated to the client).

   Originally Posted by Bjarne Stroustrup (2000-10-14)

   I get maybe two dozen requests for help with some sort of programming or design problem every day. Most have more sense than to send me hundreds of lines of code. If they do, I ask them to find the smallest example that exhibits the problem and send me that. Mostly, they then find the error themselves. «Finding the smallest program that demonstrates the error» is a powerful debugging tool.

   Look up a C++ Reference and learn How To Ask Questions The Smart Way

   ---

9. 10-15-2012

   
   

   #9

   

   
   and the hat of int overfl
   

   

   ---

   > I know it may be sloppy as I have never used C before but it works fine.
   So said the blind man who ran across the road, and survived the first time he tried it.

   Yes, it is sloppy, and you’ll get your ass handed to you on a plate at some point in the form of a segfault.
   Feel free to carry on as you are, until I tell you to add
   #include <I_told_you_so.h>

   Never make the mistake of assuming «works» == «bug free» when it comes to C programming. All sorts of crap can seem to «work» in the beginning, only to crash and burn later on.

   ---

10. 10-15-2012

    
    

    #10

    

    
    Registered User
    

    ---

    Alright thank you for the advice. I will go fix that.

    ---

Tags for this Thread

У меня есть устаревший макрос

#define ASYNC_FUNCTION(x)                                               
void x(void) {                                                          
    static void internal ## x (ASYNCIOCB *);                            
    ASYNCIOCB *aiocb = AcquireAsyncIOCB();                              
    CallAsyncNativeFunction_md(aiocb, internal ## x);                   
}                                                                       
static void internal ## x (ASYNCIOCB *aiocb)

За которым следует этот

#define ASYNC_FUNCTION_START(x)  ASYNC_FUNCTION(x) {
#define ASYNC_FUNCTION_END       ASYNC_resumeThread(); }

И их использование выглядит так:

ASYNC_FUNCTION_START(Occasional_function_name)
{
    //actual stuff    
}
ASYNC_FUNCTION_END

Он отлично компилируется с cl, но gcc дает

invalid storage class for function ‘internalOccasional_function_name’
     static void internal##x (ASYNCIOCB *);    
                 ^

Я попытался расширить их все, чтобы посмотреть, что из этого получится, и не нашел ничего сломанного. Я также искал незакрытые изогнутые скобки в файле и нашел другие макросы, подобные этому

#define Foo_Bar1()  {                                                
  extern int foo;                                                    
  int bar = foo;                                                     
  if (condition) {                                                   
    Bar_Foo();                                                       
  }                                                                  

#define Foo_Bar2()                                                   
  if (condibar != footion1){                                         
    AbortAsyncIOCB(aiocb);                                           
    return;                                                          
  }                                                                  
  if (condition) {                                                   
    Bar_Foo();                                                       
  }                                                                  
}

Никакие другие заголовки не включены, поэтому кроме этого последнего макроса, выглядящего странно, я не смог найти никаких очевидных ошибок. Я использую cygwin и довольно невежественен.

Вы не можете объявить статическую функцию внутри другой. Вам необходимо разместить объявление за пределами void(x):

#define ASYNC_FUNCTION(x)                                               
static void internal ## x (ASYNCIOCB *);                                
void x(void) {                                                          
    ASYNCIOCB *aiocb = AcquireAsyncIOCB();                              
    CallAsyncNativeFunction_md(aiocb, internal ## x);                   
}                                                                       
static void internal ## x (ASYNCIOCB *aiocb)

Затем, если вы запустите исходный код только через препроцессор через gcc -E, вы получите что-то вроде этого (добавлен дополнительный интервал):

static void internalOccasional_function_name (ASYNCIOCB *);
void Occasional_function_name(void)
{
    ASYNCIOCB *aiocb = AcquireAsyncIOCB();
    CallAsyncNativeFunction_md(aiocb, internalOccasional_function_name);
}
static void internalOccasional_function_name (ASYNCIOCB *aiocb) 
{
    {
        int a=1;
    }
    ASYNC_resumeThread();
}

В языке C объявления локальных функций могут дополнительно включать спецификатор класса хранения extern. Это единственный спецификатор класса хранения, который может иметь объявление локальной функции. Никакие другие спецификаторы класса хранения не допускаются.

6.7.1 / 7:

Объявление идентификатора функции, имеющей область видимости блока, не должно иметь явного спецификатора класса хранения, кроме extern .

При сборке кода командой:

/opt/cross/arm-cortexa7hf-linux-gnueabi/bin/arm-cortexa7hf-linux-gnueabi-gcc -o test-bin -ltinfo test.c

Выдается следующая ошибка:

In file included from /opt/cross/arm-cortexa7hf-linux-gnueabi/arm-cortexa7hf-linux-gnueabi/sysroot/usr/include/endian.h:60:0,
                 from /opt/cross/arm-cortexa7hf-linux-gnueabi/arm-cortexa7hf-linux-gnueabi/sysroot/usr/include/sys/types.h:216,
                 from /opt/cross/arm-cortexa7hf-linux-gnueabi/arm-cortexa7hf-linux-gnueabi/sysroot/usr/include/termcap.h:49,
                 from test3.c:517:
/opt/cross/arm-cortexa7hf-linux-gnueabi/arm-cortexa7hf-linux-gnueabi/sysroot/usr/include/bits/byteswap.h: In function 'main':
/opt/cross/arm-cortexa7hf-linux-gnueabi/arm-cortexa7hf-linux-gnueabi/sysroot/usr/include/bits/byteswap.h:44:1: error: invalid storage class for function '__bswap_32'
 __bswap_32 (unsigned int __bsx)
 ^
/opt/cross/arm-cortexa7hf-linux-gnueabi/arm-cortexa7hf-linux-gnueabi/sysroot/usr/include/bits/byteswap.h:75:1: error: invalid storage class for function '__bswap_64'
 __bswap_64 (__uint64_t __bsx)
 ^

Подскажите, по какой причине это происходит?

У меня по этой причине ни одна утилита/библиотека не собирается с поддержкой ncurses.