qemu-commits
[Top][All Lists]
Advanced
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

[Qemu-commits] [qemu/qemu] 1c884a: include/qemu/host-utils.h: Trivial ty

From: GitHub
Subject: [Qemu-commits] [qemu/qemu] 1c884a: include/qemu/host-utils.h: Trivial typo: ctz->cto
Date: Tue, 04 Mar 2014 12:30:04 -0800 
  Branch: refs/heads/master
  Home:   https://github.com/qemu/qemu
  Commit: 1c884abeded1fee3680a3f1db655bf70d3b49e97
      
https://github.com/qemu/qemu/commit/1c884abeded1fee3680a3f1db655bf70d3b49e97
  Author: Dr. David Alan Gilbert <address@hidden>
  Date:   2014-03-02 (Sun, 02 Mar 2014)

  Changed paths:
    M include/qemu/host-utils.h

  Log Message:
  -----------
  include/qemu/host-utils.h: Trivial typo: ctz->cto

Signed-off-by: Dr. David Alan Gilbert <address@hidden>
Signed-off-by: Michael Tokarev <address@hidden>


  Commit: de580dafade551cf67607f114ff9b5eeb881cc42
      
https://github.com/qemu/qemu/commit/de580dafade551cf67607f114ff9b5eeb881cc42
  Author: Hani Benhabiles <address@hidden>
  Date:   2014-03-02 (Sun, 02 Mar 2014)

  Changed paths:
    M qmp.c

  Log Message:
  -----------
  object: Report type in error when not user creatable.

Signed-off-by: Hani Benhabiles <address@hidden>
Reviewed-by: Hu Tao <address@hidden>
Acked-by: Paolo Bonzini <address@hidden>
Signed-off-by: Michael Tokarev <address@hidden>


  Commit: 0064aceb29faea018dfb7cf0bca5d9644272a295
      
https://github.com/qemu/qemu/commit/0064aceb29faea018dfb7cf0bca5d9644272a295
  Author: Stefan Weil <address@hidden>
  Date:   2014-03-02 (Sun, 02 Mar 2014)

  Changed paths:
    M include/qemu/bswap.h

  Log Message:
  -----------
  bswap: Modify prototype of stb_p (avoid type conversions)

The function uses an uint8_t value, so show this in the function
prototype. Non-optimizing compilers will avoid unnecessary type
conversions from (u)int8_t to int and back to uint8_t when generating
calls of this inline function.

stw_p, stl_p and stq_p already use similar prototypes.

Signed-off-by: Stefan Weil <address@hidden>
Reviewed-by: Laszlo Ersek <address@hidden>
Signed-off-by: Michael Tokarev <address@hidden>


  Commit: 55e7c29e46f845f503fda3e8cac3f5e0e8b3d984
      
https://github.com/qemu/qemu/commit/55e7c29e46f845f503fda3e8cac3f5e0e8b3d984
  Author: Stefan Weil <address@hidden>
  Date:   2014-03-02 (Sun, 02 Mar 2014)

  Changed paths:
    M include/qemu/bswap.h

  Log Message:
  -----------
  bswap: Modify prototypes of st[wl]_{le, be}_p (avoid type conversions)

The functions use uint16_t or uint32_t values, so show this in the function
prototypes. Non-optimizing compilers will avoid unnecessary type
conversions when generating calls of these inline functions.

stq_le_p, stq_be_p already use similar prototypes.

Signed-off-by: Stefan Weil <address@hidden>
Reviewed-by: Laszlo Ersek <address@hidden>
Signed-off-by: Michael Tokarev <address@hidden>


  Commit: c5d3c49896582bb338e19149b23d3613510059fb
      
https://github.com/qemu/qemu/commit/c5d3c49896582bb338e19149b23d3613510059fb
  Author: Stefan Weil <address@hidden>
  Date:   2014-03-02 (Sun, 02 Mar 2014)

  Changed paths:
    M tcg/tcg.c

  Log Message:
  -----------
  tcg: Fix typo in comment (dependancies -> dependencies)

Signed-off-by: Stefan Weil <address@hidden>
Signed-off-by: Michael Tokarev <address@hidden>


  Commit: 8ead6018837f13f1ef4a40c3eb5054512c160577
      
https://github.com/qemu/qemu/commit/8ead6018837f13f1ef4a40c3eb5054512c160577
  Author: Stefan Weil <address@hidden>
  Date:   2014-03-02 (Sun, 02 Mar 2014)

  Changed paths:
    M stubs/gdbstub.c

  Log Message:
  -----------
  stubs: Optimize dependencies for gdbstub.c

It does not need qemu-common.h. Including exec/gdbstub.h fixes a warning
from static code analyzers and avoids mismatching declarations for
xml_builtin.

Signed-off-by: Stefan Weil <address@hidden>
Signed-off-by: Michael Tokarev <address@hidden>


  Commit: 7edd9ddc979f112287ff5184590830c67ea991bf
      
https://github.com/qemu/qemu/commit/7edd9ddc979f112287ff5184590830c67ea991bf
  Author: Peter Maydell <address@hidden>
  Date:   2014-03-02 (Sun, 02 Mar 2014)

  Changed paths:
    M tests/test-int128.c

  Log Message:
  -----------
  tests/test-int128: Don't use __noclone__ attribute on clang

clang doesn't support the __noclone__ attribute and emits a warning about
it. Fortunately clang also implements a mechanism for asking if a particular
attribute is implemented; use it. We assume that if the compiler doesn't
support __has_attribute() then it must be GCC and must support __noclone__.

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


  Commit: aa830cdc28edb69c1fe81c8fd9471ab288ad0926
      
https://github.com/qemu/qemu/commit/aa830cdc28edb69c1fe81c8fd9471ab288ad0926
  Author: Peter Maydell <address@hidden>
  Date:   2014-03-02 (Sun, 02 Mar 2014)

  Changed paths:
    M include/qapi/qmp/qjson.h

  Log Message:
  -----------
  qjson.h: Remove spurious GCC_FMT_ATTR markup from qobject_from_json() 
declaration

The function qobject_from_json() doesn't actually allow its
argument to be a format string -- it passes a NULL va_list*
to qobject_from_jsonv(), and the parser code will then never
actually interpret %-escape sequences (it tests whether the
va_list pointer is NULL and will stop with a parse error).

The spurious attribute markup causes clang warnings in some
of the test cases where we programmatically construct JSON
to feed to qobject_from_json():

 tests/test-qmp-input-visitor.c:76:35: warning: format string is not a
 string literal (potentially insecure) [-Wformat-security]
    data->obj = qobject_from_json(json_string);
                            ^~~~~~~~~~~

Remove the incorrect attribute.

Signed-off-by: Peter Maydell <address@hidden>
Suggested-by: Paolo Bonzini <address@hidden>
Reviewed-by: Eric Blake <address@hidden>
Reviewed-by: Andreas Färber <address@hidden>
Signed-off-by: Michael Tokarev <address@hidden>


  Commit: cc99c6f5ff3fb2acfa52764a539e129cada104e4
      
https://github.com/qemu/qemu/commit/cc99c6f5ff3fb2acfa52764a539e129cada104e4
  Author: Stefan Weil <address@hidden>
  Date:   2014-03-02 (Sun, 02 Mar 2014)

  Changed paths:
    M util/iov.c

  Log Message:
  -----------
  util/iov: Use qemu/sockets.h instead of conditional code

Signed-off-by: Stefan Weil <address@hidden>
Reviewed-by: Paolo Bonzini <address@hidden>
Reviewed-by: Andreas Färber <address@hidden>
Signed-off-by: Michael Tokarev <address@hidden>


  Commit: 777872e5c65dabb1d80b8c13427c6a3e58dafe40
      
https://github.com/qemu/qemu/commit/777872e5c65dabb1d80b8c13427c6a3e58dafe40
  Author: Stefan Weil <address@hidden>
  Date:   2014-03-02 (Sun, 02 Mar 2014)

  Changed paths:
    M exec.c

  Log Message:
  -----------
  exec: Remove unneeded include files

This file does not depend on windows.h.

Signed-off-by: Stefan Weil <address@hidden>
Reviewed-by: Paolo Bonzini <address@hidden>
Reviewed-by: Andreas Färber <address@hidden>
Signed-off-by: Michael Tokarev <address@hidden>


  Commit: 67d065c3db489882dce1502246d5fcca10396ea4
      
https://github.com/qemu/qemu/commit/67d065c3db489882dce1502246d5fcca10396ea4
  Author: Stefan Weil <address@hidden>
  Date:   2014-03-02 (Sun, 02 Mar 2014)

  Changed paths:
    M qemu-img.c

  Log Message:
  -----------
  qemu-img: Remove unneeded include files

There is no dependency on windows.h, and the standard include files are
already included by qemu-common.h.

Signed-off-by: Stefan Weil <address@hidden>
Reviewed-by: Paolo Bonzini <address@hidden>
Reviewed-by: Andreas Färber <address@hidden>
Signed-off-by: Michael Tokarev <address@hidden>


  Commit: cae8a9289b9ee874bc8711c2629037661061b9c1
      
https://github.com/qemu/qemu/commit/cae8a9289b9ee874bc8711c2629037661061b9c1
  Author: Stefan Weil <address@hidden>
  Date:   2014-03-02 (Sun, 02 Mar 2014)

  Changed paths:
    M qga/main.c

  Log Message:
  -----------
  qga: Remove unneeded include file

This file does not depend on windows.h.

Signed-off-by: Stefan Weil <address@hidden>
Reviewed-by: Paolo Bonzini <address@hidden>
Reviewed-by: Andreas Färber <address@hidden>
Signed-off-by: Michael Tokarev <address@hidden>


  Commit: 25254a7191cdaa6cc10edb7bdbbfd0a6d02b9905
      
https://github.com/qemu/qemu/commit/25254a7191cdaa6cc10edb7bdbbfd0a6d02b9905
  Author: Stefan Weil <address@hidden>
  Date:   2014-03-02 (Sun, 02 Mar 2014)

  Changed paths:
    M vl.c

  Log Message:
  -----------
  vl: Remove unneeded include file

This file does not depend on windows.h.

Cc: Anthony Liguori <address@hidden>
Signed-off-by: Stefan Weil <address@hidden>
Reviewed-by: Paolo Bonzini <address@hidden>
Reviewed-by: Andreas Färber <address@hidden>
Signed-off-by: Michael Tokarev <address@hidden>


  Commit: ac458e121c398c8619b2a7735e87e37b73c0b687
      
https://github.com/qemu/qemu/commit/ac458e121c398c8619b2a7735e87e37b73c0b687
  Author: Peter Maydell <address@hidden>
  Date:   2014-03-04 (Tue, 04 Mar 2014)

  Changed paths:
    M exec.c
    M include/qapi/qmp/qjson.h
    M include/qemu/bswap.h
    M include/qemu/host-utils.h
    M qemu-img.c
    M qga/main.c
    M qmp.c
    M stubs/gdbstub.c
    M tcg/tcg.c
    M tests/test-int128.c
    M util/iov.c
    M vl.c

  Log Message:
  -----------
  Merge remote-tracking branch 'remotes/mjt/tags/trivial-patches-2014-03-04' 
into staging

trivial patches for 2014-03-04

# gpg: Signature made Tue 04 Mar 2014 06:13:56 GMT using RSA key ID 74F0C838
# gpg: Good signature from "Michael Tokarev <address@hidden>"
# gpg:                 aka "Michael Tokarev <address@hidden>"
# gpg:                 aka "Michael Tokarev <address@hidden>"
# gpg: WARNING: This key is not certified with a trusted signature!
# gpg:          There is no indication that the signature belongs to the owner.
# Primary key fingerprint: 6EE1 95D1 886E 8FFB 810D  4324 457C E0A0 8044 65C5
#      Subkey fingerprint: E190 8639 3B10 B51B AC2C  8B73 5253 C5AD 74F0 C838

* remotes/mjt/tags/trivial-patches-2014-03-04:
  vl: Remove unneeded include file
  qga: Remove unneeded include file
  qemu-img: Remove unneeded include files
  exec: Remove unneeded include files
  util/iov: Use qemu/sockets.h instead of conditional code
  qjson.h: Remove spurious GCC_FMT_ATTR markup from qobject_from_json() 
declaration
  tests/test-int128: Don't use __noclone__ attribute on clang
  stubs: Optimize dependencies for gdbstub.c
  tcg: Fix typo in comment (dependancies -> dependencies)
  bswap: Modify prototypes of st[wl]_{le, be}_p (avoid type conversions)
  bswap: Modify prototype of stb_p (avoid type conversions)
  object: Report type in error when not user creatable.
  include/qemu/host-utils.h: Trivial typo: ctz->cto

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


  Commit: f55ea6297cc0224fe4934b90ff5343b620b14669
      
https://github.com/qemu/qemu/commit/f55ea6297cc0224fe4934b90ff5343b620b14669
  Author: Andreas Färber <address@hidden>
  Date:   2014-03-04 (Tue, 04 Mar 2014)

  Changed paths:
    M block/gluster.c

  Log Message:
  -----------
  block/gluster: Add missing argument to qemu_gluster_init() call

Commit adccfbcd6020e928db93b2b4faf0dbd05ffbe016 (block: gluster - add
reopen support.) did not supply the qemu_gluster_init() Error **
argument, needed since commit a7451cb850d115f257080aff3fbc54f255ebf8f7
(gluster: correctly propagate errors).

Pass through qemu_gluster_reopen_prepare()'s errp, as done in
qemu_gluster_open().

Cc: Stefan Hajnoczi <address@hidden>
Signed-off-by: Andreas Färber <address@hidden>
Reviewed-by: Stefan Hajnoczi <address@hidden>
Signed-off-by: Peter Maydell <address@hidden>


Compare: https://github.com/qemu/qemu/compare/e00ef747f08d...f55ea6297cc0
reply via email to
[Prev in Thread] Current Thread [Next in Thread]