[Qemu-commits] [qemu/qemu] f61edd: vl.c: Replace fprintf(stderr) with error_report()

[GitHub]

  Branch: refs/heads/master
  Home:   https://github.com/qemu/qemu
  Commit: f61eddcb2bb5cbbdd1d911b7e937db9affc29028
      
https://github.com/qemu/qemu/commit/f61eddcb2bb5cbbdd1d911b7e937db9affc29028
  Author: Eduardo Habkost <address@hidden>
  Date:   2015-11-03 (Tue, 03 Nov 2015)

  Changed paths:
    M vl.c

  Log Message:
  -----------
  vl.c: Replace fprintf(stderr) with error_report()

Straightforward replacement, except for qemu_kill_report(), which
printed a common part of its error message first, then the applicable
special part.  Print each complete message with a single
error_report() instead.

Multi-line messages were replaced by error_report() followed by
error_printf().

The following changes were made to the error messages:

* The "invalid date format" message was reworded to better fit
  the new error_report()+error_printf() pattern.
* On the remaining messages, only the trailing newlines, "qemu:" and
  "error:" message prefixes were removed.

Signed-off-by: Eduardo Habkost <address@hidden>
Message-Id: <address@hidden>
[Squashed in
Message-Id: <address@hidden>
and updated commit message]
Signed-off-by: Markus Armbruster <address@hidden>


  Commit: 8afb900030b93122a40ef4a636d02ba888bdce12
      
https://github.com/qemu/qemu/commit/8afb900030b93122a40ef4a636d02ba888bdce12
  Author: Eduardo Habkost <address@hidden>
  Date:   2015-11-03 (Tue, 03 Nov 2015)

  Changed paths:
    M vl.c

  Log Message:
  -----------
  vl.c: Remove periods and exclamation points from error messages

Except for removing periods and exclamation points, no other changes
were made to the error messages (yet).

Suggested-by: Markus Armbruster <address@hidden>
Signed-off-by: Eduardo Habkost <address@hidden>
Message-Id: <address@hidden>
Signed-off-by: Markus Armbruster <address@hidden>


  Commit: eb177ec1843476f278a63d936b0f73f456a86024
      
https://github.com/qemu/qemu/commit/eb177ec1843476f278a63d936b0f73f456a86024
  Author: Eduardo Habkost <address@hidden>
  Date:   2015-11-03 (Tue, 03 Nov 2015)

  Changed paths:
    M vl.c

  Log Message:
  -----------
  vl.c: Use "warning:" prefix consistently on warnings

Suggested-by: Andrew Jones <address@hidden>
Signed-off-by: Eduardo Habkost <address@hidden>
Message-Id: <address@hidden>
Signed-off-by: Markus Armbruster <address@hidden>


  Commit: 3e5153732e96d870b3272e662234d664c15e3815
      
https://github.com/qemu/qemu/commit/3e5153732e96d870b3272e662234d664c15e3815
  Author: Eduardo Habkost <address@hidden>
  Date:   2015-11-03 (Tue, 03 Nov 2015)

  Changed paths:
    M vl.c

  Log Message:
  -----------
  vl.c: Remove unnecessary uppercase in error messages

Suggested-by: Andrew Jones <address@hidden>
Signed-off-by: Eduardo Habkost <address@hidden>
Message-Id: <address@hidden>
Signed-off-by: Markus Armbruster <address@hidden>


  Commit: 4cd70f34afa817cfc891f6c64fd7e277ba561784
      
https://github.com/qemu/qemu/commit/4cd70f34afa817cfc891f6c64fd7e277ba561784
  Author: Eduardo Habkost <address@hidden>
  Date:   2015-11-03 (Tue, 03 Nov 2015)

  Changed paths:
    M vl.c

  Log Message:
  -----------
  vl.c: Touch up error messages

Several small improvements:

* Use "cannot" instead of "can not"
* Use 'quotes' instead of `quotes'
* Change "fail to parse" error message to "failed to parse"

Suggested-by: Eric Blake <address@hidden>
Suggested-by: Andrew Jones <address@hidden>
Signed-off-by: Eduardo Habkost <address@hidden>
Message-Id: <address@hidden>
[Squashed in
Message-Id: <address@hidden>
Message-Id: <address@hidden>
and updated commit message]
Signed-off-by: Markus Armbruster <address@hidden>


  Commit: 515cb8ef27874822c64bc81ccde9bd7227383137
      
https://github.com/qemu/qemu/commit/515cb8ef27874822c64bc81ccde9bd7227383137
  Author: Eduardo Habkost <address@hidden>
  Date:   2015-11-03 (Tue, 03 Nov 2015)

  Changed paths:
    M vl.c

  Log Message:
  -----------
  vl.c: Improve warnings on use of deprecated options

Simplify warnings about deprecated options by rewriting them as
"warning: ignoring deprecated option".

Reword -no-kvm-pit-reinjection deprecation warning.

Suggested-by: Andrew Jones <address@hidden>
Signed-off-by: Eduardo Habkost <address@hidden>
Message-Id: <address@hidden>
[Squashed in
Message-Id: <address@hidden>
and updated commit message]
Signed-off-by: Markus Armbruster <address@hidden>


  Commit: 5dfdae814307f7580d56ab4505288224751e0801
      
https://github.com/qemu/qemu/commit/5dfdae814307f7580d56ab4505288224751e0801
  Author: Eduardo Habkost <address@hidden>
  Date:   2015-11-03 (Tue, 03 Nov 2015)

  Changed paths:
    M vl.c

  Log Message:
  -----------
  vl.c: Use "%s support is disabled" error messages consistently

Signed-off-by: Eduardo Habkost <address@hidden>
Message-Id: <address@hidden>
Signed-off-by: Markus Armbruster <address@hidden>


  Commit: 2b5a79f1d961f07332cf77f38cdf9dc2fc7e2b64
      
https://github.com/qemu/qemu/commit/2b5a79f1d961f07332cf77f38cdf9dc2fc7e2b64
  Author: Peter Maydell <address@hidden>
  Date:   2015-11-04 (Wed, 04 Nov 2015)

  Changed paths:
    M vl.c

  Log Message:
  -----------
  Merge remote-tracking branch 'remotes/armbru/tags/pull-error-2015-11-03' into 
staging

vl.c: Error message rework

# gpg: Signature made Tue 03 Nov 2015 08:40:50 GMT using RSA key ID EB918653
# gpg: Good signature from "Markus Armbruster <address@hidden>"
# gpg:                 aka "Markus Armbruster <address@hidden>"

* remotes/armbru/tags/pull-error-2015-11-03:
  vl.c: Use "%s support is disabled" error messages consistently
  vl.c: Improve warnings on use of deprecated options
  vl.c: Touch up error messages
  vl.c: Remove unnecessary uppercase in error messages
  vl.c: Use "warning:" prefix consistently on warnings
  vl.c: Remove periods and exclamation points from error messages
  vl.c: Replace fprintf(stderr) with error_report()

Signed-off-by: Peter Maydell <address@hidden>


Compare: https://github.com/qemu/qemu/compare/79cf9fad341e...2b5a79f1d961