Danek Duvall wrote:

<snip>
>>    Line 290 - def _snarf(...)
>>      No offence intended Danek, but frankly, I'm surprised that you used
>>      a slang name for this function when you seem to be very rigorous
>>      about code style most of the time.
>>      I'm not hung up on changing the name, but _snarf just strikes me
>>      as a little crass, even though I get why you named it thus.
>>     
>
> Perhaps we have a different idea of what this means?  US/UK difference,
> maybe?  It's not remotely offensive in my world.
>   
I don't think its offensive - just not very meaningful, but as you say, 
it may be a language difference between our two mighty empires ("two 
countries separated by a common language") :)
> I've put up a new webrev:
>
>     http://cr.opensolaris.org/~dduvall/pkg-780-3/
>
> Both a fresh install and an upgrade from 79 continue to verify correctly.
>
> Danek
>   

_______________________________________________
pkg-discuss mailing list
[email protected]
http://mail.opensolaris.org/mailman/listinfo/pkg-discuss

Reply via email to