Darrell,
Can we increase this default to about 10M? 1M is for 10 years ago :)
Can we increase this default to about 10M? 1M is for 10 years ago :)
Mmmmmaybe.
I found the spot in the code and created this test patch:
http://humanreadable.nfshost.com/trinity/patches/tdebase-konqueror-minimum-p...
I built and tested the patch. Please test because I'm not a C++ guru. :-)
The patch suffices as a quick fix for those who want to change the default size, but with respect to changing a default value in the sources, we probably should have some kind of nominal vote. I realize many people these days are on some kind of broadband connection and also have reasonably decent hardware. Is 10 MB a reasonable new default number? I don't know. What are other desktop environments defaulting to?
Darrell
Can we increase this default to about 10M? 1M is for 10 years ago :)
Mmmmmaybe.
I found the spot in the code and created this test patch:
http://humanreadable.nfshost.com/trinity/patches/tdebase-konqueror-minimum-p...
I built and tested the patch. Please test because I'm not a C++ guru. :-)
The patch suffices as a quick fix for those who want to change the default size, but with respect to changing a default value in the sources, we probably should have some kind of nominal vote. I realize many people these days are on some kind of broadband connection and also have reasonably decent hardware. Is 10 MB a reasonable new default number? I don't know. What are other desktop environments defaulting to?
David, please test this patch.
Darrell
On 06/08/2012 11:53 PM, Darrell Anderson wrote:
David, please test this patch.
Darrell
OK - will report back -- working on it now. Had to get #1 Son from camp this AM.
On 06/09/2012 03:31 PM, David C. Rankin wrote:
OK - will report back -- working on it now. Had to get #1 Son from camp this AM.
Darrell,
Rebuilt tdebase with todays sources to test. Lots of stuff not happy. I will have to do a complete rebuild. I suspect with all the commits in the past 4-5 days a rebuild of everything is in order. The patch applied fine and tdebase built without issue, but I can't confirm the default is 10, but I have no reason to think it wouldn't be. Will report back after I rebuild everything.
Rebuilt tdebase with todays sources to test. Lots of stuff not happy. I will have to do a complete rebuild. I suspect with all the commits in the past 4-5 days a rebuild of everything is in order. The patch applied fine and tdebase built without issue, but I can't confirm the default is 10, but I have no reason to think it wouldn't be. Will report back after I rebuild everything.
Yes, almost everything needs to be rebuilt. I don't believe TQt3 needs rebuilding. :-)
Computer-wise I have been doing little else the past few days but rebuilding. This was such a massive patch upload that I would feel awful if anything major broke. With each phase of my push I rebuilt everything. Hours and hours each time. Everything is working here. I just hope everything works for everybody else too.
Regarding the patch, temporarily create a new profile. Then you'll see the new 10MB number rather than the old 1MB. You won't see the default value with an existing profile. If you like the patch then jump up and down and make a lot of noise here in the list to obtain some sort of agreement to push the patch. :-)
Darrell
-
Darrell Anderson -
David C. Rankin