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; }}}