From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from sog-mx-1.v43.ch3.sourceforge.com ([172.29.43.191] helo=mx.sourceforge.net) by sfs-ml-1.v29.ch3.sourceforge.com with esmtp (Exim 4.76) (envelope-from ) id 1Vo1i7-0002Gi-Co for bitcoin-development@lists.sourceforge.net; Wed, 04 Dec 2013 01:57:07 +0000 Received-SPF: pass (sog-mx-1.v43.ch3.sourceforge.com: domain of gmail.com designates 209.85.192.175 as permitted sender) client-ip=209.85.192.175; envelope-from=wtogami@gmail.com; helo=mail-pd0-f175.google.com; Received: from mail-pd0-f175.google.com ([209.85.192.175]) by sog-mx-1.v43.ch3.sourceforge.com with esmtps (TLSv1:RC4-SHA:128) (Exim 4.76) id 1Vo1i6-0004sT-3I for bitcoin-development@lists.sourceforge.net; Wed, 04 Dec 2013 01:57:07 +0000 Received: by mail-pd0-f175.google.com with SMTP id w10so21162849pde.6 for ; Tue, 03 Dec 2013 17:57:00 -0800 (PST) MIME-Version: 1.0 X-Received: by 10.68.232.37 with SMTP id tl5mr42549531pbc.86.1386122220236; Tue, 03 Dec 2013 17:57:00 -0800 (PST) Received: by 10.66.158.129 with HTTP; Tue, 3 Dec 2013 17:57:00 -0800 (PST) In-Reply-To: References: Date: Tue, 3 Dec 2013 15:57:00 -1000 Message-ID: From: "Warren Togami Jr." To: Bitcoin Dev Content-Type: multipart/alternative; boundary=047d7b3395e58d77c004ecabbaa7 X-Spam-Score: -0.6 (/) X-Spam-Report: Spam Filtering performed by mx.sourceforge.net. See http://spamassassin.org/tag/ for more details. -0.0 RCVD_IN_DNSWL_NONE RBL: Sender listed at http://www.dnswl.org/, no trust [209.85.192.175 listed in list.dnswl.org] 0.0 URIBL_BLOCKED ADMINISTRATOR NOTICE: The query to URIBL was blocked. See http://wiki.apache.org/spamassassin/DnsBlocklists#dnsbl-block for more information. [URIs: github.com] -1.5 SPF_CHECK_PASS SPF reports sender host as permitted sender for sender-domain 0.0 FREEMAIL_FROM Sender email is commonly abused enduser mail provider (wtogami[at]gmail.com) -0.0 SPF_PASS SPF: sender matches SPF record 1.0 HTML_MESSAGE BODY: HTML included in message -0.1 DKIM_VALID_AU Message has a valid DKIM or DK signature from author's domain 0.1 DKIM_SIGNED Message has a DKIM or DK signature, not necessarily valid -0.1 DKIM_VALID Message has at least one valid DKIM or DK signature X-Headers-End: 1Vo1i6-0004sT-3I Subject: Re: [Bitcoin-development] Coin Control, Send crash on MacOS X X-BeenThere: bitcoin-development@lists.sourceforge.net X-Mailman-Version: 2.1.9 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Wed, 04 Dec 2013 01:57:07 -0000 --047d7b3395e58d77c004ecabbaa7 Content-Type: text/plain; charset=UTF-8 On Sun, Dec 1, 2013 at 1:19 AM, Wladimir wrote: > On Sun, Dec 1, 2013 at 12:05 PM, Warren Togami Jr. wrote: > >> https://github.com/litecoin-project/bitcoinomg/commits/0.8.5-OMG6 >> http://download1.rpmfusion.org/~warren/bitcoin-0.8.5-OMG6/ >> I've been backporting patches from master and Litecoin to make a Bitcoin >> 0.8 client with more features. It works quite well on Linux and Win32. >> >> http://pastebin.com/g8QqheGc >> Today we discovered a rare crash that can happen on MacOS X. toffoo and >> coblee reproduced it on MacOS X 10.9 and I reproduced it on 10.6.8. It >> seems to be some kind of race condition involving SendCoinsEntry::clear(). >> >> >> 1. 11 QtGui 0x00e28141 >> QWidget::setFocus(Qt::FocusReason) + 289 >> 2. 12 org.bitcoinfoundation.Bitcoin-Qt 0x002ca665 >> SendCoinsEntry::clear() + 101 >> >> > I don't think the setFocus should be in clear() in the first place. It > conflates clearing the widgets and changing the focus. > > If the automatic focus change is desirable at all it could be moved to a > seperate function "focusPayTo". > > In any case it's just a nicety and should just be removed if it causes > problems. > > Wladimir > > Did as you suggested, removed both setFocus() calls that happen after Send is clicked http://pastebin.com/j4adDpsM Now it crashes in something else within qt. I'm trying other things... Warren --047d7b3395e58d77c004ecabbaa7 Content-Type: text/html; charset=UTF-8 Content-Transfer-Encoding: quoted-printable
On S= un, Dec 1, 2013 at 1:19 AM, Wladimir <laanwj@gmail.com> wrote= :
On Sun, Dec 1, 2013 at 12:05 PM, Warren = Togami Jr. <wtogami@gmail.com> wrote:
https://github.com/l= itecoin-project/bitcoinomg/commits/0.8.5-OMG6
http://download1.rpmfusion.org/~warren/bitcoin-0.8.5-OMG6/
I've been backporting patches from master and Litecoin to make a B= itcoin 0.8 client with more features. =C2=A0It works quite well on Linux an= d Win32.

Today we discovered a rare crash that can happen on MacOS X. tof= foo and coblee reproduced it on MacOS X 10.9 and I reproduced it on 10.6.8.= It seems to be some kind of race condition involving=C2=A0SendCoinsEntry::= clear().

  1. 11 =C2=A0QtGui =C2=A0 =C2=A0 =C2=A0 =C2= =A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 = =C2=A0 0x00e28141 QWidget::setFocus(Qt::FocusReason) + 289
  2. 12 =C2=A0org.bitcoinfoundation.Bit= coin-Qt=C2=A0 =C2=A0 0x002ca665 SendCoinsEntry::clear() + 101

I = don't think the setFocus should be in clear() in the first place. It co= nflates clearing the widgets and changing the focus.

If the automatic focus change is desirable at all it could be moved to a se= perate function "focusPayTo".

In any cas= e it's just a nicety and should just be removed if it causes problems.<= /div>

Wladimir

=

Did as you suggeste= d, removed both setFocus() calls that happen after Send is clicked

http://pastebin.com/j4adDpsM
Now it crashes in something else within qt.

I'= m trying other things...

Warren

--047d7b3395e58d77c004ecabbaa7--