Re: Context parameter in public API

Nuutti Kotivuori <naked@iki.fi> writes:
> I think svn_time_to_human_nts should be modified to accept the format
> as a parameter somehow. It might be mandatory, or there might be an
> easy hard-coded default that can be used. Then the command-line client
> subcommands log and info, svnadmin and svnlook would be modified to
> just read the configuration file themselves and pass the date format
> parameter to svn_time_to_human_nts. And then issues should be filed
> about getting sense into the configuration file processing in the long
> term.
>
> Thoughts? ;-)

Well... hmmm.

Greg Stein distinguishes between "good kluges" and "bad kluges". A
bad kluge is when you do work that you *know* must later be undone
because a better (but more complex) fix will be implemented.

I feel like we're verging on a bad kluge here.

How hard would it be to do the generic configuration context thing
first, and then do configurable date parsing the Right Way?

-K

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org
Received on Fri Oct 18 01:35:33 2002