Current-Users archive

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index][Old Index]

Re: Floppy image overflow



On 2017/02/08 13:13, Christos Zoulas wrote:
Sounds, great! Thanks!

Thanks :-).

On 2017/02/09 0:42, Izumi Tsutsui wrote:
I'm afraid these unused functions should be "do {} while (/*CONSTCOND*/0)"
rather than "empty" comments because they could be used in if clauses
without braces (which is allowed in our /usr/share/misc/style):
 https://nxr.netbsd.org/xref/src/sbin/fsck_ffs/inode.c?r=1.71#421
---
			if (is_ufs2)
				swap_dinode2(inodebuf, lastinum - inumber);
			else
				swap_dinode1(inodebuf, lastinum - inumber);
			bwrite(fswritefd, (char *)inodebuf, dblk, size);
---

All other part looks fine.

Thank you so much for your detailed review. I fixed and committed it!

rin


Home | Main Index | Thread Index | Old Index