[Fasterfox] Bug in Fasterfox 0.6.5

[Tony Gentilcore]

Kai,

Thanks for pointing that out.  I will incorporate this into the next release
and credit you with the fix in the source code.

If you notice other things, please let me know, I am always looking for
improvement.

Also, if you or anyone you know would be interested in creating a
localization for Fasterfox, I'm currently looking for translators.

Thanks again,
Tony

-----Original Message-----
From: fasterfox-bounces at mozdev.org [mailto:fasterfox-bounces at mozdev.org] On
Behalf Of Kai Risku
Sent: Monday, October 03, 2005 3:10 AM
To: fasterfox at mozdev.org
Subject: [Fasterfox] Bug in Fasterfox 0.6.5

I think the following code is buggy:

	 // Don't prefetch things that look like a logout link 
           if(href.indexOf('logout')>0 || href.indexOf('logoff')>0)
continue;

because it doesn't work correctly for HTML that looks like this
	<a href="logout.html">Logout</a>

I.e. indexOf returns zero when the substring is found at the beginning
of the searched string. You should change >0 to >=0 to match that case.
IndexOf returns -1 if it does not find anything.

Regards,
    Kai

--
Kai.Risku at arrak.fi     GSM  +358-40-767 8282
Oy Arrak Software Ab   http://www.arrak.fi
 

_______________________________________________
Fasterfox mailing list
Fasterfox at mozdev.org
http://mozdev.org/mailman/listinfo/fasterfox