At 05:32 AM 5/3/2006, Joe Orton wrote:
> + me->cnt_max = max_threads;
> + me->idle_max = init_threads;
> + rv = apr_thread_mutex_create(&me->lock,
APR_THREAD_MUTEX_NESTED,
> + me->pool);
> + if (APR_SUCCESS != rv) {
Personally I find the "constant != variable" style very distracting,
not sure if this is a widely held view...
FYI, I use it almost exclusively, because if you mean to type
if (APR_SUCCESS == rv)
and instead you type
if (APR_SUCCESS = rv)
the compiler will catch it for you. If you normally do variable ==
constant instead, you have no such protection.