Print

Please login or register.
1 Hour 1 Day 1 Week 1 Month Forever
Login with username, password and session length

[darksidex]

Se que ahora amule reanuda las descargas en función de la prioridad, pero ¿no sería mejor hacerlo en función del nombre? ¿Sobre todo si se basa de reanudar la descargas de la misma categoría?

¿Que os parece?

PD Si os da pereza lo hago yo  :]

[Jacobo221]

tu mismo el patch

pero las prioridades estan para lo que estan. lo del nombre tendria que ser para cuando hay un empate de prioridades

[darksidex]

Yo no lo veo así, tal vez porque uso la autoprioridad  

Pero si realmente quereis el patch, dime como (diff, la función entera directamente..) y a quién se lo paso.

[Kry]

Eso ya esta hecho...

[darksidex]

Originally posted by Kry
Eso ya esta hecho...

Solo por curiosidad, ¿en que parte? en "void CDownloadQueue::StartNextFile(CPartFile* oldfile);" no veo que se tiene en cuenta el nombre del fichero, solo la prioridad, y yo pensaba que esa era la función que se encargaba de reanudar las descargas.

[darksidex]

Al final hago el parche o no?

[Kry]

Hazlo. pero me hace raroq ue se haya perdido esa funcionalidad.

[darksidex]

Esta es la función tal cual la tenéis ahora:

Code: [Select]

void CDownloadQueue::StartNextFile(CPartFile* oldfile)
{
if ( thePrefs::StartNextFile() ) {
CPartFile* tFile = NULL;

{
wxMutexLocker lock(m_mutex);

for ( uint16 i = 0; i < m_filelist.size(); i++ ) {
CPartFile* file = m_filelist[i];

if ( file->GetStatus() == PS_PAUSED ) {
if ( !tFile || file->GetDownPriority() > tFile->GetDownPriority() ) {
if (tFile && thePrefs::StartNextFileSame()) {
if (tFile->GetCategory() == oldfile->GetCategory()) {
// Already found a file for this category
if (file->GetCategory() == oldfile->GetCategory()) {
// But the new one is also for this category
if (file->GetDownPriority() > tFile->GetDownPriority()) {
// So, higher prio?
tFile = file;
} else {
; // Lower prio, ignore this file.
}
} else {
; // It's not from the same category, so nothing
}
} else {
// Found no file yet with the same category
if (file->GetDownPriority() > tFile->GetDownPriority()) {
// So, higher prio?
tFile = file;
} else {
; // Lower prio, ignore this file.
}
}
} else {
// There is no file found yet or no category selection.
tFile = file;
}

if ( tFile->GetDownPriority() == PR_HIGH ) {
if (thePrefs::StartNextFileSame()) {
// It's from the same category?
if (tFile->GetCategory() == oldfile->GetCategory()) {
// There can't be any higher
break;
} else {
; // Maybe we'll find another file from the same category
}
} else {
// There can't be any higher
break;
}
}
}
}
}
}

if ( tFile ) {
tFile->ResumeFile();
}
}
}

Solo tiene en cuenta las prioridades.

A ver si este finde me pongo y la hago, que no tengo mucho tiempo libre ultimamente.

[Kry]

ok, gracias. Yo sigo emparanoyado con que lo HACIA, mas que nada porque habia añadido yo lo de las categorias y eso... por otra parte, normalmente no recuerdo ni lo que desayuno, asi que....

Tu monta el patch con las categorias, añana en preferencias un thePrefs::StartNextFileAlpha() equivalente al thePrefs::StartNextFileSame() pero para alfabetico, y yo me encargo de añadirlo al gui.

[darksidex]

Ya me he perdido, ahora no se si tengo que hacer una función que tenga en cuenta SOLO los nombres, que es lo que te he entendido a ti, o que tenga también en cuenta las prioridades del archivo, que es lo que creo que quería jacobo.

A mi me da igual montar una o la otra, pero haced un pequeño consenso, y me decís, más que nada para que esté listo cuanto antes.

[Kry]

Simplemente tienes que extender la funcionalidad actual para que DESPUES de comprobar las prioridades, a prioridades equivalentes compruebe el nombre. Pero opcionalmente, asi que "if (tFile && thePrefs::StartNextFileSame()) {" en el codigo, que es para elegir solo por categorias, deberia tener su equivalente, por ejemplo thePrefs::StartNextFileAlpha() para permitir elegir si queremos este comportamiento o no.


Por otra parte, puedes hacer que no sea opcional, y si me aburro ya lo pondre opcional yo mas tarde.

[darksidex]

ok, el finde me pongo y os paso el código.
En que formato? supongo que diff, pero con que parámetros lo monto (nunca lo he usado )

[Jacobo221]

diff -u

[darksidex]

Ya está

No lo he probado porque no he cambiado el tema de las preferencias para que soporte thePrefs::StartNextFileAlpha(), así que no podría compilar, pero he usado el mismo método que uso en el emule, o que usaba en amule cuando lo compilaba en FreeBSD, salvo por lo de StartNextFileAlpha().

En el tar.gz hay dos ficheros, el primer parche es para que tenga en cuenta el orden alfabético, da igual si debe estar en la misma categoría o no. En el segundo solo tiene en cuenta el orden alfabético si está en la misma categoría, así podeis implementar el tema de las preferencias como mejor os venga. En ambos casos tiene en cuenta la prioridad.