Displaying dupes plzzz

PI · Post by PI » 2004-02-12 10:34

Why not displaying dupes which are removed from my sharing list ?

BSOD2600 · Post by **BSOD2600** » 2004-02-13 03:13

Why do you want to see them? What useful purpose would it serve? The only thing I would find useful is some notification somewhere that says how many dups it found and removed.

PI · Post by PI » 2004-02-13 08:41

I've lot of files, and displaying dupes is a good idea to tidy my drives

And I think it's not difficult to add this function into DC++ because removing the dupes => filenames are known somewhere, so adding it to a List is not difficult , isn't it ?
I don't like having same files in differents directories, I don't want to have a "fake" share, even if the size is substracted ... the files already exist ...
please understand me :-)

I've probably made some mistakes. I'm still learning English ...

gemmell1 · Post by **gemmell1** » 2004-02-13 10:39

BSOD2600 wrote:Why do you want to see them? What useful purpose would it serve? The only thing I would find useful is some notification somewhere that says how many dups it found and removed.

I think that is what he means, he wants DC++ to tell him where the files are so he can manually remove them. I don't think he wants to show them in his share.

PI · Post by PI » 2004-02-13 11:04

yes it's that ! :lol:

Gasman1015 · Post by **Gasman1015** » 2004-02-13 18:22

Then use a file manager. Which DC++ is not. :roll:

PI · Post by PI » 2004-02-13 19:29

GRRRRR It's easy to add this function, all the base is done ...

Gasman1015 · Post by **Gasman1015** » 2004-02-13 19:59

If it's so easy, supply a patch.

Twink · Post by **Twink** » 2004-02-13 23:45

I'd agree that there doesn't seem to be alot of code needed for this, except where would it be displayed? a new window? a new log file?

PI · Post by PI » 2004-02-14 00:01

I think display dupes files list in a new window will be the best (window accessible by clicking on a button placed in sharing tab in settings , dupes files list refreshed when global list is generated)

Post by **GargoyleMT** » 2004-02-14 23:41

There are some changes in 0.307 that make it a little easier to add "system log"-like messages. I'll code this and submit it with one of my normal patches to arne.

(In the future, please don't use "easy" in a request for a feature... Essentially calling someone lazy [whether rightfully or wrongfully] is a poor way to motivate them to do something for you for free.)

Wisp · Post by **Wisp** » 2004-02-18 09:05

would be a bad idea to automatically remove dupes

let's give some examples..

you have a game, like a car racing game, you shared the whole install directory like it appears on the install-cd

the cd has multiple directories with wave files which are needed during the game, there is a possibility that a wave file appears more than one time in different directories, when dc++ deletes all the dupes, the game can be corruptes because it misses files

another example is when you have a lot of complete mp3 albums, but you also made some personal compilations.. the chance is that dc++ first find the mp3 in the compilation folder, and then deletes the same mp3 on all the seperate albums the mp3 appears

that way you get a lot of incomplete albums (which is ver annoying for a user when he finds out there is a missing mp3 in the album while the user he downloaded from is already offline)

TheParanoidOne · Post by **TheParanoidOne** » 2004-02-18 10:44

Not true.

Settings --> Advanced --> Remove dupes completely from your share (otherwise only their size is subtracted, but the files can be seen by others)

That incredibly verbose option seems to cover what you are talking about Wisp.

TheParanoidOne · Post by **TheParanoidOne** » 2004-02-18 10:46

GargoyleMT wrote:There are some changes in 0.307 that make it a little easier to add "system log"-like messages. I'll code this and submit it with one of my normal patches to arne.

Cool. Generic logging system = good. Can you please point out what part of the code it's in?

Post by **GargoyleMT** » 2004-02-19 16:48

TheParanoidOne wrote:Cool. Generic logging system = good. Can you please point out what part of the code it's in?

The basic functionality is LogManager::getInstance()->message(string);

In BCDC's code, I extended it to keep 10 of the messages in the tooltip... I may reduce that and add a SystemLog window (that only remembers messages while it's open)... I'm not sure yet. I also inserted a simple logging line right before the fire() in message(), dependent upon a boolean setting.

PI · Post by PI » 2004-02-22 20:06

The logging is a good idea !

Back to my question : A solution is putting the removed dupes filenames into the system log instead of creating a new window just for this feature ...

Post by **GargoyleMT** » 2004-02-23 03:42

PI wrote:The logging is a good idea !

Back to my question : A solution is putting the removed dupes filenames into the system log instead of creating a new window just for this feature ...

Hashing is included in the log, as are start/ending a file list refresh, and disconnecting users who leave the hub.

Code: Select all

C:\Program Files\DC++\Logs>tail system.log

2004-02-22 07:52:18: File list refresh finished
2004-02-22 08:53:07: File list refresh initiated
2004-02-22 08:53:18: File list refresh finished
2004-02-22 09:46:46: Disconnected user leaving the hub: Ichitaka_Seto
2004-02-22 09:54:07: File list refresh initiated
2004-02-22 09:54:30: File list refresh finished
2004-02-22 10:55:07: File list refresh initiated
2004-02-22 10:55:25: File list refresh finished
2004-02-22 11:56:07: File list refresh initiated

PI · Post by PI » 2004-02-23 07:59

... I don't see some filenames in your system.log ...

Post by **GargoyleMT** » 2004-02-28 17:33

PI wrote:... I don't see some filenames in your system.log ...

There are none, I was showing ParanoidOne what I'd done so far, not demonstrating what I said I'd code for you. =)

PI · Post by PI » 2004-03-01 06:25

lol

BSOD2600 · Post by **BSOD2600** » 2004-03-12 19:52

Ok, the addition of knowing what dupes are being removed is nice, but it still could be improved a little.

Code: Select all

2004-03-12 10:48:45: Duplicate file will not be shared: Acdc - Problem child.mp3 (size: 5484853 bytes)
2004-03-12 10:48:45: Duplicate file will not be shared: Acdc - Let there be rock.mp3 (size: 5852721 bytes)
2004-03-12 10:48:45: Duplicate file will not be shared: Acdc - Hell ain't a bad place to be.mp3 (size: 4008696 bytes)
2004-03-12 10:48:45: Duplicate file will not be shared: Acdc - Bad boy boogie.mp3 (size: 4311703 bytes)
2004-03-12 10:48:45: Duplicate file will not be shared: folder.jpg (size: 18033 bytes)
2004-03-12 10:48:45: Duplicate file will not be shared: folder.jpg (size: 18033 bytes)
2004-03-12 10:48:46: Duplicate file will not be shared: Maynard Ferguson - 05 - Fiesta.mp3 (size: 7742754 bytes)
2004-03-12 10:48:46: Duplicate file will not be shared: Maynard Ferguson - 01 - Primal Scream.MP3 (size: 10255116 bytes)
2004-03-12 10:48:46: Duplicate file will not be shared: 03 - ManÃƒÂ¡ - Cachito.mp3 (size: 4616280 bytes)

Thats what I get in the log file. It sure would be nice if it told which other file its a dup of and the full path of each.

Post by **GargoyleMT** » 2004-03-13 00:40

BSOD2600 wrote:Thats what I get in the log file. It sure would be nice if it told which other file its a dup of and the full path of each.

Full path? Ok, that can be done (though I haven't written the code). The loop is fairly simple currently, so I'm not sure how you can print "the original" file just once if you have multiple duplicates of it shared...

BSOD2600 · Post by **BSOD2600** » 2004-03-13 01:29

It just saves me the hassle of using crappy windows search to try and find the same named file. DC++ thinks files are dups if they are the exact same byte size or does it look at the filenames too?

Removing the file sorta creates a problem for those who are downloading file sets (ok, an album in my case) since DC++ has now removed one track from it. Yes, they can do a search for that specific missing track, but that creates more hassle.

Twink · Post by **Twink** » 2004-03-13 02:30

BSOD2600 wrote:It just saves me the hassle of using crappy windows search to try and find the same named file. DC++ thinks files are dups if they are the exact same byte size or does it look at the filenames too?

Removing the file sorta creates a problem for those who are downloading file sets (ok, an album in my case) since DC++ has now removed one track from it. Yes, they can do a search for that specific missing track, but that creates more hassle.

maybe there should be an option to leave the file in the list but just not count it towards your share total?

Post by **GargoyleMT** » 2004-03-13 02:32

The dupe check works on exact name and filesize, though hopefully that can be eliminated in favor of hash-based duplicates. =)

Post by **GargoyleMT** » 2004-03-13 02:33

Twink wrote:maybe there should be an option to leave the file in the list but just not count it towards your share total?

That's tongue in cheek, right?

Code: Select all

SETTINGS_REMOVE_DUPES, // "Remove dupes completely from your share (otherwise, only their size is subtracted, but the files can be seen by others)"

YaRi · Post by **YaRi** » 2004-03-27 06:46

GargoyleMT wrote:The dupe check works on exact name and filesize, though hopefully that can be eliminated in favor of hash-based duplicates. =)

Could this be extended so, that when downloading files, which I already have, would give a duplicate error:
"Error downloading: You already have this file".
The comparison would be based on hash of course.

Todi · Post by **Todi** » 2004-03-27 10:28

That's actually a pretty good idea.. i don't know how many dupes i download =) 'course, it would either require you to have the hash before loading, or do the comparison after the file has already loaded (which makes it a bit unnecessary)

Post by **GargoyleMT** » 2004-03-27 21:27

YaRi wrote:"Error downloading: You already have this file".

Agreed - check out this RFE for that:

[ 896494 ] Eradicate duplicate downloads

PI · Post by PI » 2004-05-13 20:51

Thank you you all for your help !
I'm looking forward to seeing this changes ...

+

foxyshadis · Post by **foxyshadis** » 2004-05-15 17:03

Now that TTH is implemented it makes this thread's feature far more accurate. I'd have posted my new thread last week in here instead if I'd seen it. ^.~

DC++

Displaying dupes plzzz

Displaying dupes plzzz

the purpose of displaying this files is obvious !

System Log: Good Idea !

Re: System Log: Good Idea !

Thanks

Who is online