<?xml version="1.0"?>
<?xml-stylesheet type="text/css" href="http://techbase.kde.org/skins/common/feed.css?0.2"?>
<feed xmlns="http://www.w3.org/2005/Atom" xml:lang="en">
		<id>http://techbase.kde.org/index.php?title=Talk:Policies/Kdelibs_Coding_Style&amp;feed=atom&amp;action=history</id>
		<title>Talk:Policies/Kdelibs Coding Style - Revision history</title>
		<link rel="self" type="application/atom+xml" href="http://techbase.kde.org/index.php?title=Talk:Policies/Kdelibs_Coding_Style&amp;feed=atom&amp;action=history"/>
		<link rel="alternate" type="text/html" href="http://techbase.kde.org/index.php?title=Talk:Policies/Kdelibs_Coding_Style&amp;action=history"/>
		<updated>2013-05-20T19:17:49Z</updated>
		<subtitle>Revision history for this page on the wiki</subtitle>
		<generator>MediaWiki 1.20.2</generator>

	<entry>
		<id>http://techbase.kde.org/index.php?title=Talk:Policies/Kdelibs_Coding_Style&amp;diff=38211&amp;oldid=prev</id>
		<title>Pkpatel88 at 06:01, 7 February 2009</title>
		<link rel="alternate" type="text/html" href="http://techbase.kde.org/index.php?title=Talk:Policies/Kdelibs_Coding_Style&amp;diff=38211&amp;oldid=prev"/>
				<updated>2009-02-07T06:01:11Z</updated>
		
		<summary type="html">&lt;p&gt;&lt;/p&gt;
&lt;table class='diff diff-contentalign-left'&gt;
				&lt;col class='diff-marker' /&gt;
				&lt;col class='diff-content' /&gt;
				&lt;col class='diff-marker' /&gt;
				&lt;col class='diff-content' /&gt;
			&lt;tr style='vertical-align: top;'&gt;
			&lt;td colspan='2' style=&quot;background-color: white; color:black;&quot;&gt;← Older revision&lt;/td&gt;
			&lt;td colspan='2' style=&quot;background-color: white; color:black;&quot;&gt;Revision as of 06:01, 7 February 2009&lt;/td&gt;
			&lt;/tr&gt;&lt;tr&gt;&lt;td colspan=&quot;2&quot; class=&quot;diff-lineno&quot;&gt;Line 9:&lt;/td&gt;
&lt;td colspan=&quot;2&quot; class=&quot;diff-lineno&quot;&gt;Line 9:&lt;/td&gt;&lt;/tr&gt;
&lt;tr&gt;&lt;td class='diff-marker'&gt;&amp;#160;&lt;/td&gt;&lt;td style=&quot;background: #eee; color:black; font-size: smaller;&quot;&gt;&lt;/td&gt;&lt;td class='diff-marker'&gt;&amp;#160;&lt;/td&gt;&lt;td style=&quot;background: #eee; color:black; font-size: smaller;&quot;&gt;&lt;/td&gt;&lt;/tr&gt;
&lt;tr&gt;&lt;td class='diff-marker'&gt;&amp;#160;&lt;/td&gt;&lt;td style=&quot;background: #eee; color:black; font-size: smaller;&quot;&gt;&lt;div&gt;::: indeed i did, as that was what was discussed and agreed upon in trysil. given that a majority of the active kdelibs developers were there, including probably our most central ones, i think that's good enough =) --[[User:Aseigo|Aseigo]] 23:01, 16 February 2007 (CET)&lt;/div&gt;&lt;/td&gt;&lt;td class='diff-marker'&gt;&amp;#160;&lt;/td&gt;&lt;td style=&quot;background: #eee; color:black; font-size: smaller;&quot;&gt;&lt;div&gt;::: indeed i did, as that was what was discussed and agreed upon in trysil. given that a majority of the active kdelibs developers were there, including probably our most central ones, i think that's good enough =) --[[User:Aseigo|Aseigo]] 23:01, 16 February 2007 (CET)&lt;/div&gt;&lt;/td&gt;&lt;/tr&gt;
&lt;tr&gt;&lt;td colspan=&quot;2&quot;&gt;&amp;#160;&lt;/td&gt;&lt;td class='diff-marker'&gt;+&lt;/td&gt;&lt;td style=&quot;background: #cfc; color:black; font-size: smaller;&quot;&gt;&lt;div&gt;&lt;ins style=&quot;color: red; font-weight: bold; text-decoration: none;&quot;&gt;&lt;/ins&gt;&lt;/div&gt;&lt;/td&gt;&lt;/tr&gt;
&lt;tr&gt;&lt;td colspan=&quot;2&quot;&gt;&amp;#160;&lt;/td&gt;&lt;td class='diff-marker'&gt;+&lt;/td&gt;&lt;td style=&quot;background: #cfc; color:black; font-size: smaller;&quot;&gt;&lt;div&gt;&lt;ins style=&quot;color: red; font-weight: bold; text-decoration: none;&quot;&gt;&lt;/ins&gt;&lt;/div&gt;&lt;/td&gt;&lt;/tr&gt;
&lt;tr&gt;&lt;td colspan=&quot;2&quot;&gt;&amp;#160;&lt;/td&gt;&lt;td class='diff-marker'&gt;+&lt;/td&gt;&lt;td style=&quot;background: #cfc; color:black; font-size: smaller;&quot;&gt;&lt;div&gt;&lt;ins style=&quot;color: red; font-weight: bold; text-decoration: none;&quot;&gt;Where it says &amp;quot;Kdelibs coding style follows the Qt 4 coding style.&amp;quot; There should be maybe a link that points out Qt 4 coding style. --[[User:Pkpatel88|Pkpatel88]] 23:59, 06 February (CST)&lt;/ins&gt;&lt;/div&gt;&lt;/td&gt;&lt;/tr&gt;
&lt;/table&gt;</summary>
		<author><name>Pkpatel88</name></author>	</entry>

	<entry>
		<id>http://techbase.kde.org/index.php?title=Talk:Policies/Kdelibs_Coding_Style&amp;diff=8039&amp;oldid=prev</id>
		<title>Aseigo at 22:01, 16 February 2007</title>
		<link rel="alternate" type="text/html" href="http://techbase.kde.org/index.php?title=Talk:Policies/Kdelibs_Coding_Style&amp;diff=8039&amp;oldid=prev"/>
				<updated>2007-02-16T22:01:09Z</updated>
		
		<summary type="html">&lt;p&gt;&lt;/p&gt;
&lt;table class='diff diff-contentalign-left'&gt;
				&lt;col class='diff-marker' /&gt;
				&lt;col class='diff-content' /&gt;
				&lt;col class='diff-marker' /&gt;
				&lt;col class='diff-content' /&gt;
			&lt;tr style='vertical-align: top;'&gt;
			&lt;td colspan='2' style=&quot;background-color: white; color:black;&quot;&gt;← Older revision&lt;/td&gt;
			&lt;td colspan='2' style=&quot;background-color: white; color:black;&quot;&gt;Revision as of 22:01, 16 February 2007&lt;/td&gt;
			&lt;/tr&gt;&lt;tr&gt;&lt;td colspan=&quot;2&quot; class=&quot;diff-lineno&quot;&gt;Line 7:&lt;/td&gt;
&lt;td colspan=&quot;2&quot; class=&quot;diff-lineno&quot;&gt;Line 7:&lt;/td&gt;&lt;/tr&gt;
&lt;tr&gt;&lt;td class='diff-marker'&gt;&amp;#160;&lt;/td&gt;&lt;td style=&quot;background: #eee; color:black; font-size: smaller;&quot;&gt;&lt;/td&gt;&lt;td class='diff-marker'&gt;&amp;#160;&lt;/td&gt;&lt;td style=&quot;background: #eee; color:black; font-size: smaller;&quot;&gt;&lt;/td&gt;&lt;/tr&gt;
&lt;tr&gt;&lt;td class='diff-marker'&gt;&amp;#160;&lt;/td&gt;&lt;td style=&quot;background: #eee; color:black; font-size: smaller;&quot;&gt;&lt;div&gt;::meanwhile aaron already changed it, wheee ;) --[[User:Dhaumann|Dhaumann]]&lt;/div&gt;&lt;/td&gt;&lt;td class='diff-marker'&gt;&amp;#160;&lt;/td&gt;&lt;td style=&quot;background: #eee; color:black; font-size: smaller;&quot;&gt;&lt;div&gt;::meanwhile aaron already changed it, wheee ;) --[[User:Dhaumann|Dhaumann]]&lt;/div&gt;&lt;/td&gt;&lt;/tr&gt;
&lt;tr&gt;&lt;td colspan=&quot;2&quot;&gt;&amp;#160;&lt;/td&gt;&lt;td class='diff-marker'&gt;+&lt;/td&gt;&lt;td style=&quot;background: #cfc; color:black; font-size: smaller;&quot;&gt;&lt;div&gt;&lt;ins style=&quot;color: red; font-weight: bold; text-decoration: none;&quot;&gt;&lt;/ins&gt;&lt;/div&gt;&lt;/td&gt;&lt;/tr&gt;
&lt;tr&gt;&lt;td colspan=&quot;2&quot;&gt;&amp;#160;&lt;/td&gt;&lt;td class='diff-marker'&gt;+&lt;/td&gt;&lt;td style=&quot;background: #cfc; color:black; font-size: smaller;&quot;&gt;&lt;div&gt;&lt;ins style=&quot;color: red; font-weight: bold; text-decoration: none;&quot;&gt;::: indeed i did, as that was what was discussed and agreed upon in trysil. given that a majority of the active kdelibs developers were there, including probably our most central ones, i think that's good enough =) --[[User:Aseigo|Aseigo]] 23:01, 16 February 2007 (CET)&lt;/ins&gt;&lt;/div&gt;&lt;/td&gt;&lt;/tr&gt;
&lt;/table&gt;</summary>
		<author><name>Aseigo</name></author>	</entry>

	<entry>
		<id>http://techbase.kde.org/index.php?title=Talk:Policies/Kdelibs_Coding_Style&amp;diff=8035&amp;oldid=prev</id>
		<title>Dhaumann: +e</title>
		<link rel="alternate" type="text/html" href="http://techbase.kde.org/index.php?title=Talk:Policies/Kdelibs_Coding_Style&amp;diff=8035&amp;oldid=prev"/>
				<updated>2007-02-15T22:36:17Z</updated>
		
		<summary type="html">&lt;p&gt;+e&lt;/p&gt;
&lt;table class='diff diff-contentalign-left'&gt;
				&lt;col class='diff-marker' /&gt;
				&lt;col class='diff-content' /&gt;
				&lt;col class='diff-marker' /&gt;
				&lt;col class='diff-content' /&gt;
			&lt;tr style='vertical-align: top;'&gt;
			&lt;td colspan='2' style=&quot;background-color: white; color:black;&quot;&gt;← Older revision&lt;/td&gt;
			&lt;td colspan='2' style=&quot;background-color: white; color:black;&quot;&gt;Revision as of 22:36, 15 February 2007&lt;/td&gt;
			&lt;/tr&gt;&lt;tr&gt;&lt;td colspan=&quot;2&quot; class=&quot;diff-lineno&quot;&gt;Line 6:&lt;/td&gt;
&lt;td colspan=&quot;2&quot; class=&quot;diff-lineno&quot;&gt;Line 6:&lt;/td&gt;&lt;/tr&gt;
&lt;tr&gt;&lt;td class='diff-marker'&gt;&amp;#160;&lt;/td&gt;&lt;td style=&quot;background: #eee; color:black; font-size: smaller;&quot;&gt;&lt;div&gt;:my pov: According to Zack's attachement this is not the case. As it makes sense to stick to the Qt style for the libs I'd say the current rule is fine. The attachemet also had the final rule '''Feel free to break a rule if it makes your code look bad'''. We should include it as well :) I did not and will not read the whole thread again ;) --[[User:Dhaumann|Dhaumann]] 23:30, 15 February 2007 (CET)&lt;/div&gt;&lt;/td&gt;&lt;td class='diff-marker'&gt;&amp;#160;&lt;/td&gt;&lt;td style=&quot;background: #eee; color:black; font-size: smaller;&quot;&gt;&lt;div&gt;:my pov: According to Zack's attachement this is not the case. As it makes sense to stick to the Qt style for the libs I'd say the current rule is fine. The attachemet also had the final rule '''Feel free to break a rule if it makes your code look bad'''. We should include it as well :) I did not and will not read the whole thread again ;) --[[User:Dhaumann|Dhaumann]] 23:30, 15 February 2007 (CET)&lt;/div&gt;&lt;/td&gt;&lt;/tr&gt;
&lt;tr&gt;&lt;td class='diff-marker'&gt;&amp;#160;&lt;/td&gt;&lt;td style=&quot;background: #eee; color:black; font-size: smaller;&quot;&gt;&lt;/td&gt;&lt;td class='diff-marker'&gt;&amp;#160;&lt;/td&gt;&lt;td style=&quot;background: #eee; color:black; font-size: smaller;&quot;&gt;&lt;/td&gt;&lt;/tr&gt;
&lt;tr&gt;&lt;td class='diff-marker'&gt;−&lt;/td&gt;&lt;td style=&quot;background: #ffa; color:black; font-size: smaller;&quot;&gt;&lt;div&gt;::meanwhile aaron &lt;del class=&quot;diffchange diffchange-inline&quot;&gt;alrady &lt;/del&gt;changed it, wheee ;) --[[User:Dhaumann|Dhaumann]]&lt;/div&gt;&lt;/td&gt;&lt;td class='diff-marker'&gt;+&lt;/td&gt;&lt;td style=&quot;background: #cfc; color:black; font-size: smaller;&quot;&gt;&lt;div&gt;::meanwhile aaron &lt;ins class=&quot;diffchange diffchange-inline&quot;&gt;already &lt;/ins&gt;changed it, wheee ;) --[[User:Dhaumann|Dhaumann]]&lt;/div&gt;&lt;/td&gt;&lt;/tr&gt;
&lt;/table&gt;</summary>
		<author><name>Dhaumann</name></author>	</entry>

	<entry>
		<id>http://techbase.kde.org/index.php?title=Talk:Policies/Kdelibs_Coding_Style&amp;diff=8034&amp;oldid=prev</id>
		<title>Dhaumann at 22:34, 15 February 2007</title>
		<link rel="alternate" type="text/html" href="http://techbase.kde.org/index.php?title=Talk:Policies/Kdelibs_Coding_Style&amp;diff=8034&amp;oldid=prev"/>
				<updated>2007-02-15T22:34:15Z</updated>
		
		<summary type="html">&lt;p&gt;&lt;/p&gt;
&lt;table class='diff diff-contentalign-left'&gt;
				&lt;col class='diff-marker' /&gt;
				&lt;col class='diff-content' /&gt;
				&lt;col class='diff-marker' /&gt;
				&lt;col class='diff-content' /&gt;
			&lt;tr style='vertical-align: top;'&gt;
			&lt;td colspan='2' style=&quot;background-color: white; color:black;&quot;&gt;← Older revision&lt;/td&gt;
			&lt;td colspan='2' style=&quot;background-color: white; color:black;&quot;&gt;Revision as of 22:34, 15 February 2007&lt;/td&gt;
			&lt;/tr&gt;&lt;tr&gt;&lt;td colspan=&quot;2&quot; class=&quot;diff-lineno&quot;&gt;Line 5:&lt;/td&gt;
&lt;td colspan=&quot;2&quot; class=&quot;diff-lineno&quot;&gt;Line 5:&lt;/td&gt;&lt;/tr&gt;
&lt;tr&gt;&lt;td class='diff-marker'&gt;&amp;#160;&lt;/td&gt;&lt;td style=&quot;background: #eee; color:black; font-size: smaller;&quot;&gt;&lt;/td&gt;&lt;td class='diff-marker'&gt;&amp;#160;&lt;/td&gt;&lt;td style=&quot;background: #eee; color:black; font-size: smaller;&quot;&gt;&lt;/td&gt;&lt;/tr&gt;
&lt;tr&gt;&lt;td class='diff-marker'&gt;&amp;#160;&lt;/td&gt;&lt;td style=&quot;background: #eee; color:black; font-size: smaller;&quot;&gt;&lt;div&gt;:my pov: According to Zack's attachement this is not the case. As it makes sense to stick to the Qt style for the libs I'd say the current rule is fine. The attachemet also had the final rule '''Feel free to break a rule if it makes your code look bad'''. We should include it as well :) I did not and will not read the whole thread again ;) --[[User:Dhaumann|Dhaumann]] 23:30, 15 February 2007 (CET)&lt;/div&gt;&lt;/td&gt;&lt;td class='diff-marker'&gt;&amp;#160;&lt;/td&gt;&lt;td style=&quot;background: #eee; color:black; font-size: smaller;&quot;&gt;&lt;div&gt;:my pov: According to Zack's attachement this is not the case. As it makes sense to stick to the Qt style for the libs I'd say the current rule is fine. The attachemet also had the final rule '''Feel free to break a rule if it makes your code look bad'''. We should include it as well :) I did not and will not read the whole thread again ;) --[[User:Dhaumann|Dhaumann]] 23:30, 15 February 2007 (CET)&lt;/div&gt;&lt;/td&gt;&lt;/tr&gt;
&lt;tr&gt;&lt;td colspan=&quot;2&quot;&gt;&amp;#160;&lt;/td&gt;&lt;td class='diff-marker'&gt;+&lt;/td&gt;&lt;td style=&quot;background: #cfc; color:black; font-size: smaller;&quot;&gt;&lt;div&gt;&lt;ins style=&quot;color: red; font-weight: bold; text-decoration: none;&quot;&gt;&lt;/ins&gt;&lt;/div&gt;&lt;/td&gt;&lt;/tr&gt;
&lt;tr&gt;&lt;td colspan=&quot;2&quot;&gt;&amp;#160;&lt;/td&gt;&lt;td class='diff-marker'&gt;+&lt;/td&gt;&lt;td style=&quot;background: #cfc; color:black; font-size: smaller;&quot;&gt;&lt;div&gt;&lt;ins style=&quot;color: red; font-weight: bold; text-decoration: none;&quot;&gt;::meanwhile aaron alrady changed it, wheee ;) --[[User:Dhaumann|Dhaumann]]&lt;/ins&gt;&lt;/div&gt;&lt;/td&gt;&lt;/tr&gt;
&lt;/table&gt;</summary>
		<author><name>Dhaumann</name></author>	</entry>

	<entry>
		<id>http://techbase.kde.org/index.php?title=Talk:Policies/Kdelibs_Coding_Style&amp;diff=8032&amp;oldid=prev</id>
		<title>Dhaumann: my pov</title>
		<link rel="alternate" type="text/html" href="http://techbase.kde.org/index.php?title=Talk:Policies/Kdelibs_Coding_Style&amp;diff=8032&amp;oldid=prev"/>
				<updated>2007-02-15T22:30:11Z</updated>
		
		<summary type="html">&lt;p&gt;my pov&lt;/p&gt;
&lt;table class='diff diff-contentalign-left'&gt;
				&lt;col class='diff-marker' /&gt;
				&lt;col class='diff-content' /&gt;
				&lt;col class='diff-marker' /&gt;
				&lt;col class='diff-content' /&gt;
			&lt;tr style='vertical-align: top;'&gt;
			&lt;td colspan='2' style=&quot;background-color: white; color:black;&quot;&gt;← Older revision&lt;/td&gt;
			&lt;td colspan='2' style=&quot;background-color: white; color:black;&quot;&gt;Revision as of 22:30, 15 February 2007&lt;/td&gt;
			&lt;/tr&gt;&lt;tr&gt;&lt;td colspan=&quot;2&quot; class=&quot;diff-lineno&quot;&gt;Line 3:&lt;/td&gt;
&lt;td colspan=&quot;2&quot; class=&quot;diff-lineno&quot;&gt;Line 3:&lt;/td&gt;&lt;/tr&gt;
&lt;tr&gt;&lt;td class='diff-marker'&gt;&amp;#160;&lt;/td&gt;&lt;td style=&quot;background: #eee; color:black; font-size: smaller;&quot;&gt;&lt;div&gt;I thought we decided that we always want curly braces, no?&lt;/div&gt;&lt;/td&gt;&lt;td class='diff-marker'&gt;&amp;#160;&lt;/td&gt;&lt;td style=&quot;background: #eee; color:black; font-size: smaller;&quot;&gt;&lt;div&gt;I thought we decided that we always want curly braces, no?&lt;/div&gt;&lt;/td&gt;&lt;/tr&gt;
&lt;tr&gt;&lt;td class='diff-marker'&gt;&amp;#160;&lt;/td&gt;&lt;td style=&quot;background: #eee; color:black; font-size: smaller;&quot;&gt;&lt;div&gt;--[[User:Mkretz|Mkretz]] 21:19, 15 February 2007 (CET)&lt;/div&gt;&lt;/td&gt;&lt;td class='diff-marker'&gt;&amp;#160;&lt;/td&gt;&lt;td style=&quot;background: #eee; color:black; font-size: smaller;&quot;&gt;&lt;div&gt;--[[User:Mkretz|Mkretz]] 21:19, 15 February 2007 (CET)&lt;/div&gt;&lt;/td&gt;&lt;/tr&gt;
&lt;tr&gt;&lt;td colspan=&quot;2&quot;&gt;&amp;#160;&lt;/td&gt;&lt;td class='diff-marker'&gt;+&lt;/td&gt;&lt;td style=&quot;background: #cfc; color:black; font-size: smaller;&quot;&gt;&lt;div&gt;&lt;ins style=&quot;color: red; font-weight: bold; text-decoration: none;&quot;&gt;&lt;/ins&gt;&lt;/div&gt;&lt;/td&gt;&lt;/tr&gt;
&lt;tr&gt;&lt;td colspan=&quot;2&quot;&gt;&amp;#160;&lt;/td&gt;&lt;td class='diff-marker'&gt;+&lt;/td&gt;&lt;td style=&quot;background: #cfc; color:black; font-size: smaller;&quot;&gt;&lt;div&gt;&lt;ins style=&quot;color: red; font-weight: bold; text-decoration: none;&quot;&gt;:my pov: According to Zack's attachement this is not the case. As it makes sense to stick to the Qt style for the libs I'd say the current rule is fine. The attachemet also had the final rule '''Feel free to break a rule if it makes your code look bad'''. We should include it as well :) I did not and will not read the whole thread again ;) --[[User:Dhaumann|Dhaumann]] 23:30, 15 February 2007 (CET)&lt;/ins&gt;&lt;/div&gt;&lt;/td&gt;&lt;/tr&gt;
&lt;/table&gt;</summary>
		<author><name>Dhaumann</name></author>	</entry>

	<entry>
		<id>http://techbase.kde.org/index.php?title=Talk:Policies/Kdelibs_Coding_Style&amp;diff=8028&amp;oldid=prev</id>
		<title>Mkretz at 20:19, 15 February 2007</title>
		<link rel="alternate" type="text/html" href="http://techbase.kde.org/index.php?title=Talk:Policies/Kdelibs_Coding_Style&amp;diff=8028&amp;oldid=prev"/>
				<updated>2007-02-15T20:19:49Z</updated>
		
		<summary type="html">&lt;p&gt;&lt;/p&gt;
&lt;p&gt;&lt;b&gt;New page&lt;/b&gt;&lt;/p&gt;&lt;div&gt;The document says: &amp;quot;Use curly braces when the body of a conditional statement contains more than one line, and also if a single line statement is somewhat complex.&amp;quot;&lt;br /&gt;
&lt;br /&gt;
I thought we decided that we always want curly braces, no?&lt;br /&gt;
--[[User:Mkretz|Mkretz]] 21:19, 15 February 2007 (CET)&lt;/div&gt;</summary>
		<author><name>Mkretz</name></author>	</entry>

	</feed>