Submit Hint Search The Forums LinksStatsPollsHeadlinesRSS
14,000 hints and counting!


Click here to return to the 'Selectively delete Safari favicons from the icon cache' hint
The following comments are owned by whoever posted them. This site is not responsible for what they say.
Selectively delete Safari favicons from the icon cache
Authored by: hypert on Dec 23, '04 12:45:30PM
Excellent hint, and very nicely-written script. One comment - please use mktemp to create temporary files, as discussed before.

[ Reply to This | # ]
Selectively delete Safari favicons from the icon cache
Authored by: kps on Dec 27, '04 01:53:12PM
Actually, it's horribly written. That temporary file is completely unnecessary in the first place. So is the extra grep used to remove the error messages that result from applying the useful grep to directories (clue: find -type f). So is the gratuitous cut to select the file name from the grep output (clue: grep -l).

I wouldn't normally pick on a neophyte's clumsy mistakes, but the author of this script is actually selling a book on writing shell scripts! Ignorance is no vice; packaging it as wisdom is.

[ Reply to This | # ]