On Feb. 9, 2015, 12:59 a.m., Mark Gaiser wrote:
-1
You use C-Style casts. Oke, the frameworks coding style doesn't seem to
explicitly forbid it (casts are not mentioned), but if i recall correctly
we use the Qt style + some of our own which means we should obey the Qt
style [1]
On Feb. 9, 2015, 12:59 a.m., Mark Gaiser wrote:
-1
You use C-Style casts. Oke, the frameworks coding style doesn't seem to
explicitly forbid it (casts are not mentioned), but if i recall correctly
we use the Qt style + some of our own which means we should obey the Qt
style [1]
On Feb. 9, 2015, 12:59 a.m., Mark Gaiser wrote:
-1
You use C-Style casts. Oke, the frameworks coding style doesn't seem to
explicitly forbid it (casts are not mentioned), but if i recall correctly
we use the Qt style + some of our own which means we should obey the Qt
style [1]
---
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/122493/
---
Review request for KDE Frameworks and Marco Martin.
Repository:
---
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/122493/#review75648
---
-1
You use C-Style casts. Oke, the frameworks coding style
---
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/122493/#review75646
---
`qRound`?
- Kevin Funk
On Feb. 8, 2015, 11:12 p.m., Jeremy