<div dir="ltr">Minor feedback, this comment should say "which shader model to use" instead of "which entry point to use"<div><br></div><div><div class="" style="font-family:monospace;white-space:pre;color:rgb(0,136,0);line-height:normal">+    // Figure out which entry point to use if any</div><div class="" style="font-family:monospace;white-space:pre;color:rgb(0,136,0);line-height:normal">+    if (const char* sm = sf->GetProperty("VS_SHADER_MODEL"))</div></div><div class="" style="font-family:monospace;white-space:pre;color:rgb(0,136,0);line-height:normal"><br></div><div class="" style="font-family:monospace;white-space:pre;color:rgb(0,136,0);line-height:normal"><br></div></div><div class="gmail_extra"><br><div class="gmail_quote">On Wed, Dec 3, 2014 at 11:08 AM, Cedric Perthuis <span dir="ltr"><<a href="mailto:cedric.perthuis@gmail.com" target="_blank">cedric.perthuis@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr">That sounds great, thanks Brad!</div><div class="HOEnZb"><div class="h5"><div class="gmail_extra"><br><div class="gmail_quote">On Wed, Dec 3, 2014 at 7:12 AM, Brad King <span dir="ltr"><<a href="mailto:brad.king@kitware.com" target="_blank">brad.king@kitware.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span>On 12/02/2014 10:27 AM, Brad King wrote:<br>
>  VS: Add source file properties to set the hlsl shader entry point and model<br>
>  <a href="http://cmake.org/gitweb?p=cmake.git;a=commitdiff;h=5ab06dcd" target="_blank">http://cmake.org/gitweb?p=cmake.git;a=commitdiff;h=5ab06dcd</a><br>
<br>
</span>I updated the patch to sort the cmake-properties manual list in order:<br>
<span><br>
 VS: Add source file properties to set the hlsl shader entry point and model<br>
</span> <a href="http://cmake.org/gitweb?p=cmake.git;a=commitdiff;h=2a224b4c" target="_blank">http://cmake.org/gitweb?p=cmake.git;a=commitdiff;h=2a224b4c</a><br>
<br>
I've queued this for inclusion in the next 3.1 release candidate because<br>
otherwise the new hlsl feature it not very useful.<br>
<br>
Thanks,<br>
-Brad<br>
<br>
</blockquote></div><br></div>
</div></div></blockquote></div><br></div>