Hi!

> Split generic and x86 s2ram code: the new files (s2ram.{c,h}) contains
> platform specific code moved out of the main file.
> 
> Signed-Off-By: Luca Tettamanti <[EMAIL PROTECTED]>

> --- a/s2ram-x86.c     1970-01-01 01:00:00.000000000 +0100
> +++ b/s2ram-x86.c     2007-04-10 21:38:51.000000000 +0200
> @@ -0,0 +1,331 @@

I'd add some header there, with short explanation, GPL, copyrights,
etc.

> +void s2ram_usage_platform() {

{ on new line, please


> +     printf(
> +            "    -n, --test:       test if the machine is in the database.\n"
> +            "                      returns 0 if known and supported\n"
> +            "the following options are only available with --force:\n"
> +            "    -s, --vbe_save:   save VBE state before suspending and "
> +                                    "restore after resume.\n"

Otherwise it looks ok to me.
                                                                        Pavel
-- 
(english) http://www.livejournal.com/~pavelmachek
(cesky, pictures) 
http://atrey.karlin.mff.cuni.cz/~pavel/picture/horses/blog.html

-------------------------------------------------------------------------
Take Surveys. Earn Cash. Influence the Future of IT
Join SourceForge.net's Techsay panel and you'll get the chance to share your
opinions on IT & business topics through brief surveys-and earn cash
http://www.techsay.com/default.php?page=join.php&p=sourceforge&CID=DEVDEV
_______________________________________________
Suspend-devel mailing list
Suspend-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/suspend-devel

Reply via email to