On Fri, 2010-02-26 at 17:09 -0500, dar...@chaosreigns.com wrote:
> On 02/26, Karsten Bräckelmann wrote:
> > opinions there. If the code and idea is deemed good, eventually sign a
> > CLA (assuming it's a non-trivial change), so the code can be accepted
> > for upstream inclusion.
> 
> Thanks.  I'm looking more for the requirements for the code being accepted
> into SA.  It looks nice and straight forward, I'd just like some
> confirmation that it's likely to be accepted before I do the work.

Write nice and clean code. Good documentation strongly preferred. Sign
CLA. Be sure the service is OK with the usage / reporting feature. (OK,
we got the latter. ;)

If it requires an account, be sure the code doesn't just fail if there
is none given, but simply make it a no-op in that case, failing
transparently without harming anything else.

Other than that, there's not too much we can say in advance.

> Maybe I should start by filing a feature request bug against SA for this
> feature?

Feel free to create a bug report to track progress and attach the actual
code or patches. However, do not expect a lot of comments before there's
any substance.


-- 
char *t="\10pse\0r\0dtu...@ghno\x4e\xc8\x79\xf4\xab\x51\x8a\x10\xf4\xf4\xc4";
main(){ char h,m=h=*t++,*x=t+2*h,c,i,l=*x,s=0; for (i=0;i<l;i++){ i%8? c<<=1:
(c=*++x); c&128 && (s+=h); if (!(h>>=1)||!t[s+h]){ putchar(t[s]);h=m;s=0; }}}

Reply via email to