aMule Bug Tracker - aMule
View Issue Details
0000057aMuleMiscpublic2004-05-30 23:372004-08-02 15:47
Hetfield 
Xaignar 
normaltweakalways
resolvedfixed 
 
 
0000057: tray speed control
If I set my DownloadCapacity to 3 and my UploadCapacity to 3,
then my ConnectionLimits are lost at aMule-Restart.

I read through the code and found two lines in SysTray.cpp
that are involved in this (line 154-155):

        if (theApp.glob_prefs->GetMaxGraphDownloadRate() <
theApp.glob_prefs->GetMaxDownload())
theApp.glob_prefs->SetDownloadlimit(UNLIMITED);
        if (theApp.glob_prefs->GetMaxGraphUploadRate() <
theApp.glob_prefs->GetMaxUpload())
theApp.glob_prefs->SetUploadlimit(UNLIMITED);

Shouldn't LineCapacity and SpeedLimit not interfere
each other (reasons are explained in the sourcecode yet)?!
In fact they do at this point.

My suggestion as replacement for these two lines, something
like:

        if (theApp.glob_prefs->GetMaxGraphDownloadRate() <
theApp.glob_prefs->GetMaxDownload())
theApp.glob_prefs->SetDownloadlimit(theApp.glob_prefs->GetMaxGraphDownloadRate());
        if (theApp.glob_prefs->GetMaxGraphUploadRate() <
theApp.glob_prefs->GetMaxUpload())
theApp.glob_prefs->SetUploadlimit(theApp.glob_prefs->GetMaxGraphUploadRate() );

reported by Andreas Brandmaier <andy@brandmaier.de>
the problem is that when i made the tray i couldn't access lots of functions.
now things are changed.
as u can see speed code is cut & paste from old prefs code.

please make the tray call th unified prefs code and not a cut&paste.
i can't do for time reason actually
No tags attached.
Issue History
2004-05-30 23:37HetfieldNew Issue
2004-08-02 15:47XaignarStatusnew => resolved
2004-08-02 15:47XaignarResolutionopen => fixed
2004-08-02 15:47XaignarAssigned To => Xaignar
2004-08-02 15:47XaignarNote Added: 0000186

Notes
(0000186)
Xaignar   
2004-08-02 15:47   
Fixed. We now call the PrefsUnifiedDlg function.