bugmldonkey, a multi-networks file-sharing client - Bugs: bug #9817, Friends filelist is reasked too...

 
 

You are not allowed to post comments on this tracker with your current authentication level.

bug #9817: Friends filelist is reasked too often

Submitted by:  spiralvoice <spiralvoice>
Submitted on:  Fri 30 Jul 2004 02:56:17 AM UTC  
 
Category: eDonkey-PluginSeverity: 4 - Important
Item Group: Program malfunctionStatus: Wont Fix
Assigned to: NoneOpen/Closed: Closed
Release: Release: 2-5-16
Operating System: NoneBinaries Origin: CVS / Self compiled
CPU type: None

Thu 19 May 2005 02:05:09 PM UTC, comment #4:

Problem was not present in 2.5.30.15

spiralvoice <spiralvoice>
Project Administrator
Tue 10 Aug 2004 08:40:30 AM UTC, comment #3:

client_last_filereqs is not used anymore, so it won't even be saved in friends.ini in 2.5.27. Indeed, client_next_view_files could be saved, but from my point of view, it is better to always have a view of your friends files, and not wait for 6 hours after a restart if you had just asked before the stop. Anyway, I will try to reproduce the bug at home tonight.

mldonkey admin <mldonkey>
Project Administrator
Mon 09 Aug 2004 03:39:00 PM UTC, comment #2:

Please have a look at donkeyClient.ml, in line 650 this should happen:
c.client_last_filereqs <- last_time ();

client_last_filereqs is saved in friends.ini, but is IMHO always zero. Correct me if I am wrong but I think a whole code segment is commented out in donkeyClient.ml, from line 676-710, so client_last_filereqs is never filled with a value.

The other part dealing with that is in donkeyComplexOptions.ml, and also commented out:

(* Is it really useful ? I don't think so. It is only used when restarting
the client several times, which is not that current...

(try
let last_filereqs =
(min (get_value "client_last_filereqs" value_to_float)
(BasicSocket.last_time ())) in
l.client_last_filereqs <- last_filereqs
with _ -> ()); *)

spiralvoice <spiralvoice>
Project Administrator
Mon 09 Aug 2004 02:46:53 PM UTC, comment #1:

I will try to reproduce this bug, but I'm not completely convinced by the explanation: friends are supposed to be kept in a list corresponding to the option, and thus, the "client" structure is never removed, even when converted to source and should be used when converted back from source, so that the field client_next_view_files should be kept from one connection to the other.

mldonkey admin <mldonkey>
Project Administrator
Fri 30 Jul 2004 02:56:17 AM UTC, original submission:

Core does not look at our self-proposed time limit of six hours.
Pango wrote about it:
"the problem was that client_next_view_files was discarded when the client was converted to source, so the timer couldn't be remembered between connections.
So fixing it will have a memory cost (adding that timer to sources), hopefully acceptable."
More information about it:
http://mldonkey.berlios.de/modules.php?name=Forums&file=viewtopic&t=2689

spiralvoice <spiralvoice>
Project Administrator

 

No files currently attached

 

Depends on the following items: None found

Items that depend on this one: None found

 

CC list is empty

 

Do you think this task is very important?
If so, you can click here to add your encouragement to it.
This task has 0 encouragements so far.

Only logged-in users can vote.

 

Please enter the title of George Orwell's famous dystopian book (it's a date):

 

 

Follow 2 latest changes.

Date Changed By Updated Field Previous Value => Replaced By
Thu 19 May 2005 02:05:09 PM UTCspiralvoiceStatusNone=>Wont Fix
  Open/ClosedOpen=>Closed

Back to the top


Powered by Savane 3.1-cleanup1