On 28/08/2014 16:21, Nick Treleaven wrote:
Code that uses doc->is_valid or DOC_VALID() outside of iterating documents_array probably should be using document_find_by_id() instead, because closed document pointers only stay invalid until another document needs to be opened.
Additional: if there is no chance of another document having been opened since, but there is a chance a document was closed, it should be fine to check doc->is_valid.