Thread: To exit using return 1

Looks good. My comments:

• Using return 0; is fine, but if you want to use return 1, then it should be because you say, anticipate running this via a shell script where you specifically want to to inspect error return values. If not, then you should #include <stdlib.h> and return EXIT_FAILURE (and correspondingly change return 0 to return EXIT_SUCCESS).
• If you do want to return non-zero from the main function, then it makes sense that the error output should go to stderr instead of stdout. This can be useful, e.g., when a shell script that invokes your program wants to pipe error output to an error log.
• Instead of a while loop, perhaps a for loop is more appropriate since you are decrementing a counter.
• Your use of if statements with else blocks is fine. However, sometimes, especially when the nesting level gets deep because of validation, a somewhat different style can be used that returns early when validation fails, e.g.,
  
  Code:

  #include <stdio.h>
  #include <stdlib.h>
  
  int main(int argc, char *argv[]) {
      int count;
  
      if (argc != 3) {
          printf("\t%s [number] [name]\n", argv[0]);
          return EXIT_SUCCESS;
      }
  
      if (sscanf(argv[1], "%i", &count) != 1) {
          fprintf(stderr, "Error: not an integer!\n");
          return EXIT_FAILURE;
      }
  
      for (; count > 0; count--) {
          printf("Hello %s!\n", argv[2]);
      }
  
      return EXIT_SUCCESS;
  }

  If you are compiling with respect to C99 or later, move the declaration of count to just before the sscanf call, like in the code you posted, following the principle that variables should be declared near first use.