<div dir="ltr">You can submit them as three separate patches.</div><div class="gmail_extra"><br><div class="gmail_quote">On Mon, Mar 12, 2018 at 2:47 AM, Guillaume Chéreau <span dir="ltr"><<a href="mailto:guillaume.chereau@gmail.com" target="_blank">guillaume.chereau@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">How do I submit a branch to developer.blender instead of a patch?  My<br>
current changes consist of three patches, should I make one patch<br>
submission for each, or concatenate them into a a single patch?<br>
<br>
On Mon, Mar 12, 2018 at 5:50 AM, Brecht Van Lommel<br>
<div class="HOEnZb"><div class="h5"><<a href="mailto:brechtvanlommel@pandora.be">brechtvanlommel@pandora.be</a>> wrote:<br>
> To get code included in the main Cycles repository, please submit it for<br>
> code review here:<br>
> <a href="https://developer.blender.org/differential/diff/create/" rel="noreferrer" target="_blank">https://developer.blender.org/<wbr>differential/diff/create/</a><br>
><br>
> A GitHub URL is not enough, we want to be able to have a record of the code<br>
> review and be able to comment on lines of code on <a href="http://developer.blender.org" rel="noreferrer" target="_blank">developer.blender.org</a>.<br>
><br>
> Thanks,<br>
> Brecht.<br>
><br>
> On Sun, Mar 11, 2018 at 3:42 PM, Guillaume Chéreau<br>
> <<a href="mailto:guillaume.chereau@gmail.com">guillaume.chereau@gmail.com</a>> wrote:<br>
>><br>
>> OK, then here is the link:<br>
>> <a href="https://github.com/guillaumechereau/cycles/tree/remove_oiio" rel="noreferrer" target="_blank">https://github.com/<wbr>guillaumechereau/cycles/tree/<wbr>remove_oiio</a><br>
>><br>
>> This is the branch 'remove_oiio' in my cycles github repo. Please<br>
>> ignore the master branch, as it's mostly quick hacks I did in order to<br>
>> have something working for goxel.<br>
>><br>
>> Guillaume<br>
>><br>
>> On Sun, Mar 11, 2018 at 9:10 PM, Nathan Letwory <<a href="mailto:nathan@mcneel.com">nathan@mcneel.com</a>> wrote:<br>
>> > The github URL to it should be ok.<br>
>> ><br>
>> > On 11 Mar 2018 1:01 pm, "Guillaume Chéreau"<br>
>> > <<a href="mailto:guillaume.chereau@gmail.com">guillaume.chereau@gmail.com</a>><br>
>> > wrote:<br>
>> >><br>
>> >> I started to work on a proper branch, it's not done yet, but I can<br>
>> >> push it online if anybody is interested to have a look.<br>
>> >> What would be the easiest way to share a branch for review?<br>
>> >><br>
>> >> Regards,<br>
>> >> Guillaume<br>
>> >><br>
>> >> On Wed, Mar 7, 2018 at 6:05 AM, Brecht Van Lommel<br>
>> >> <<a href="mailto:brechtvanlommel@pandora.be">brechtvanlommel@pandora.be</a>> wrote:<br>
>> >> > Using std::string instead of ustring will lead to a lot more memory<br>
>> >> > allocations, which can be slow but may be ok in practice for the way<br>
>> >> > Cycles<br>
>> >> > uses it.<br>
>> >> ><br>
>> >> > Feel free to submit a patch on <a href="http://developer.blender.org" rel="noreferrer" target="_blank">developer.blender.org</a> when you think<br>
>> >> > the<br>
>> >> > code<br>
>> >> > is in a good state.<br>
>> >> ><br>
>> >> > On Tue, Mar 6, 2018 at 4:47 AM, Guillaume Chéreau<br>
>> >> > <<a href="mailto:guillaume.chereau@gmail.com">guillaume.chereau@gmail.com</a>> wrote:<br>
>> >> >><br>
>> >> >> I just started a branch of Cycles on github for the goxel version.<br>
>> >> >><br>
>> >> >> My first commit allows to use std::string for ustring and a custom<br>
>> >> >> implementation of TypeDef:<br>
>> >> >><br>
>> >> >><br>
>> >> >><br>
>> >> >> <a href="https://github.com/guillaumechereau/cycles/commit/0f286bc4da5536a19640eb4ede640423e317e733" rel="noreferrer" target="_blank">https://github.com/<wbr>guillaumechereau/cycles/<wbr>commit/<wbr>0f286bc4da5536a19640eb4ede6404<wbr>23e317e733</a><br>
>> >> >><br>
>> >> >> Regards,<br>
>> >> >> Guillaume<br>
>> >> >><br>
>> >> >> On Tue, Mar 6, 2018 at 10:05 AM, Guillaume Chéreau<br>
>> >> >> <<a href="mailto:guillaume.chereau@gmail.com">guillaume.chereau@gmail.com</a>> wrote:<br>
>> >> >> > It is actually not difficult to replace oiio ustring with<br>
>> >> >> > std::string<br>
>> >> >> > in the code, since the api are almost the same, but the<br>
>> >> >> > performance<br>
>> >> >> > are probably not as good with std::string.<br>
>> >> >> ><br>
>> >> >> > In util/util_param.h, I can comment out the OpenImageIO/ustring.h<br>
>> >> >> > include and add those lines:<br>
>> >> >> ><br>
>> >> >> > // ustring implementation based on std::string.<br>
>> >> >> > class ustring : public std::string<br>
>> >> >> > {<br>
>> >> >> > public:<br>
>> >> >> >     ustring() {}<br>
>> >> >> >     explicit ustring(const char *str) : std::string(str) {}<br>
>> >> >> >     ustring (const ustring &str) : ustring(str.c_str()) {}<br>
>> >> >> >     ustring (const std::string &str) : ustring(str.c_str()) {}<br>
>> >> >> >     const std::string &string() const { return *this; }<br>
>> >> >> >     operator int(void) const { return !empty(); }<br>
>> >> >> >     const ustring &operator=(const char *str) { *this = str;<br>
>> >> >> > return<br>
>> >> >> > *this; }<br>
>> >> >> > };<br>
>> >> >> > typedef std::hash<std::string> ustringHash;<br>
>> >> >> ><br>
>> >> >> > And the compilation works.<br>
>> >> >> ><br>
>> >> >> > Similar treatment can be done for typedesc.<br>
>> >> >> ><br>
>> >> >> ><br>
>> >> >> > Regards,<br>
>> >> >> > Guillaume<br>
>> >> >> ><br>
>> >> >> > --<br>
>> >> >> > Guillaume<br>
>> >> >> > <a href="mailto:guillaume.chereau@gmail.com">guillaume.chereau@gmail.com</a><br>
>> >> >> > <a href="tel:%2B886%20970422910" value="+886970422910">+886 970422910</a><br>
>> >> >><br>
>> >> >><br>
>> >> >><br>
>> >> >> --<br>
>> >> >> Guillaume<br>
>> >> >> <a href="mailto:guillaume.chereau@gmail.com">guillaume.chereau@gmail.com</a><br>
>> >> >> <a href="tel:%2B886%20970422910" value="+886970422910">+886 970422910</a><br>
>> >> >> ______________________________<wbr>_________________<br>
>> >> >> Bf-cycles mailing list<br>
>> >> >> <a href="mailto:Bf-cycles@blender.org">Bf-cycles@blender.org</a><br>
>> >> >> <a href="https://lists.blender.org/mailman/listinfo/bf-cycles" rel="noreferrer" target="_blank">https://lists.blender.org/<wbr>mailman/listinfo/bf-cycles</a><br>
>> >> ><br>
>> >> ><br>
>> >> ><br>
>> >> > ______________________________<wbr>_________________<br>
>> >> > Bf-cycles mailing list<br>
>> >> > <a href="mailto:Bf-cycles@blender.org">Bf-cycles@blender.org</a><br>
>> >> > <a href="https://lists.blender.org/mailman/listinfo/bf-cycles" rel="noreferrer" target="_blank">https://lists.blender.org/<wbr>mailman/listinfo/bf-cycles</a><br>
>> >> ><br>
>> >><br>
>> >><br>
>> >><br>
>> >> --<br>
>> >> Guillaume<br>
>> >> <a href="mailto:guillaume.chereau@gmail.com">guillaume.chereau@gmail.com</a><br>
>> >> <a href="tel:%2B886%20970422910" value="+886970422910">+886 970422910</a><br>
>> >> ______________________________<wbr>_________________<br>
>> >> Bf-cycles mailing list<br>
>> >> <a href="mailto:Bf-cycles@blender.org">Bf-cycles@blender.org</a><br>
>> >> <a href="https://lists.blender.org/mailman/listinfo/bf-cycles" rel="noreferrer" target="_blank">https://lists.blender.org/<wbr>mailman/listinfo/bf-cycles</a><br>
>> ><br>
>> ><br>
>> > ______________________________<wbr>_________________<br>
>> > Bf-cycles mailing list<br>
>> > <a href="mailto:Bf-cycles@blender.org">Bf-cycles@blender.org</a><br>
>> > <a href="https://lists.blender.org/mailman/listinfo/bf-cycles" rel="noreferrer" target="_blank">https://lists.blender.org/<wbr>mailman/listinfo/bf-cycles</a><br>
>> ><br>
>><br>
>><br>
>><br>
>> --<br>
>> Guillaume<br>
>> <a href="mailto:guillaume.chereau@gmail.com">guillaume.chereau@gmail.com</a><br>
>> <a href="tel:%2B886%20970422910" value="+886970422910">+886 970422910</a><br>
>> ______________________________<wbr>_________________<br>
>> Bf-cycles mailing list<br>
>> <a href="mailto:Bf-cycles@blender.org">Bf-cycles@blender.org</a><br>
>> <a href="https://lists.blender.org/mailman/listinfo/bf-cycles" rel="noreferrer" target="_blank">https://lists.blender.org/<wbr>mailman/listinfo/bf-cycles</a><br>
><br>
><br>
><br>
> ______________________________<wbr>_________________<br>
> Bf-cycles mailing list<br>
> <a href="mailto:Bf-cycles@blender.org">Bf-cycles@blender.org</a><br>
> <a href="https://lists.blender.org/mailman/listinfo/bf-cycles" rel="noreferrer" target="_blank">https://lists.blender.org/<wbr>mailman/listinfo/bf-cycles</a><br>
><br>
<br>
<br>
<br>
--<br>
Guillaume<br>
<a href="mailto:guillaume.chereau@gmail.com">guillaume.chereau@gmail.com</a><br>
<a href="tel:%2B886%20970422910" value="+886970422910">+886 970422910</a><br>
______________________________<wbr>_________________<br>
Bf-cycles mailing list<br>
<a href="mailto:Bf-cycles@blender.org">Bf-cycles@blender.org</a><br>
<a href="https://lists.blender.org/mailman/listinfo/bf-cycles" rel="noreferrer" target="_blank">https://lists.blender.org/<wbr>mailman/listinfo/bf-cycles</a><br>
</div></div></blockquote></div><br></div>