Replace failwith (sprintf...) with failwithf
[virt-top.git] / virt-top / virt_top.ml
old mode 100755 (executable)
new mode 100644 (file)
index b3e2628..1a39e06
@@ -21,12 +21,15 @@ open Printf
 open ExtList
 open Curses
 
+open Virt_top_gettext.Gettext
 open Virt_top_utils
 
 module C = Libvirt.Connect
 module D = Libvirt.Domain
 module N = Libvirt.Network
 
+let rcfile = ".virt-toprc"
+
 (* Hook for XML support (see virt_top_xml.ml). *)
 let parse_device_xml : (int -> [>`R] D.t -> string list * string list) ref =
   ref (
@@ -36,7 +39,7 @@ let parse_device_xml : (int -> [>`R] D.t -> string list * string list) ref =
 (* Hooks for CSV support (see virt_top_csv.ml). *)
 let csv_start : (string -> unit) ref =
   ref (
-    fun _ -> failwith "virt-top was compiled without support for CSV files"
+    fun _ -> failwith (s_ "virt-top was compiled without support for CSV files")
   )
 let csv_write : (string list -> unit) ref =
   ref (
@@ -47,7 +50,7 @@ let csv_write : (string list -> unit) ref =
 let parse_date_time : (string -> float) ref =
   ref (
     fun _ ->
-      failwith "virt-top was compiled without support for dates and times"
+      failwith (s_ "virt-top was compiled without support for dates and times")
   )
 
 (* Sort order. *)
@@ -59,15 +62,15 @@ let all_sort_fields = [
   NetRX; NetTX; BlockRdRq; BlockWrRq
 ]
 let printable_sort_order = function
-  | Processor -> "%CPU"
-  | Memory -> "%MEM"
-  | Time -> "TIME (CPU time)"
-  | DomainID -> "Domain ID"
-  | DomainName -> "Domain name"
-  | NetRX -> "Net RX bytes"
-  | NetTX -> "Net TX bytes"
-  | BlockRdRq -> "Block read reqs"
-  | BlockWrRq -> "Block write reqs"
+  | Processor -> s_ "%CPU"
+  | Memory -> s_ "%MEM"
+  | Time -> s_ "TIME (CPU time)"
+  | DomainID -> s_ "Domain ID"
+  | DomainName -> s_ "Domain name"
+  | NetRX -> s_ "Net RX bytes"
+  | NetTX -> s_ "Net TX bytes"
+  | BlockRdRq -> s_ "Block read reqs"
+  | BlockWrRq -> s_ "Block write reqs"
 let sort_order_of_cli = function
   | "cpu" | "processor" -> Processor
   | "mem" | "memory" -> Memory
@@ -76,7 +79,9 @@ let sort_order_of_cli = function
   | "name" -> DomainName
   | "netrx" -> NetRX | "nettx" -> NetTX
   | "blockrdrq" -> BlockRdRq | "blockwrrq" -> BlockWrRq
-  | str -> failwith (str ^ ": sort order should be: cpu|mem|time|id|name|netrx|nettx|blockrdrq|blockwrrq")
+  | str ->
+      failwithf (f_ "%s: sort order should be: %s")
+       str "cpu|mem|time|id|name|netrx|nettx|blockrdrq|blockwrrq"
 let cli_of_sort_order = function
   | Processor -> "cpu"
   | Memory -> "mem"
@@ -96,7 +101,8 @@ let display_of_cli = function
   | "pcpu" -> PCPUDisplay
   | "block" -> BlockDisplay
   | "net" -> NetDisplay
-  | str -> failwith (str ^ ": display should be task|pcpu|block|net")
+  | str ->
+      failwithf (f_ "%s: display should be %s") str "task|pcpu|block|net"
 let cli_of_display = function
   | TaskDisplay -> "task"
   | PCPUDisplay -> "pcpu"
@@ -135,7 +141,7 @@ let start_up () =
   (* Read command line arguments. *)
   let rec set_delay newdelay =
     if newdelay <= 0. then
-      failwith "-d: cannot set a negative delay";
+      failwith (s_ "-d: cannot set a negative delay");
     delay := int_of_float (newdelay *. 1000.)
   and set_uri = function "" -> uri := None | u -> uri := Some u
   and set_sort order = sort_order := sort_order_of_cli order
@@ -148,31 +154,58 @@ let start_up () =
   and no_init_file () = init_file := NoInitFile
   and set_init_file filename = init_file := InitFile filename
   and set_end_time time = end_time := Some ((!parse_date_time) time)
+  and display_version () =
+    printf "virt-top %s ocaml-libvirt %s\n"
+      Virt_top_version.version Libvirt_version.version;
+    exit 0
   in
   let argspec = Arg.align [
-    "-1", Arg.Unit set_pcpu_mode, " Start by displaying pCPUs (default: tasks)";
-    "-2", Arg.Unit set_net_mode, " Start by displaying network interfaces";
-    "-3", Arg.Unit set_block_mode, " Start by displaying block devices";
-    "-b", Arg.Set batch_mode, " Batch mode";
-    "-c", Arg.String set_uri, "uri Connect to URI (default: Xen)";
-    "--connect", Arg.String set_uri, "uri Connect to URI (default: Xen)";
-    "--csv", Arg.String set_csv, "file Log statistics to CSV file";
-    "--no-csv-cpu", Arg.Clear csv_cpu, " Disable CPU stats in CSV";
-    "--no-csv-block", Arg.Clear csv_block, " Disable block device stats in CSV";
-    "--no-csv-net", Arg.Clear csv_net, " Disable net stats in CSV";
-    "-d", Arg.Float set_delay, "delay Delay time interval (seconds)";
-    "--debug", Arg.Set_string debug_file, "file Send debug messages to file";
-    "--end-time", Arg.String set_end_time, "time Exit at given time";
-    "--hist-cpu", Arg.Set_int historical_cpu_delay, "secs Historical CPU delay";
-    "--init-file", Arg.String set_init_file, "file Set name of init file";
-    "--no-init-file", Arg.Unit no_init_file, " Do not read init file";
-    "-n", Arg.Set_int iterations, "iterations Number of iterations to run";
-    "-o", Arg.String set_sort, "sort Set sort order (cpu|mem|time|id|name)";
-    "-s", Arg.Set secure_mode, " Secure (\"kiosk\") mode";
-    "--script", Arg.Set script_mode, " Run from a script (no user interface)";
+    "-1", Arg.Unit set_pcpu_mode,
+      " " ^ s_ "Start by displaying pCPUs (default: tasks)";
+    "-2", Arg.Unit set_net_mode,
+      " " ^ s_ "Start by displaying network interfaces";
+    "-3", Arg.Unit set_block_mode,
+      " " ^ s_ "Start by displaying block devices";
+    "-b", Arg.Set batch_mode,
+      " " ^ s_ "Batch mode";
+    "-c", Arg.String set_uri,
+      "uri " ^ s_ "Connect to URI (default: Xen)";
+    "--connect", Arg.String set_uri,
+      "uri " ^ s_ "Connect to URI (default: Xen)";
+    "--csv", Arg.String set_csv,
+      "file " ^ s_ "Log statistics to CSV file";
+    "--no-csv-cpu", Arg.Clear csv_cpu,
+      " " ^ s_ "Disable CPU stats in CSV";
+    "--no-csv-block", Arg.Clear csv_block,
+      " " ^ s_ "Disable block device stats in CSV";
+    "--no-csv-net", Arg.Clear csv_net,
+      " " ^ s_ "Disable net stats in CSV";
+    "-d", Arg.Float set_delay,
+      "delay " ^ s_ "Delay time interval (seconds)";
+    "--debug", Arg.Set_string debug_file,
+      "file " ^ s_ "Send debug messages to file";
+    "--end-time", Arg.String set_end_time,
+      "time " ^ s_ "Exit at given time";
+    "--hist-cpu", Arg.Set_int historical_cpu_delay,
+      "secs " ^ s_ "Historical CPU delay";
+    "--init-file", Arg.String set_init_file,
+      "file " ^ s_ "Set name of init file";
+    "--no-init-file", Arg.Unit no_init_file,
+      " " ^ s_ "Do not read init file";
+    "-n", Arg.Set_int iterations,
+      "iterations " ^ s_ "Number of iterations to run";
+    "-o", Arg.String set_sort,
+      "sort " ^ sprintf (f_ "Set sort order (%s)") "cpu|mem|time|id|name";
+    "-s", Arg.Set secure_mode,
+      " " ^ s_ "Secure (\"kiosk\") mode";
+    "--script", Arg.Set script_mode,
+      " " ^ s_ "Run from a script (no user interface)";
+    "--version", Arg.Unit display_version,
+      " " ^ s_ "Display version number and exit";
   ] in
-  let anon_fun str = raise (Arg.Bad (str ^ ": unknown parameter")) in
-  let usage_msg = "virt-top : a 'top'-like utility for virtualization
+  let anon_fun str =
+    raise (Arg.Bad (sprintf (f_ "%s: unknown parameter") str)) in
+  let usage_msg = s_ "virt-top : a 'top'-like utility for virtualization
 
 SUMMARY
   virt-top [-options]
@@ -202,7 +235,7 @@ OPTIONS" in
       | _, "end-time", t -> set_end_time t
       | _, "overwrite-init-file", "false" -> no_init_file ()
       | lineno, key, _ ->
-         eprintf "%s:%d: configuration item ``%s'' ignored\n%!"
+         eprintf (f_ "%s:%d: configuration item ``%s'' ignored\n%!")
            filename lineno key
     ) config
   in
@@ -210,7 +243,7 @@ OPTIONS" in
    | NoInitFile -> ()
    | DefaultInitFile ->
        let home = try Sys.getenv "HOME" with Not_found -> "/" in
-       let filename = home // ".virt-toprc" in
+       let filename = home // rcfile in
        try_to_read_init_file filename
    | InitFile filename ->
        try_to_read_init_file filename
@@ -227,7 +260,7 @@ OPTIONS" in
        prerr_endline (Libvirt.Virterror.to_string err);
        (* If non-root and no explicit connection URI, print a warning. *)
        if Unix.geteuid () <> 0 && name = None then (
-         print_endline "NB: If you want to monitor a local Xen hypervisor, you usually need to be root";
+         print_endline (s_ "NB: If you want to monitor a local Xen hypervisor, you usually need to be root");
        );
        exit 1 in
 
@@ -269,13 +302,13 @@ OPTIONS" in
 
   (* Curses voodoo (see ncurses(3)). *)
   if not !script_mode then (
-    initscr ();
-    cbreak ();
-    noecho ();
+    ignore (initscr ());
+    ignore (cbreak ());
+    ignore (noecho ());
     nonl ();
     let stdscr = stdscr () in
-    intrflush stdscr false;
-    keypad stdscr true;
+    ignore (intrflush stdscr false);
+    ignore (keypad stdscr true);
     ()
   );
 
@@ -309,10 +342,10 @@ let usleep n =
  * This helper function also enables echo temporarily.
  *)
 let get_string maxlen =
-  echo ();
+  ignore (echo ());
   let str = String.create maxlen in
   let ok = getstr str in (* Safe because binding calls getnstr. *)
-  noecho ();
+  ignore (noecho ());
   if not ok then ""
   else (
     (* Chop at first '\0'. *)
@@ -331,8 +364,8 @@ let header_lineno = 4
 let domains_lineno = 5
 
 (* Print in the "message area". *)
-let clear_msg () = move message_lineno 0; clrtoeol ()
-let print_msg str = clear_msg (); mvaddstr message_lineno 0 str; ()
+let clear_msg () = ignore (move message_lineno 0); clrtoeol ()
+let print_msg str = clear_msg (); ignore (mvaddstr message_lineno 0 str)
 
 (* Intermediate "domain + stats" structure that we use to collect
  * everything we know about a domain within the collect function.
@@ -718,12 +751,13 @@ let redraw =
     let lines, cols = get_size () in
 
     (* Time. *)
-    mvaddstr top_lineno 0 ("virt-top " ^ printable_time ^ " - ");
+    ignore (mvaddstr top_lineno 0 ("virt-top " ^ printable_time ^ " - "));
 
     (* Basic node_info. *)
-    addstr (sprintf "%s %d/%dCPU %dMHz %LdMB "
-             node_info.C.model node_info.C.cpus nr_pcpus node_info.C.mhz
-             (node_info.C.memory /^ 1024L));
+    ignore (addstr
+             (sprintf "%s %d/%dCPU %dMHz %LdMB "
+                node_info.C.model node_info.C.cpus nr_pcpus node_info.C.mhz
+                (node_info.C.memory /^ 1024L)));
     (* Save the cursor position for when we come to draw the
      * historical CPU times (down in this function).
      *)
@@ -804,8 +838,9 @@ let redraw =
 
         (* Print domains. *)
         attron A.reverse;
-        mvaddstr header_lineno 0
-          (pad cols "   ID S RDRQ WRRQ RXBY TXBY %CPU %MEM    TIME   NAME");
+        ignore (
+          mvaddstr header_lineno 0
+            (pad cols "   ID S RDRQ WRRQ RXBY TXBY %CPU %MEM    TIME   NAME"));
         attroff A.reverse;
 
         let rec loop lineno = function
@@ -828,7 +863,7 @@ let redraw =
                   rd.rd_domid state rd_req wr_req rx_bytes tx_bytes
                   percent_cpu percent_mem time name in
                 let line = pad cols line in
-                mvaddstr lineno 0 line;
+                ignore (mvaddstr lineno 0 line);
                 loop (lineno+1) doms
               )
           | (name, Inactive) :: doms -> (* inactive domain *)
@@ -838,7 +873,7 @@ let redraw =
                     "    -                                           (%s)"
                     name in
                 let line = pad cols line in
-                mvaddstr lineno 0 line;
+                ignore (mvaddstr lineno 0 line);
                 loop (lineno+1) doms
               )
         in
@@ -861,16 +896,17 @@ let redraw =
             ) doms
           ) in
         attron A.reverse;
-        mvaddstr header_lineno 0 (pad cols ("PHYCPU %CPU " ^ dom_names));
+        ignore (
+          mvaddstr header_lineno 0 (pad cols ("PHYCPU %CPU " ^ dom_names)));
         attroff A.reverse;
 
         Array.iteri (
           fun p row ->
-            mvaddstr (p+domains_lineno) 0 (sprintf "%4d   " p);
+            ignore (mvaddstr (p+domains_lineno) 0 (sprintf "%4d   " p));
             let cpu_time = pcpus_cpu_time.(p) in (* ns used on this CPU *)
             let percent_cpu = 100. *. cpu_time /. total_cpu_per_pcpu in
-            addstr (Show.percent percent_cpu);
-            addch 32;
+            ignore (addstr (Show.percent percent_cpu));
+            ignore (addch 32);
 
             List.iteri (
               fun di (domid, name, _, _, _, _, _) ->
@@ -886,7 +922,7 @@ let redraw =
                       (if is_average then '=' else ' ')
                       (if is_running then '#' else ' ')
                   ) in
-                addstr (pad width str);
+                ignore (addstr (pad width str));
                 ()
             ) doms
         ) pcpus;
@@ -960,8 +996,9 @@ let redraw =
 
         (* Print the header for network devices. *)
         attron A.reverse;
-        mvaddstr header_lineno 0
-          (pad cols "   ID S RXBY TXBY RXPK TXPK DOMAIN       INTERFACE");
+        ignore (
+          mvaddstr header_lineno 0
+            (pad cols "   ID S RXBY TXBY RXPK TXPK DOMAIN       INTERFACE"));
         attroff A.reverse;
 
         (* Print domains and devices. *)
@@ -993,7 +1030,7 @@ let redraw =
                   rx_packets tx_packets
                   (pad 12 name) dev in
                 let line = pad cols line in
-                mvaddstr lineno 0 line;
+                ignore (mvaddstr lineno 0 line);
                 loop (lineno+1) devs
               )
         in
@@ -1068,8 +1105,9 @@ let redraw =
 
         (* Print the header for block devices. *)
         attron A.reverse;
-        mvaddstr header_lineno 0
-          (pad cols "   ID S RDBY WRBY RDRQ WRRQ DOMAIN       DEVICE");
+        ignore (
+          mvaddstr header_lineno 0
+            (pad cols "   ID S RDBY WRBY RDRQ WRRQ DOMAIN       DEVICE"));
         attroff A.reverse;
 
         (* Print domains and devices. *)
@@ -1101,7 +1139,7 @@ let redraw =
                   rd_req wr_req
                   (pad 12 name) dev in
                 let line = pad cols line in
-                mvaddstr lineno 0 line;
+                ignore (mvaddstr lineno 0 line);
                 loop (lineno+1) devs
               )
         in
@@ -1112,16 +1150,18 @@ let redraw =
         crashed, active, inactive,
         total_cpu_time, total_memory, total_domU_memory) = totals in
 
-    mvaddstr summary_lineno 0
-      (sprintf "%d domains, %d active, %d running, %d sleeping, %d paused, %d inactive D:%d O:%d X:%d"
-        count active running blocked paused inactive shutdown shutoff
-        crashed);
+    ignore (
+      mvaddstr summary_lineno 0
+       (sprintf (f_ "%d domains, %d active, %d running, %d sleeping, %d paused, %d inactive D:%d O:%d X:%d")
+          count active running blocked paused inactive shutdown shutoff
+          crashed));
 
     (* Total %CPU used, and memory summary. *)
     let percent_cpu = 100. *. total_cpu_time /. total_cpu in
-    mvaddstr (summary_lineno+1) 0
-      (sprintf "CPU: %2.1f%%  Mem: %Ld MB (%Ld MB by guests)"
-        percent_cpu (total_memory /^ 1024L) (total_domU_memory /^ 1024L));
+    ignore (
+      mvaddstr (summary_lineno+1) 0
+       (sprintf (f_ "CPU: %2.1f%%  Mem: %Ld MB (%Ld MB by guests)")
+          percent_cpu (total_memory /^ 1024L) (total_domU_memory /^ 1024L)));
 
     (* Time to grab another historical %CPU for the list? *)
     if time >= !historical_cpu_last_time +. float !historical_cpu_delay
@@ -1138,11 +1178,12 @@ let redraw =
        String.concat " "
          (List.map (sprintf "%2.1f%%") !historical_cpu) in
       let line = pad maxwidth line in
-      mvaddstr y x line;
+      ignore (mvaddstr y x line);
       () in
 
-    move message_lineno 0; (* Park cursor in message area, as with top. *)
-    refresh ();                   (* Refresh the display. *)
+    (* Park cursor in message area, as with top. *)
+    ignore (move message_lineno 0);
+    ignore (refresh ());               (* Refresh the display. *)
     ()
 
 (* Write CSV header row. *)
@@ -1151,7 +1192,9 @@ let write_csv_header () =
     [ "Hostname"; "Time"; "Arch"; "Physical CPUs";
       "Count"; "Running"; "Blocked"; "Paused"; "Shutdown";
       "Shutoff"; "Crashed"; "Active"; "Inactive";
-      "%CPU"; "Total memory (KB)"; "Total guest memory (KB)";
+      "%CPU";
+      "Total hardware memory (KB)";
+      "Total memory (KB)"; "Total guest memory (KB)";
       "Total CPU time (ns)" ] @
       (* These fields are repeated for each domain: *)
     [ "Domain ID"; "Domain name"; ] @
@@ -1182,6 +1225,7 @@ let append_csv
     string_of_int paused; string_of_int shutdown; string_of_int shutoff;
     string_of_int crashed; string_of_int active; string_of_int inactive;
     sprintf "%2.1f" percent_cpu;
+    Int64.to_string node_info.C.memory;
     Int64.to_string total_memory; Int64.to_string total_domU_memory;
     Int64.to_string (Int64.of_float total_cpu_time)
   ] in
@@ -1287,32 +1331,33 @@ and get_key_press setup =
   )
 
 and change_delay () =
-  print_msg (sprintf "Change delay from %.1f to: " (float !delay /. 1000.));
+  print_msg
+    (sprintf (f_ "Change delay from %.1f to: ") (float !delay /. 1000.));
   let str = get_string 16 in
   (* Try to parse the number. *)
   let error =
     try
       let newdelay = float_of_string str in
       if newdelay <= 0. then (
-       print_msg "Delay must be > 0"; true
+       print_msg (s_ "Delay must be > 0"); true
       ) else (
        delay := int_of_float (newdelay *. 1000.); false
       )
     with
       Failure "float_of_string" ->
-       print_msg "Not a valid number"; true in
-  refresh ();
+       print_msg (s_ "Not a valid number"); true in
+  ignore (refresh ());
   sleep (if error then 2 else 1)
 
 and change_sort_order () =
   clear ();
   let lines, cols = get_size () in
 
-  mvaddstr top_lineno 0 "Set sort order for main display";
-  mvaddstr summary_lineno 0 "Type key or use up and down cursor keys.";
+  ignore (mvaddstr top_lineno 0 (s_ "Set sort order for main display"));
+  ignore (mvaddstr summary_lineno 0 (s_ "Type key or use up and down cursor keys."));
 
   attron A.reverse;
-  mvaddstr header_lineno 0 (pad cols "KEY   Sort field");
+  ignore (mvaddstr header_lineno 0 (pad cols "KEY   Sort field"));
   attroff A.reverse;
 
   let accelerator_key = function
@@ -1340,15 +1385,15 @@ and change_sort_order () =
     fun i ord ->
       let selected = !sort_order = ord in
       if selected then selected_index := i;
-      mvaddstr (domains_lineno+i) 0
-       (sprintf "  %c %s %s %s"
-          (key_of_int i) (if selected then "*" else " ")
-          (printable_sort_order ord)
-          (accelerator_key ord))
+      ignore (mvaddstr (domains_lineno+i) 0
+               (sprintf "  %c %s %s %s"
+                  (key_of_int i) (if selected then "*" else " ")
+                  (printable_sort_order ord)
+                  (accelerator_key ord)))
   ) all_sort_fields;
 
-  move message_lineno 0;
-  refresh ();
+  ignore (move message_lineno 0);
+  ignore (refresh ());
   let k = getch () in
   if k >= 0 && k <> 32 && k <> Char.code 'q' && k <> 13 then (
     let new_order, loop =
@@ -1392,7 +1437,7 @@ and change_sort_order () =
         print_msg (sprintf "Sort order changed to: %s"
                      (printable_sort_order new_order));
         if not loop then (
-          refresh ();
+          ignore (refresh ());
           sleep 1
         )
     );
@@ -1433,7 +1478,7 @@ and write_init_file () =
   | NoInitFile -> ()                   (* Do nothing if --no-init-file *)
   | DefaultInitFile ->
       let home = try Sys.getenv "HOME" with Not_found -> "/" in
-      let filename = home // ".virt-toprc" in
+      let filename = home // rcfile in
       _write_init_file filename
   | InitFile filename ->
       _write_init_file filename
@@ -1458,8 +1503,8 @@ and _write_init_file filename =
 
     let fp = fprintf in
     let nl () = fp chan "\n" in
-    fp chan "# .virt-toprc virt-top configuration file\n";
-    fp chan "# generated on %s by %s\n" printable_date_time username;
+    let () = fp chan (f_ "# %s virt-top configuration file\n") rcfile in
+    let () = fp chan (f_ "# generated on %s by %s\n") printable_date_time username in
     nl ();
     fp chan "display %s\n" (cli_of_display !display_mode);
     fp chan "delay %g\n" (float !delay /. 1000.);
@@ -1473,13 +1518,13 @@ and _write_init_file filename =
     if !batch_mode = true then fp chan "batch true\n";
     if !secure_mode = true then fp chan "secure true\n";
     nl ();
-    fp chan "# To send debug and error messages to a file, uncomment next line\n";
+    output_string chan (s_ "# To send debug and error messages to a file, uncomment next line\n");
     fp chan "#debug virt-top.out\n";
     nl ();
-    fp chan "# Enable CSV output to the named file\n";
+    output_string chan (s_ "# Enable CSV output to the named file\n");
     fp chan "#csv virt-top.csv\n";
     nl ();
-    fp chan "# To protect this file from being overwritten, uncomment next line\n";
+    output_string chan (s_ "# To protect this file from being overwritten, uncomment next line\n");
     fp chan "#overwrite-init-file false\n";
 
     close_out chan;
@@ -1491,14 +1536,16 @@ and _write_init_file filename =
     (* Rename filename.new to filename. *)
     Unix.rename (filename ^ ".new") filename;
 
-    print_msg (sprintf "Wrote settings to %s" filename);
-    refresh ();
+    print_msg (sprintf (f_ "Wrote settings to %s") filename);
+    ignore (refresh ());
     sleep 2
   with
-  | Sys_error err -> print_msg "Error: %s"; refresh (); sleep 2
+  | Sys_error err ->
+      ignore (print_msg (s_ "Error" ^ ": " ^ err));
+      ignore (refresh ()); sleep 2
   | Unix.Unix_error (err, fn, str) ->
-      print_msg (sprintf "Error: %s %s %s" (Unix.error_message err) fn str);
-      refresh ();
+      ignore (print_msg (s_ ("Error" ^ ": " ^ Unix.error_message err ^ fn ^ str)));
+      ignore (refresh ());
       sleep 2
 
 and show_help (_, _, _, _, _, hostname,
@@ -1510,29 +1557,31 @@ and show_help (_, _, _, _, _, hostname,
 
   (* Banner at the top of the screen. *)
   let banner =
-    sprintf "virt-top %s (libvirt %d.%d.%d) by Red Hat"
-      Libvirt_version.version libvirt_major libvirt_minor libvirt_release in
+    sprintf (f_ "virt-top %s ocaml-libvirt %s libvirt %d.%d.%d by Red Hat")
+      Virt_top_version.version
+      Libvirt_version.version
+      libvirt_major libvirt_minor libvirt_release in
   let banner = pad cols banner in
   attron A.reverse;
-  mvaddstr 0 0 banner;
+  ignore (mvaddstr 0 0 banner);
   attroff A.reverse;
 
   (* Status. *)
-  mvaddstr 1 0
-    (sprintf "Delay: %.1f secs; Batch: %s; Secure: %s; Sort: %s"
-       (float !delay /. 1000.)
-       (if !batch_mode then "On" else "Off")
-       (if !secure_mode then "On" else "Off")
-       (printable_sort_order !sort_order));
-  mvaddstr 2 0
-    (sprintf "Connect: %s; Hostname: %s"
-       (match !uri with None -> "default" | Some s -> s)
-       hostname);
+  ignore (mvaddstr 1 0
+           (sprintf (f_ "Delay: %.1f secs; Batch: %s; Secure: %s; Sort: %s")
+              (float !delay /. 1000.)
+              (if !batch_mode then "On" else "Off")
+              (if !secure_mode then "On" else "Off")
+              (printable_sort_order !sort_order)));
+  ignore (mvaddstr 2 0
+           (sprintf (f_ "Connect: %s; Hostname: %s")
+              (match !uri with None -> "default" | Some s -> s)
+              hostname));
 
   (* Misc keys on left. *)
-  let banner = pad 38 "MAIN KEYS" in
+  let banner = pad 38 (s_ "MAIN KEYS") in
   attron A.reverse;
-  mvaddstr header_lineno 1 banner;
+  ignore (mvaddstr header_lineno 1 banner);
   attroff A.reverse;
 
   let get_lineno =
@@ -1541,31 +1590,31 @@ and show_help (_, _, _, _, _, hostname,
   in
   let key keys description =
     let lineno = get_lineno () in
-    move lineno 1; attron A.bold; addstr keys; attroff A.bold;
-    move lineno 10; addstr description; ()
+    ignore (move lineno 1); attron A.bold; ignore (addstr keys); attroff A.bold;
+    ignore (move lineno 10); ignore (addstr description)
   in
-  key "space ^L" "Update display";
-  key "q"        "Quit";
-  key "d s"      "Set update interval";
-  key "h"        "Help";
+  key "space ^L" (s_ "Update display");
+  key "q"        (s_ "Quit");
+  key "d s"      (s_ "Set update interval");
+  key "h"        (s_ "Help");
 
   (* Sort order. *)
   ignore (get_lineno ());
-  let banner = pad 38 "SORTING" in
+  let banner = pad 38 (s_ "SORTING") in
   attron A.reverse;
-  mvaddstr (get_lineno ()) 1 banner;
+  ignore (mvaddstr (get_lineno ()) 1 banner);
   attroff A.reverse;
 
-  key "P" "Sort by %CPU";
-  key "M" "Sort by %MEM";
-  key "T" "Sort by TIME";
-  key "N" "Sort by ID";
-  key "F" "Select sort field";
+  key "P" (s_ "Sort by %CPU");
+  key "M" (s_ "Sort by %MEM");
+  key "T" (s_ "Sort by TIME");
+  key "N" (s_ "Sort by ID");
+  key "F" (s_ "Select sort field");
 
   (* Display modes on right. *)
-  let banner = pad 39 "DISPLAY MODES" in
+  let banner = pad 39 (s_ "DISPLAY MODES") in
   attron A.reverse;
-  mvaddstr header_lineno 40 banner;
+  ignore (mvaddstr header_lineno 40 banner);
   attroff A.reverse;
 
   let get_lineno =
@@ -1574,21 +1623,21 @@ and show_help (_, _, _, _, _, hostname,
   in
   let key keys description =
     let lineno = get_lineno () in
-    move lineno 40; attron A.bold; addstr keys; attroff A.bold;
-    move lineno 49; addstr description; ()
+    ignore (move lineno 40); attron A.bold; ignore (addstr keys); attroff A.bold;
+    ignore (move lineno 49); ignore (addstr description)
   in
-  key "0" "Domains display";
-  key "1" "Toggle physical CPUs";
-  key "2" "Toggle network interfaces";
-  key "3" "Toggle block devices";
+  key "0" (s_ "Domains display");
+  key "1" (s_ "Toggle physical CPUs");
+  key "2" (s_ "Toggle network interfaces");
+  key "3" (s_ "Toggle block devices");
 
   (* Update screen and wait for key press. *)
-  mvaddstr (lines-1) 0
-    "More help in virt-top(1) man page. Press any key to return.";
-  refresh ();
+  ignore (mvaddstr (lines-1) 0
+           (s_ "More help in virt-top(1) man page. Press any key to return."));
+  ignore (refresh ());
   ignore (getch ())
 
 and unknown_command k =
-  print_msg "Unknown command - try 'h' for help";
-  refresh ();
+  print_msg (s_ "Unknown command - try 'h' for help");
+  ignore (refresh ());
   sleep 1