[Qemu-devel] [RFC PATCH 27/32] os-posix: Drop misleading comment

Markus Armbruster posted 32 patches 8 years, 4 months ago
[Qemu-devel] [RFC PATCH 27/32] os-posix: Drop misleading comment
Posted by Markus Armbruster 8 years, 4 months ago
Comments explaining why we include a header tend to go bad.  This
one's almost comical: not only doesn't qemu-options.hx use
MAP_POPULATE anymore (since commit ef36fa1, v2.0.0, 2013), even the
include it applies to got moved away in commit 02d0e09 (v2.7.0).

Signed-off-by: Markus Armbruster <armbru@redhat.com>
---
 os-posix.c | 1 -
 1 file changed, 1 deletion(-)

diff --git a/os-posix.c b/os-posix.c
index 92e9d85215..b9c2343b1e 100644
--- a/os-posix.c
+++ b/os-posix.c
@@ -25,7 +25,6 @@
 
 #include "qemu/osdep.h"
 #include <sys/wait.h>
-/*needed for MAP_POPULATE before including qemu-options.h */
 #include <pwd.h>
 #include <grp.h>
 #include <libgen.h>
-- 
2.13.6


Re: [Qemu-devel] [RFC PATCH 27/32] os-posix: Drop misleading comment
Posted by Marc-André Lureau 8 years, 4 months ago
CC qemu-trivial

On Mon, Oct 2, 2017 at 5:25 PM, Markus Armbruster <armbru@redhat.com> wrote:
> Comments explaining why we include a header tend to go bad.  This
> one's almost comical: not only doesn't qemu-options.hx use
> MAP_POPULATE anymore (since commit ef36fa1, v2.0.0, 2013), even the
> include it applies to got moved away in commit 02d0e09 (v2.7.0).
>
> Signed-off-by: Markus Armbruster <armbru@redhat.com>

Reviewed-by: Marc-André Lureau <marcandre.lureau@redhat.com>


> ---
>  os-posix.c | 1 -
>  1 file changed, 1 deletion(-)
>
> diff --git a/os-posix.c b/os-posix.c
> index 92e9d85215..b9c2343b1e 100644
> --- a/os-posix.c
> +++ b/os-posix.c
> @@ -25,7 +25,6 @@
>
>  #include "qemu/osdep.h"
>  #include <sys/wait.h>
> -/*needed for MAP_POPULATE before including qemu-options.h */
>  #include <pwd.h>
>  #include <grp.h>
>  #include <libgen.h>
> --
> 2.13.6
>
>



-- 
Marc-André Lureau