Looking around the online I have yet to find a definitive answer related to the fgets() character pointer array (char *restrict s) in terms of efficient memory allocation.
synopsis: char *fgets(char *restrict s, int n, FILE *restrict stream);
Looking at the fgets() - specification
From my understanding of this spec, you should be allocating based on the LINE_MAX macro defined in the <limits.h> - specification, simply because you don't know how many characters each line has.
If I run - printf("LINE_MAX BYTES: %d\n", LINE_MAX); the result is 2048.
That being said, declaring char *line[LINE_MAX] -or- char *line[2048] seems inefficiently large to me, however, this may be the best way of doing it?
//PROGRAM
#include <stdio.h>
#include <limits.h>
int main(void){
char line[LINE_MAX];
FILE *fp = fopen("file.txt", "r");
while(fgets(line, LINE_MAX + 1, fp)){
printf("%s", line);
}
fclose(fp);
return(0);
}
//FILE (file.txt)
Lorem ipsum dolor sit amet, consectetur adipiscing elit, sed do eiusmod tempor incididunt ut labore et dolore magna aliqua.
Lorem ipsum dolor sit amet, consectetur adipiscing elit, sed do eiusmod tempor incididunt ut labore et dolore magna aliqua.
Lorem ipsum dolor sit amet, consectetur adipiscing elit, sed do eiusmod tempor incididunt ut labore et dolore magna aliqua.
Lorem ipsum dolor sit amet, consectetur adipiscing elit, sed do eiusmod tempor incididunt ut labore et dolore magna aliqua.
Lorem ipsum dolor sit amet, consectetur adipiscing elit, sed do eiusmod tempor incididunt ut labore et dolore magna aliqua.
Lorem ipsum dolor sit amet, consectetur adipiscing elit, sed do eiusmod tempor incididunt ut labore et dolore magna aliqua.
Lorem ipsum dolor sit amet, consectetur adipiscing elit, sed do eiusmod tempor incididunt ut labore et dolore magna aliqua.
Lorem ipsum dolor sit amet, consectetur adipiscing elit, sed do eiusmod tempor incididunt ut labore et dolore magna aliqua.
Lorem ipsum dolor sit amet, consectetur adipiscing elit, sed do eiusmod tempor incididunt ut labore et dolore magna aliqua.
Lorem ipsum dolor sit amet, consectetur adipiscing elit, sed do eiusmod tempor incididunt ut labore et dolore magna aliqua.
Unless your file has some special properties, allocating LINE_MAX bytes is the best approach. Although you may be wasting some memory there, allocation in the automatic memory is really cheap, because the space is being "reserved" more than "allocated", with lots of hardware support on most modern architectures.
On the other hand, if you know that due to the format of your file the lines cannot exceed a certain length MY_LINE, you could use MY_LINE+2* as your limit instead:
char line[MY_LINE+2];
For example, a program reading a file in uuendode format needs at most 62 characters per line, so it could define
#define MY_LINE 62
* Assuming you're on UNIX, you need space for '\n' and '\0', hence the +2 part. If you are on Windows, do +3 to accommodate the additional '\r' character.