Prevent deletion of buffers while creating new windows

`kak -n -E 'hook global WinCreate .* %{ delete-buffer }'` was crashing
because we would delete the buffer during window construction, which
would not be able to delete the window as it was not fully constructed
and registered yet. This led to a window referencing a deleted buffer.

Fixing this by deleting the window later on failed because we can enter
an infinite loop where we constantly create a new *scratch* buffer, then
a window to display it, which deletes that buffer.

Make it an error to try to delete a buffer while a new window is
being setup by adding a Locked flag to buffers and checking that in
BufferManager::delete_buffer

Fixes #5311

1 files changed, 1 insertions, 0 deletions

diff --git a/src/buffer.hh b/src/buffer.hh
index 3a885ec8..2bcd8cc3 100644
--- a/src/buffer.hh
+++ b/src/buffer.hh
@@ -121,6 +121,7 @@ public:
         Debug    = 1 << 5,
         ReadOnly = 1 << 6,
         NoBufSetOption = 1 << 7,
+        Locked = 1 << 8,
     };
     friend constexpr bool with_bit_ops(Meta::Type<Flags>) { return true; }
     friend class BufferIterator;