On 2009-07-19 00:16:17, Omer F. USTA wrote:
Hi , just to learn why or instead of common || ? in
if (m_bottom_right == 1 or m_bottom_left == 1) {
Sujith H wrote:
I changed it to '||'.
This looks committed with r999070 and r999083
it will be nice to mark as submitted
- Omer F.
On Sunday 19 July 2009, Michael Jansen wrote:
On Sunday 19 July 2009 02:55:06 Sebastian Kügler wrote:
On Sunday 19 July 2009 02:16:13 Omer F. USTA wrote:
Hi , just to learn why or instead of common || ? in
if (m_bottom_right == 1 or m_bottom_left == 1) {
I personally find or more
On 2009-07-18 18:23:09, Anne-Marie Mahfouf wrote:
I tested and indeed it works fine and it fixes the corner problem. It's a
ship it and backport it from me.
Thanks for testing it :)
- Sujith
---
This is an automatically generated
On 2009-07-19 00:16:17, Omer F. USTA wrote:
Hi , just to learn why or instead of common || ? in
if (m_bottom_right == 1 or m_bottom_left == 1) {
I changed it to '||'.
- Sujith
---
This is an automatically generated e-mail. To
On Sunday 19 July 2009 02:55:06 Sebastian Kügler wrote:
On Sunday 19 July 2009 02:16:13 Omer F. USTA wrote:
Hi , just to learn why or instead of common || ? in
if (m_bottom_right == 1 or m_bottom_left == 1) {
I personally find or more readable, though I think || is more common in
KDE
---
This is an automatically generated e-mail. To reply, visit:
http://reviewboard.kde.org/r/1054/#review1653
---
I tested and indeed it works fine and it fixes the corner problem.
On Sunday 19 July 2009 02:16:13 Omer F. USTA wrote:
Hi , just to learn why or instead of common || ? in
if (m_bottom_right == 1 or m_bottom_left == 1) {
I personally find or more readable, though I think || is more common in KDE
code.
Maybe there are some compiler intricacies to it.
---
This is an automatically generated e-mail. To reply, visit:
http://reviewboard.kde.org/r/1054/#review1655
---
Hi , just to learn why or instead of common || ? in
if
---
This is an automatically generated e-mail. To reply, visit:
http://reviewboard.kde.org/r/1054/
---
Review request for Plasma, Aaron Seigo and Sebastian Kügler.
Summary
---
---
This is an automatically generated e-mail. To reply, visit:
http://reviewboard.kde.org/r/1054/
---
(Updated 2009-07-17 23:37:46.001490)
Review request for Plasma, Aaron Seigo,
10 matches
Mail list logo