[Asterisk-code-review] res calendar: delete old calendars after reload (asterisk[master])

[Mark Michelson]

Mark Michelson has posted comments on this change. ( https://gerrit.asterisk.org/4683 )

Change subject: res_calendar: delete old calendars after reload
......................................................................


Patch Set 1: Code-Review-1

(1 comment)

I have just one minor finding. Other than that, this looks good to me.

https://gerrit.asterisk.org/#/c/4683/1/res/res_calendar.c
File res/res_calendar.c:

Line 412: 		cal->pending_deletion = 1;
This line is not necessary. All calendars already have pending_deletion set to 1 during a reload.


-- 
To view, visit https://gerrit.asterisk.org/4683
To unsubscribe, visit https://gerrit.asterisk.org/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: Ic17fba9371c5a8b26a6bc54ea4957c13a32a343e
Gerrit-PatchSet: 1
Gerrit-Project: asterisk
Gerrit-Branch: master
Gerrit-Owner: Martin Tomec <tomec.martin at gmail.com>
Gerrit-Reviewer: Anonymous Coward #1000019
Gerrit-Reviewer: Mark Michelson <mmichelson at digium.com>
Gerrit-HasComments: Yes