Patchwork vnc: fix option misspelling ("non-adapative" -> "non-adaptive")

login
register
mail settings
Submitter Catalin Patulea
Date Nov. 10, 2012, 12:01 a.m.
Message ID <1352505686-12733-1-git-send-email-catalinp@google.com>
Download mbox | patch
Permalink /patch/198163/
State New
Headers show

Comments

Catalin Patulea - Nov. 10, 2012, 12:01 a.m.
Signed-off-by: Catalin Patulea <catalinp@google.com>
---
D'oh, that was really stupid. Here it is with the correct length.

 ui/vnc.c |    2 +-
 1 files changed, 1 insertions(+), 1 deletions(-)
Peter Maydell - Nov. 10, 2012, 12:19 a.m.
On 10 November 2012 01:01, Catalin Patulea <catalinp@google.com> wrote:
>
> Signed-off-by: Catalin Patulea <catalinp@google.com>
> ---
> D'oh, that was really stupid. Here it is with the correct length.
>
>  ui/vnc.c |    2 +-
>  1 files changed, 1 insertions(+), 1 deletions(-)
>
> diff --git a/ui/vnc.c b/ui/vnc.c
> index 61f120e..ba30362 100644
> --- a/ui/vnc.c
> +++ b/ui/vnc.c
> @@ -2945,7 +2945,7 @@ void vnc_display_open(DisplayState *ds, const char *display, Error **errp)
>  #endif
>          } else if (strncmp(options, "lossy", 5) == 0) {
>              vs->lossy = true;
> -        } else if (strncmp(options, "non-adapative", 13) == 0) {
> +        } else if (strncmp(options, "non-adaptive", 12) == 0) {
>              vs->non_adaptive = true;
>          } else if (strncmp(options, "share=", 6) == 0) {
>              if (strncmp(options+6, "ignore", 6) == 0) {
> --
> 1.7.7.3
>

Reviewed-by: Peter Maydell <peter.maydell@linaro.org>

If we're gonna do this I guess it should go into 1.3...
cc'ing Anthony.

-- PMM
Catalin Patulea - Nov. 20, 2012, 9:06 a.m.
On Fri, Nov 9, 2012 at 7:19 PM, Peter Maydell <peter.maydell@linaro.org> wrote:
> If we're gonna do this I guess it should go into 1.3...
> cc'ing Anthony.
Friendly ping for trivial patch :)
Stefan Hajnoczi - Nov. 20, 2012, 1:07 p.m.
On Tue, Nov 20, 2012 at 04:06:31AM -0500, Catalin Patulea wrote:
> On Fri, Nov 9, 2012 at 7:19 PM, Peter Maydell <peter.maydell@linaro.org> wrote:
> > If we're gonna do this I guess it should go into 1.3...
> > cc'ing Anthony.
> Friendly ping for trivial patch :)

Hi Catalin,
In the future feel free to email qemu-trivial@nongnu.org and I'll make
sure trival patches get merged.

For the 1.3-rc cycle this patch can go directly through Anthony.

Stefan
Peter Maydell - Nov. 20, 2012, 1:11 p.m.
On 20 November 2012 13:07, Stefan Hajnoczi <stefanha@gmail.com> wrote:
> On Tue, Nov 20, 2012 at 04:06:31AM -0500, Catalin Patulea wrote:
>> On Fri, Nov 9, 2012 at 7:19 PM, Peter Maydell <peter.maydell@linaro.org> wrote:
>> > If we're gonna do this I guess it should go into 1.3...
>> > cc'ing Anthony.
>> Friendly ping for trivial patch :)
>
> Hi Catalin,
> In the future feel free to email qemu-trivial@nongnu.org and I'll make
> sure trival patches get merged.
>
> For the 1.3-rc cycle this patch can go directly through Anthony.

Yes, I cc'd Anthony rather than -trivial when I reviewed this
patch because I felt we were far enough into the freeze/release
process that it shouldn't go via trivial.

-- PMM

Patch

diff --git a/ui/vnc.c b/ui/vnc.c
index 61f120e..ba30362 100644
--- a/ui/vnc.c
+++ b/ui/vnc.c
@@ -2945,7 +2945,7 @@  void vnc_display_open(DisplayState *ds, const char *display, Error **errp)
 #endif
         } else if (strncmp(options, "lossy", 5) == 0) {
             vs->lossy = true;
-        } else if (strncmp(options, "non-adapative", 13) == 0) {
+        } else if (strncmp(options, "non-adaptive", 12) == 0) {
             vs->non_adaptive = true;
         } else if (strncmp(options, "share=", 6) == 0) {
             if (strncmp(options+6, "ignore", 6) == 0) {