Yea, since these only get printed when an error occurs, performance shouldn't be an issue.
Steve On Fri, Jan 30, 2009 at 10:48 AM, Ali Saidi <[email protected]> wrote: > I would be worried that the collision probability would be too high > with a crc. > > Ali > > On Jan 30, 2009, at 1:46 PM, Gabe Black wrote: > > > Would we need such a strong hash? A dumb CRC would likely be > > sufficient > > if that would be easier to bring in. > > > > Gabe > > > > Ali Saidi wrote: > >> Anyone else? To make this work we would either have to include a md5 > >> algorithm is base (I found a BSD licensed one) or require libopenssl. > >> > >> Ali > >> > >> On Jan 30, 2009, at 11:52 AM, Ali Saidi wrote: > >> > >> > >>> Yea I thought about that. Pretty much if you update an error > >>> message, > >>> you need to move the old hash to the new hash on the wiki. > >>> > >>> Ali > >>> > >>> On Jan 30, 2009, at 2:48 AM, Gabe Black wrote: > >>> > >>> > >>>> That sounds like a good idea, but what if I had panic("Soemthing > >>>> bad") > >>>> and changed it to panic("Something bad")? They'd be the same thing > >>>> but > >>>> with different hashes. > >>>> > >>>> Gabe > >>>> > >>>> Ali Saidi wrote: > >>>> > >>>>> We've talked for a while about needing better error messages, > >>>>> links, > >>>>> more descriptions on the website, etc. How about we modify > >>>>> panic(), > >>>>> fatal(), and warn() to hash the format string for the message and > >>>>> print out a url with some portion of that hash. > >>>>> > >>>>> So: > >>>>> fatal("Parameter %s.%s was not specified") > >>>>> would turn into: > >>>>> Fatal: Parameter foo.bar was not specified > >>>>> See http://www.m5sim.org/wiki/index.php/9c6d054cbec3 > >>>>> > >>>>> As people asked questions we could populate the hashes with > >>>>> answers > >>>>> in > >>>>> the wiki. (We could also use url re-writing to make the url a bit > >>>>> smaller). > >>>>> > >>>>> Thoughts? > >>>>> > >>>>> Ali > >>>>> > >>>>> > >>>>> _______________________________________________ > >>>>> m5-dev mailing list > >>>>> [email protected] > >>>>> http://m5sim.org/mailman/listinfo/m5-dev > >>>>> > >>>>> > >>>> _______________________________________________ > >>>> m5-dev mailing list > >>>> [email protected] > >>>> http://m5sim.org/mailman/listinfo/m5-dev > >>>> > >>>> > >>> _______________________________________________ > >>> m5-dev mailing list > >>> [email protected] > >>> http://m5sim.org/mailman/listinfo/m5-dev > >>> > >>> > >> > >> _______________________________________________ > >> m5-dev mailing list > >> [email protected] > >> http://m5sim.org/mailman/listinfo/m5-dev > >> > > > > _______________________________________________ > > m5-dev mailing list > > [email protected] > > http://m5sim.org/mailman/listinfo/m5-dev > > > > _______________________________________________ > m5-dev mailing list > [email protected] > http://m5sim.org/mailman/listinfo/m5-dev >
_______________________________________________ m5-dev mailing list [email protected] http://m5sim.org/mailman/listinfo/m5-dev
