Hi,

In the course turning down some verbosity in the diags to check out a
problem with the bitzi code I did this:

diff -u -b -r1.3 dmesh.c
--- src/core/dmesh.c    6 Oct 2004 05:12:18 -0000       1.3
+++ src/core/dmesh.c    18 Oct 2004 23:21:06 -0000
@@ -1,4 +1,4 @@
-/*
+/* -*- mode: cc-mode; tab-width:4; -*-
  * $Id: dmesh.c,v 1.3 2004/10/06 05:12:18 cbiere Exp $
  *
  * Copyright (c) 2002-2003, Raphael Manfredi
@@ -801,9 +801,9 @@
        dme->url.idx = idx;
        dme->url.name = atom_str_get(name);

-    if (dbg)
-               printf("dmesh entry created, name %p: %s\n",
-                               dme->url.name, dme->url.name);
+       if (dbg>2)
+               printf("dmesh entry created, count %d, name %p: %s\n",
+                               dm->count, dme->url.name,
dme->url.name);

        /*
         * The entries are sorted by time.  We're going to
unconditionally add
@@ -827,6 +827,14 @@
        } else
                dm->count++;

+       if (dbg && dme==NULL)
+       {
+           if (file_info_active(sha1)!=NULL)
+               g_warning("dmesh entry for active file thrown away: name
%p: %s",
+                       sha1, sha1_base32(sha1));
+       }
+
+

And I saw a load of these:

04/10/19 00:23:05 (WARNING): dmesh entry for active file thrown away:
name 0x83cf1b4: FRS.....

I double checked the SHA1 and sure enough they match active downloads on
my system. Now the dmesh code only actually tries to swarm with a new
entry is successfully added to the mesh which fails if the mesh has more
than a fixed number of entries.

My questions are these:

* Should the MAX_ENTRIES be ignored for files that we are actively
downloading?

* Is the oldest timestamp a suitable test for goodness of the dmesh
entry? Could we be throwing away "good" entries for potentially more
upto date entries that still may not be currently available?

--  Alex, homepage: http://www.bennee.com/~alex/ "It is better to have
tried and failed than to have failed to try, but the result's the same."
- Mike Dennison 



-------------------------------------------------------
This SF.net email is sponsored by: IT Product Guide on ITManagersJournal
Use IT products in your business? Tell us what you think of them. Give us
Your Opinions, Get Free ThinkGeek Gift Certificates! Click to find out more
http://productguide.itmanagersjournal.com/guidepromo.tmpl
_______________________________________________
Gtk-gnutella-devel mailing list
[EMAIL PROTECTED]
https://lists.sourceforge.net/lists/listinfo/gtk-gnutella-devel

Reply via email to