Is passing by reference to thread here bad practice?

Question

This is the code listening from one C++ book.

void edit_document(std::string const& filename)
{
    open_document_and_display_gui(filename);
    while(!do