
<!DOCTYPE html><html><head><title></title><style type="text/css">p.MsoNormal,p.MsoNoSpacing{margin:0}</style></head><body><div><br></div><div>On Wed, Jan 8, 2020, at 18:25, Vincenzo Nicosia wrote:<br></div><blockquote type="cite" id="qt"><div>On Wed, Jan 08, 2020 at 05:00:39PM +0100, Thomas Guillem wrote:<br></div><div>> <br></div><div>> <br></div><div>> On Wed, Jan 8, 2020, at 14:48, Rémi Denis-Courmont wrote:<br></div><div>> > This has nothing to do with the raw TCP protocol. By that logic why don't you merge tcp.c with file.c which is essentially doing the same thing except on Open...?<br></div><div>> > <br></div><div>> > This makes no architectural sense. A separate protocol is a separate modulew especially if it's niche, so it can be left out.<br></div><div>> <br></div><div>> I don't have strong opinions about that but I still tend to prefer having both module into one.<br></div><div>> <br></div><div>> I can revert and merge the initial patch tomorrow, if you all agree with Rémi.<br></div><div>> <br></div><div><br></div><div>Either option is fine by me.<br></div><div><br></div><div>If you agree about going for a separate gopher module, I will re-send<br></div><div>the patch to be applied for review, since we have had at least three<br></div><div>different ones in this thread and there is a risk to apply the wrong<br></div><div>one.<br></div><div><br></div><div>Waiting for the final verdict :)<br></div></blockquote><div><br></div><div>OK, let separate the two modules then. Sorry for misleading you.<br></div><div><br></div><blockquote type="cite" id="qt"><div><br></div><div>Thanks<br></div><div><br></div><div><br></div><div>_______________________________________________<br></div><div>vlc-devel mailing list<br></div><div>To unsubscribe or modify your subscription options:<br></div><div>https://mailman.videolan.org/listinfo/vlc-devel<br></div><div><br></div><div><b>Attachments:</b><br></div><ul><li>signature.asc<br></li></ul></blockquote><div><br></div></body></html>