The attached patch :
From 1dcbc61b812837bed0262d503de27d987fcb92e0 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Thorbj=C3=B8rn=20Lindeijer?= <thorbj...@lindeijer.nl>
Date: Thu, 25 Feb 2010 22:52:54 +0100
Subject: [PATCH] Fix the X11 paste code to actually return true on success
Reviewed-by: Jared Adams
---
src/utils/copynpaste.cpp | 2 ++
1 files changed, 2 insertions(+), 0 deletions(-)
diff --git a/src/utils/copynpaste.cpp b/src/utils/copynpaste.cpp
index 9292fb0..31aa7bf 100644
--- a/src/utils/copynpaste.cpp
+++ b/src/utils/copynpaste.cpp
@@ -308,6 +308,8 @@ bool RetrieveBuffer(std::string& text,
std::string::size_type& pos)
text.insert(pos, data);
pos += strlen(data);
XFree(data);
+
+ return true;
}
}
return false;
--
1.6.6.1
Le 25/02/2010 23:40, Thorbjørn Lindeijer a écrit :
> Dear Wormux Team,
>
> As a developer of the The Mana World (now called the Mana client),
> I've borrowed your paste code. When using it though, I noticed there
> was a small bug in the return code on X11.
>
> The attached patch fixes that, I suppose it will still apply to your
> copynpaste.cpp file since I didn't make any other modifications
> compared to your current trunk.
>
> If you want to see the commits to the Mana client that borrow this
> code, you can see it here:
>
> http://gitorious.org/mana/mana/commit/d1f7e4f5090ac61142415edaa51f2af86aecc188
> http://gitorious.org/mana/mana/commit/01152e00c2099e4f3d2029413718b48d501b153c
> http://gitorious.org/mana/mana/commit/34ce2c58ecfe9bfb04688af321e68f80236d104d
>
> Best regards,
> Bjørn
>
>
>
> _______________________________________________
> Wormux-dev mailing list
> Wormux-dev@gna.org
> https://mail.gna.org/listinfo/wormux-dev
_______________________________________________
Wormux-dev mailing list
Wormux-dev@gna.org
https://mail.gna.org/listinfo/wormux-dev
