Ditto. The last time I wondered about this, I grepped through the
code and found [1] to be the most prevalent. So, in code reviews I
have been recommending people do [1].
I think that's the nicest style (just like a variable, no special
prefix). Let's go with that and update the coding
On Jul 8, 2009, at 11:08 PM, KwangYul Seo wrote:
I found another style which starts with k.
[4] platform/chromium/PopupMenuChromium.cpp
static const int kMaxVisibleRows = 20;
static const int kMaxHeight = 500;
static const int kBorderSize = 1;
static const TimeStamp kTypeAheadTimeoutMs =
If we're voting, then +1 to [1]. If we're not voting, then oops.
:DG
On Thu, Jul 9, 2009 at 12:50 PM, Darin Adlerda...@apple.com wrote:
On Jul 8, 2009, at 9:08 PM, KwangYul Seo wrote:
I found another style which starts with k.
[4] platform/chromium/PopupMenuChromium.cpp
static const int
Ditto. The last time I wondered about this, I grepped through the code and
found [1] to be the most prevalent. So, in code reviews I have been
recommending people do [1].
-Darin
On Thu, Jul 9, 2009 at 8:16 PM, Dimitri Glazkov dglaz...@chromium.orgwrote:
If we're voting, then +1 to [1]. If
On Jul 9, 2009, at 8:55 PM, Darin Fisher wrote:
Ditto. The last time I wondered about this, I grepped through the
code and found [1] to be the most prevalent. So, in code reviews I
have been recommending people do [1].
I think that's the nicest style (just like a variable, no special
Hi,
It seems that there are three coding styles regarding static const int
constants.
[1] rendering/RenderImage.cpp
static const int maxAltTextWidth = 1024;
static const int maxAltTextHeight = 256;
[2] rendering/RenderVideo.cpp (prefixed with c)
static const int cDefaultWidth = 300;
static
I found another style which starts with k.
[4] platform/chromium/PopupMenuChromium.cpp
static const int kMaxVisibleRows = 20;
static const int kMaxHeight = 500;
static const int kBorderSize = 1;
static const TimeStamp kTypeAheadTimeoutMs = 1000;
2009/7/9 KwangYul Seo kwangyul@gmail.com
7 matches
Mail list logo
