<html>
<head>
<meta http-equiv="Content-Type" content="text/html; charset=gb2312">
<style type="text/css" style="display:none;"> P {margin-top:0;margin-bottom:0;} </style>
</head>
<body dir="ltr">
<div style="color: rgb(0, 0, 0); font-family: Calibri,Helvetica,sans-serif; font-size: 12pt;">
Got it! Thank you Joshua!</div>
<div style="color: rgb(0, 0, 0); font-family: Calibri,Helvetica,sans-serif; font-size: 12pt;">
YimingWu<br>
</div>
<hr style="display:inline-block;width:98%" tabindex="-1">
<div id="divRplyFwdMsg" dir="ltr"><font face="Calibri, sans-serif" style="font-size:11pt" color="#000000"><b>发件人:</b> Soc-2018-dev <soc-2018-dev-bounces@blender.org> 代表 Joshua Leung <aligorith@gmail.com><br>
<b>发送时间:</b> 2018年5月19日 7:51<br>
<b>收件人:</b> soc-2018-dev@blender.org<br>
<b>主题:</b> Re: [Soc-2018-dev] Several questions about working on the code</font>
<div> </div>
</div>
<meta content="text/html; charset=utf-8">
<div>
<div dir="auto">Hi Yiming,
<div dir="auto"><br>
</div>
<div dir="auto">1) That depends a lot on where the code will go, though it is preferable that all new code conforms to these guidelines.</div>
<div dir="auto">-If it goes in source/blender and integrates with the rest of our C code (blenkernel, draw engines, editors, etc) you'll need to make it match our code style.</div>
<div dir="auto">- If it is going into intern or extern libs folders, we are a bit less strict about this, mainly since we bundle a few externally maintained projects (eg audaspace, bullet, gl...) that need to be updated to upstream releases in future.</div>
<div dir="auto"><br>
</div>
<div dir="auto">2) For all non core modules (eg freestyle, bullet/rigid body, cycles, ffmpeg, and some physics sims/modifiers, we typically have cmake options to turn those modules off when compiling so people can have faster/lighter builds.</div>
<div dir="auto"><br>
</div>
<div dir="auto">If these are just configuration options for testing different code paths, we can look into this later.</div>
<div dir="auto"><br>
</div>
<div dir="auto"><br>
</div>
<div dir="auto"><br>
</div>
<div dir="auto">Hope that helps,</div>
<div dir="auto">Joshua</div>
<br>
<br>
<div class="x_gmail_quote" dir="auto">
<div dir="ltr">On Sat, May 19, 2018, 3:02 AM n w <<a href="mailto:xp8110@live.com">xp8110@live.com</a>> wrote:<br>
</div>
<blockquote class="x_gmail_quote" style="margin:0 0 0 .8ex; border-left:1px #ccc solid; padding-left:1ex">
<div dir="ltr">
<div style="color:rgb(0,0,0); font-family:Calibri,Helvetica,sans-serif; font-size:12pt">
Hi devs,</div>
<div style="color:rgb(0,0,0); font-family:Calibri,Helvetica,sans-serif; font-size:12pt">
<br>
</div>
<div style="color:rgb(0,0,0); font-family:Calibri,Helvetica,sans-serif; font-size:12pt">
I have two quick questions on working on the code:</div>
<div style="color:rgb(0,0,0); font-family:Calibri,Helvetica,sans-serif; font-size:12pt">
1) I've read the code style guidelines on blenderwiki, since I'll be porting code from exiting project, do I have to chage all functions and struct naming to match the guideline?</div>
<div style="color:rgb(0,0,0); font-family:Calibri,Helvetica,sans-serif; font-size:12pt">
2) Is it required to add cmake options for building seperate module? If it is so could anyone share a way that achieves this? I'm not quite familiar with cmake files but making #ifdef brackets for source code works just fine on me.</div>
<div style="color:rgb(0,0,0); font-family:Calibri,Helvetica,sans-serif; font-size:12pt">
<br>
</div>
<div style="color:rgb(0,0,0); font-family:Calibri,Helvetica,sans-serif; font-size:12pt">
Thanks,</div>
<div style="color:rgb(0,0,0); font-family:Calibri,Helvetica,sans-serif; font-size:12pt">
YimingWu<br>
</div>
</div>
-- <br>
Soc-2018-dev mailing list<br>
<a href="mailto:Soc-2018-dev@blender.org" target="_blank" rel="noreferrer">Soc-2018-dev@blender.org</a><br>
<a href="https://lists.blender.org/mailman/listinfo/soc-2018-dev" rel="noreferrer noreferrer" target="_blank">https://lists.blender.org/mailman/listinfo/soc-2018-dev</a><br>
</blockquote>
</div>
</div>
</div>
</body>
</html>