Closed GoogleCodeExporter closed 9 years ago
One more minor thing..
In do_item_get() function,
following code is used to check whether the fetched item is valid.
if (it != NULL) {
if (settings.oldest_live != 0 && settings.oldest_live <= current_time &&
it->time <= settings.oldest_live) {
do_item_unlink(it, hv);
do_item_remove(it);
it = NULL;
if (was_found) {
fprintf(stderr, " -nuked by flush");
}
} else if (it->exptime != 0 && it->exptime <= current_time) {
do_item_unlink(it, hv);
do_item_remove(it);
it = NULL;
if (was_found) {
fprintf(stderr, " -nuked by expire");
}
} else {
it->it_flags |= ITEM_FETCHED;
DEBUG_REFCNT(it, '+');
}
}
In the above code,
"it->time <= settings.oldest_live" is used
instead of "it->time < settings.oldest_live".
So, it might be better to use the same condition in the code of do_item_alloc().
Original comment by jhpark...@gmail.com
on 2 Feb 2012 at 3:07
ah I mean to file a new bug in the future. I've fixed your reported bug and
we've already released 1.4.12.
does 1.4.12 look okay to you?
Original comment by dorma...@rydia.net
on 2 Feb 2012 at 3:13
Yes.. 1.4.12 is okay.
Original comment by jhpark...@gmail.com
on 2 Feb 2012 at 4:11
thanks :P Resolving!
Original comment by dorma...@rydia.net
on 2 Feb 2012 at 6:43
Original issue reported on code.google.com by
jhpark...@gmail.com
on 2 Feb 2012 at 3:07