Hi Jan,

> > +START_TEST(test_determine_FAT_bits_sec_per_clus_zero)
> > +{
> > +   struct fat_boot_sector sector = {
> > +           .sec_per_clus = 0, /* test we do not divide by
> > this value */  
> 
> This explicit assigment is not done in
> test_determine_FAT_bits_fat_sector_size_zero. Should it be for
> documentation purposes?

yes, it's just for documentation purposes (since the default value
would be 0 anyway, but it's too implicit for my taste). 

> Space before tab - fixing it up on merge.

Oops, thanks!

Kind regards,
  Michael

-- 
Michael Adler

Siemens AG
Technology
Connectivity & Edge
Smart Embedded Systems
T CED SES-DE
Otto-Hahn-Ring 6
81739 Munich, Germany

Siemens Aktiengesellschaft: Chairman of the Supervisory Board: Jim
Hagemann Snabe; Managing Board: Roland Busch, Chairman, President and
Chief Executive Officer; Cedrik Neike, Matthias Rebellius, Ralf P.
Thomas, Judith Wiese; Registered offices: Berlin and Munich, Germany;
Commercial registries: Berlin-Charlottenburg, HRB 12300, Munich, HRB
6684; WEEE-Reg.-No. DE 23691322

-- 
You received this message because you are subscribed to the Google Groups "EFI 
Boot Guard" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to [email protected].
To view this discussion on the web visit 
https://groups.google.com/d/msgid/efibootguard-dev/20230816154637.165d8adb%40backstein.

Reply via email to