Bitcoin Forum
May 27, 2024, 01:44:54 AM *
News: Latest Bitcoin Core release: 27.0 [Torrent]
 
   Home   Help Search Login Register More  
Pages: [1]
  Print  
Author Topic: [PULL] add 'settxfee' RPC  (Read 1543 times)
xf2_org (OP)
Member
**
Offline Offline

Activity: 98
Merit: 13


View Profile
April 12, 2011, 10:55:36 PM
 #1


URL: https://github.com/bitcoin/bitcoin/pull/150

Add 'settxfee' RPC, to change TX fee setting at runtime

This clamps TX fee, if not zero, between "network minimum" (0.01) and 5 BTC (IMO sanity limit).

Presumably, the "network minimum" might be lowered, as is occasionally discussed on IRC and forum. This RPC prevents setting TX fee below network minimum.
Luke-Jr
Legendary
*
expert
Offline Offline

Activity: 2576
Merit: 1186



View Profile
April 12, 2011, 11:56:53 PM
 #2

This introduces a regression, since -settxfee accepts <0.01 right now, and it's sane to do so.

xf2_org (OP)
Member
**
Offline Offline

Activity: 98
Merit: 13


View Profile
April 13, 2011, 12:14:53 AM
 #3

This introduces a regression, since -settxfee accepts <0.01 right now, and it's sane to do so.

That's intentional, not a regression.

j16sdiz
Newbie
*
Offline Offline

Activity: 37
Merit: 0


View Profile
April 13, 2011, 12:38:24 AM
 #4

Remove the    if (nAmount < CENT) and   if (nAmount > (5 * COIN))  check.
API should be as powerful as (if not more powerful then) command line.
Luke-Jr
Legendary
*
expert
Offline Offline

Activity: 2576
Merit: 1186



View Profile
April 13, 2011, 01:48:34 AM
 #5

This introduces a regression, since -settxfee accepts <0.01 right now, and it's sane to do so.
That's intentional, not a regression.
It's a regression by definition, because it works today and your patch breaks it.

Gavin Andresen
Legendary
*
qt
Offline Offline

Activity: 1652
Merit: 2217


Chief Scientist


View Profile WWW
April 14, 2011, 01:54:09 AM
 #6

I can see the GUI not allowing a less-than-CENT to save fat-fingered users from themselves, but I think the RPC should allow it; at the very least it makes it possible for a kind of grass-roots movement to arise between miners and people generating transactions so if we start seeing a lot of transactions with less-than-CENT-per-kbyte-fees then that's a really good indication the default definition of "free" needs to change.

And now that the RPC and GUI allow full-precision amounts for send/move, there is no problem with "I got 50.000001 bitcoins from mining, how do I send ALL of them?"

How often do you get the chance to work on a potentially world-changing project?
xf2_org (OP)
Member
**
Offline Offline

Activity: 98
Merit: 13


View Profile
April 14, 2011, 04:57:25 AM
 #7

Updated pull request, removing the ValidFee() check.

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!