[Bf-codereview] Use DerivedMesh instead of Mesh in COLLADA GeometryExporter (issue4387052)
Daniel Tavares
danielmtavares at gmail.com
Thu May 5 17:28:53 CEST 2011
Hi official.address243,
I already looked at some of your changes last night. I'll apply your
changes and continue working based off of that.
I'm not try to go after all trailing spaces/tabs. But I have the habit
of cleaning some of these minor things as I browse through the code.
Since these changes tends to clutter the patch, I started putting them
in a separate branch. That way I can send that as a separate patch.
That goes for typos as well.
Is the stuff that you're working on part of a GSoc2011 project?
I look forward to working with you on this.
Cheers,
Daniel
On May 5, 2011, at 2:42, "official.address243 at googlemail.com"
<official.address243 at googlemail.com> wrote:
> Hello "dBugSlayer",
> I suggest we work together. It would be great if you would apply my
> patch: http://codereview.appspot.com/4425084/, and continue your work
> with that patch. I fixed already some of the stuff Brecht mentioned. I
> suggest we fix at first the code, so it works flawless, then going after
> all wrong spaces, newlines, etc..
>
> http://codereview.appspot.com/4387052/
More information about the Bf-codereview
mailing list