On Mon, 2014-05-26 at 21:47 -0700, Jay Sorg wrote: > Ok, I submitted a patch to add the xrdp sink. > I'm can submit the source after this gets review. > I know the doc says split the patch up into < 200 line or something > but sense this is a new sink I can [not] split it anymore. I'll write another reply to this discussion, but here I just want to note that being a new sink doesn't prevent you from implementing the module in multiple phases. But don't worry, the 200 line thing is just a recommendation, not a hard requirement. -- Tanu