Bitcoin Forum
May 10, 2024, 10:13:19 PM *
News: Latest Bitcoin Core release: 27.0 [Torrent]
 
   Home   Help Search Login Register More  
Pages: [1]
  Print  
Author Topic: [PATCH] (Mostly) Working backport to wx2.8  (Read 962 times)
Matt Corallo (OP)
Hero Member
*****
expert
Offline Offline

Activity: 755
Merit: 515


View Profile
May 24, 2011, 02:02:38 AM
 #1

I decided to actually look at what is required to backport to wx2.8.  Actually, its mostly copy/paste/sed crap. 
Its done in the worst possible way and is ridiculously inefficient, I just wanted to see if it could be done.
As it turns out, it seems completely possible.
The GUI shows up and the majority seems to work.  Translations appear to work, but that is only in very brief testing.
I'm sure I've made 100 or more typos, so I'd like some corrections (and pointing out of much better ways of converting crap between wxString and std::string, if someone can find one).
You can grab the version at https://github.com/TheBlueMatt/bitcoin/tree/wx2.8

Bitcoin Core, rust-lightning, http://bitcoinfibre.org etc.
PGP ID: 07DF 3E57 A548 CCFB 7530  7091 89BB B866 3E2E65CE
1715379199
Hero Member
*
Offline Offline

Posts: 1715379199

View Profile Personal Message (Offline)

Ignore
1715379199
Reply with quote  #2

1715379199
Report to moderator
1715379199
Hero Member
*
Offline Offline

Posts: 1715379199

View Profile Personal Message (Offline)

Ignore
1715379199
Reply with quote  #2

1715379199
Report to moderator
1715379199
Hero Member
*
Offline Offline

Posts: 1715379199

View Profile Personal Message (Offline)

Ignore
1715379199
Reply with quote  #2

1715379199
Report to moderator
It is a common myth that Bitcoin is ruled by a majority of miners. This is not true. Bitcoin miners "vote" on the ordering of transactions, but that's all they do. They can't vote to change the network rules.
Advertised sites are not endorsed by the Bitcoin Forum. They may be unsafe, untrustworthy, or illegal in your jurisdiction.
1715379199
Hero Member
*
Offline Offline

Posts: 1715379199

View Profile Personal Message (Offline)

Ignore
1715379199
Reply with quote  #2

1715379199
Report to moderator
Pieter Wuille
Legendary
*
qt
Offline Offline

Activity: 1072
Merit: 1174


View Profile WWW
May 24, 2011, 09:09:11 AM
 #2

Very nice! It doesn't seem to work completely yet, as I get weird character in the status column and status bar below. Apart from that, really good to see we don't need a dev branch of wxwindows.

One suggestion: is it possible to define _ yourself, so all the GetTranslationChar calls aren't necessary?

I do Bitcoin stuff.
Matt Corallo (OP)
Hero Member
*****
expert
Offline Offline

Activity: 755
Merit: 515


View Profile
May 24, 2011, 11:29:20 AM
 #3

Very nice! It doesn't seem to work completely yet, as I get weird character in the status column and status bar below. Apart from that, really good to see we don't need a dev branch of wxwindows.
Yea, that showed up in a couple places because I didn't handle temporary pointers right, I think I've fixed the majority of those now.
One suggestion: is it possible to define _ yourself, so all the GetTranslationChar calls aren't necessary?
Not really, _ has to return 3 different types (wxString, std::string, char*) and you can't override based on return value (AFAIK, but I may very well be wrong).  wx2.9 will convert properly automatically, but not with 2.8.

Bitcoin Core, rust-lightning, http://bitcoinfibre.org etc.
PGP ID: 07DF 3E57 A548 CCFB 7530  7091 89BB B866 3E2E65CE
Pages: [1]
  Print  
 
Jump to:  

Powered by MySQL Powered by PHP Powered by SMF 1.1.19 | SMF © 2006-2009, Simple Machines Valid XHTML 1.0! Valid CSS!