Re: git commit: set the tracker_event to NULL if we cancelled the event.

[James Peach <jp...@apache.org>]

On Nov 19, 2013, at 11:52 PM, weijin@apache.org wrote:

> Updated Branches:
>  refs/heads/master de4c7d233 -> 898d55ff2
> 
> 
> set the tracker_event to NULL if we cancelled the event.

Where's the JIRA ticket for this commit?

> 
> 
> Project: http://git-wip-us.apache.org/repos/asf/trafficserver/repo
> Commit: http://git-wip-us.apache.org/repos/asf/trafficserver/commit/898d55ff
> Tree: http://git-wip-us.apache.org/repos/asf/trafficserver/tree/898d55ff
> Diff: http://git-wip-us.apache.org/repos/asf/trafficserver/diff/898d55ff
> 
> Branch: refs/heads/master
> Commit: 898d55ff248cb5b179bed07e3e59057f6a261bfc
> Parents: de4c7d2
> Author: weijin <ta...@taobao.com>
> Authored: Wed Nov 20 15:51:07 2013 +0800
> Committer: weijin <ta...@taobao.com>
> Committed: Wed Nov 20 15:51:07 2013 +0800
> 
> ----------------------------------------------------------------------
> proxy/signals.cc | 4 +++-
> 1 file changed, 3 insertions(+), 1 deletion(-)
> ----------------------------------------------------------------------
> 
> 
> http://git-wip-us.apache.org/repos/asf/trafficserver/blob/898d55ff/proxy/signals.cc
> ----------------------------------------------------------------------
> diff --git a/proxy/signals.cc b/proxy/signals.cc
> index 968c1ac..83f275d 100644
> --- a/proxy/signals.cc
> +++ b/proxy/signals.cc
> @@ -380,8 +380,10 @@ init_tracker(const char *config_var, RecDataT /* type ATS_UNUSED */, RecData dat
>   else
>     dump_mem_info_frequency = REC_ConfigReadInteger("proxy.config.dump_mem_info_frequency");
>   Debug("tracker", "init_tracker called [%d]\n", dump_mem_info_frequency);
> -  if (tracker_event)
> +  if (tracker_event) {
>     tracker_event->cancel();
> +    tracker_event = NULL;
> +  }
>   if (dump_mem_info_frequency > 0) {
>     tracker_event = eventProcessor.schedule_every(new TrackerContinuation,
>                                                   HRTIME_SECONDS(dump_mem_info_frequency), ET_CALL);
>