First line if pnote->expire is less then current_time and the board isn't got a short name of Stories
Both of those being true it will go in and free the note it just loaded (another words this board isn't stories and the note has expired).
If the note hasn't expired or the board is the stories board it will add the note to the list of notes for the board and continue.
The resave is only done if a note is removed from the board other wise the note is kept and no reason to resave the board.
Alright...I think I get it...but wouldn't we want it to read:
!str_cmp( board->short_name, "Stories" ) )
This way, it's looking for NOT short_name Stories? This would exclude short_name "Stories" from being freed, instead of including it.
And, to try and show that I understand with your suggested change...
We have pnote->expire time of 1 with current_time of 2 (for simplicities sake). pnote->expire < current_time = TRUE
We have a string comparison of board->short_name and "Stories" and the pnote is from the "Stories" board .... board->short_name = TRUE
With both TRUE, we execute the check:
We no longer have a note from the "Stories" board if it expires.
Did I get through that correctly? heh
For some idiotic notion, I was under the assumption this is where the note was being saved, not freed. *LOL* Thanks for spelling it out to me.