<div dir="ltr">Hello,<div><br></div><div>I have tried to make changes in the module, but I do not have the rights for this.</div><div><div class="gmail_extra"><br clear="all"><div><div class="gmail_signature"><div dir="ltr"><div><font face="Arial, Tahoma, Verdana, sans-serif" color="#666666"><span style="line-height:18.2000007629395px">________________________________</span></font></div><div><font face="Arial, Tahoma, Verdana, sans-serif" color="#666666"><span style="line-height:18.2000007629395px">Andrey Rybkin</span></font></div><div><font face="Arial, Tahoma, Verdana, sans-serif" color="#666666"><span style="line-height:18.2000007629395px">JID: <a href="mailto:rybkin.a@bks.tv" target="_blank">rybkin.a@bks.tv</a></span></font></div><div><font face="Arial, Tahoma, Verdana, sans-serif" color="#666666"><span style="line-height:18.2000007629395px">E-Mail: <a href="mailto:rybkin.a@bks.tv" target="_blank">rybkin.a@bks.tv</a></span></font></div><div><br></div></div></div></div><div class="gmail_quote">2015-01-06 14:25 GMT+03:00 Daniel-Constantin Mierla <span dir="ltr"><<a href="mailto:miconda@gmail.com" target="_blank">miconda@gmail.com</a>></span>:<br><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex">Hello,<br>
<br>
your pull request to merge the usrloc_dmq module was accepted few days ago.<br>
<br>
I wrote in the merge comment some remarks about few thinks to be cared of:<br>
<br>
- add the license text at the top of .c/.h files (license must be gplv2<br>
or bsd, at your choice)<br>
- variables to be declared at the beginning of functions or blocks, to<br>
be sure it still compiles with old strict C compilers<br>
<br>
Giving a second thought and considering the current common naming<br>
patterns, I propose to rename the module from usrloc_dmq to dmq_usrloc<br>
(we have similar for pua_usrloc or pua_reginfo) -- the idea is that the<br>
main purpose of the module is extending dmq with a new data flow type,<br>
rather than usrloc specific extensions. Is that ok for you? If you don't<br>
have time for it in short term, I can do it.<br>
<br>
Cheers,<br>
Daniel<br>
<span class=""><font color="#888888"><br>
--<br>
Daniel-Constantin Mierla<br>
<a href="http://twitter.com/#!/miconda" target="_blank">http://twitter.com/#!/miconda</a> - <a href="http://www.linkedin.com/in/miconda" target="_blank">http://www.linkedin.com/in/miconda</a><br>
<br>
</font></span></blockquote></div><br></div></div></div>