<div dir="ltr"><div dir="ltr"><div class="gmail_default" style="font-size:small">Hi Petr,</div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">Petr Vorel <<a href="mailto:pvorel@suse.cz" target="_blank">pvorel@suse.cz</a>> wrote:<br></div><div dir="ltr" class="gmail_attr"><br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><span class="gmail_default" style="font-size:small">...</span><br>
> > +get_btrfs_size()<br>
<br>
<br>
> What about renaming at_least_1G_mem() or check_space_for_btrfs()?<br>
Good point. I'm slightly for check_space_for_btrfs().<br>
<br>
at_least_1G_mem() is also good, but for that I'd also move tst_res TINFO "not<br>
enough space for Btrfs" out of the function and put it into generate_vars(). But<br>
since it's used only for btrfs I slightly prefer check_space_for_btrfs(). But no<br>
strong opinion about it.<br></blockquote><div><br></div><div><div class="gmail_default" style="font-size:small">Agree, thanks!</div><div class="gmail_default" style="font-size:small"><br></div><div class="gmail_default" style="font-size:small">Btw I suddenly think that we could have a nicer name initialize_vars()</div><div class="gmail_default" style="font-size:small">to replace generate_vars(), because we just use it once to initiate the</div><div class="gmail_default" style="font-size:small">test variables in the setup phase. </div><div class="gmail_default" style="font-size:small"><br></div><div class="gmail_default" style="font-size:small">Anyway, it's only my feelings and also depends on your preference too.</div><br></div></div><div><br></div>-- <br><div dir="ltr"><div dir="ltr"><div>Regards,<br></div><div>Li Wang<br></div></div></div></div>