[PATCH 1/3] [MTD-UTILS] Unified reading from standard input and from file
Jehan Bing
jehan at orb.com
Wed Jun 10 13:11:03 EDT 2009
Artem Bityutskiy wrote:
> On Tue, 2009-06-09 at 16:04 -0700, Jehan Bing wrote:
>
>> - readlen = meminfo.writesize;
>>
>> - if (ifd != STDIN_FILENO) {
>> - int tinycnt = 0;
>> -
>> - if (pad && (imglen < readlen))
>> - {
>> - readlen = imglen;
>> - erase_buffer(writebuf + readlen, meminfo.writesize - readlen);
>> - }
>> + {
>> + readlen = meminfo.writesize;
>>
>>
>
> Err, why do you need these spare { } ?
>
Right, I wanted to comment on that but forgot by the time I started the
email.
My idea was to make the patches clearer. Patch 3/3 puts that code inside
a condition. So to avoid extra changes just because of the indentation,
I added the extra { } in this patch instead since I was already heavily
modifying it anyway.
Do you prefer me to remove them?
>> - /* Read Page Data from input file */
>> - while(tinycnt < readlen) {
>> - cnt = read(ifd, writebuf + tinycnt, readlen - tinycnt);
>> - if (cnt == 0) { // EOF
>> - break;
>> - } else if (cnt < 0) {
>> - perror ("File I/O error on input file");
>> - goto closeall;
>> - }
>> - tinycnt += cnt;
>> - }
>> - } else {
>> int tinycnt = 0;
>>
>
> Err, is it normal C do do stuff like
> {
> readlen = meminfo.writesize;
> int tinycnt += cnt;
>
> ? I think this is C++.
>
gcc didn't complain. Easy to fix.
More information about the linux-mtd
mailing list