<div class="gmail_quote">On Tue, Jul 6, 2010 at 1:13 PM, Marco Nolden <span dir="ltr"><<a href="mailto:m.nolden@dkfz-heidelberg.de">m.nolden@dkfz-heidelberg.de</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;">

<div class="im">On 07/06/2010 06:11 PM, Marcus D. Hanwell wrote:</div><div class="im"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
Sorry if I missed this one, the change looks reasonable and I can take a<br>
look at it. It would be good to see improved unicode support too. It is<br>
something that the ARB should probably be discussing, but it would be<br>
great to involve the community as far as is possible.<br>
<br>
Marcus<br>
<br>
</blockquote></div>
Actually I revised it a bit, so the link is here:<br>
<br>
<a href="http://github.com/nolden/VTK/commit/9b76af72e6016fcd4a7b2e1a58edb77eb5549eef" target="_blank">http://github.com/nolden/VTK/commit/9b76af72e6016fcd4a7b2e1a58edb77eb5549eef</a><br>
<br>
and the patch is attached.<br>
<br></blockquote><div>This is now merged, and was actually the first topic branch merged into VTK using the staging repository we have been working on. I actually made a second commit in that topic where I fixed the public API to use unsigned char, and also use static_cast instead of the C style casts.</div>

<div><br></div><div>The commit prefix is normally BUG: rather than FIX: too, although this is something else we need to make clear (are we requiring the log message prefixes). Thanks for your patch, I encourage you to keep an eye on the dashboards (although I will be doing that too).</div>

<div><br></div><div>Marcus</div></div>