From 23a650b40a03d4dc8f0742bfeb52657c0c4124ab Mon Sep 17 00:00:00 2001 From: ParadoxV5 Date: Sun, 24 May 2026 16:45:07 -0600 Subject: [PATCH] Fix operator typo in `||` chain MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit There is a `,` breaking `Table_map_log_event::write_data_body()`’s `||` chain. This “comma operator” still connects the expressions, but if a step in the first sub-chain fails, the code will disregard the failure and continue with the second half-chain. This change was originally **unsolicitedly** suggested by Gemini; that is, I did not activate Gemini nor provide any prompt, but rather the CI-triggered Gemini comment came to me like junk mail. By verifying the bot output and creating this patch independently *without AI assistance*, I confirm I understand the change and can claim its authorship and responsibility, and *I remain have never provided AI-generated code*. Co-authored-by: gemini-code-assist <200291788+gemini-code-assist@users.noreply.github.com> --- sql/log_event_server.cc | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/sql/log_event_server.cc b/sql/log_event_server.cc index 2bc64a5b3a1fd..27be35ca2530b 100644 --- a/sql/log_event_server.cc +++ b/sql/log_event_server.cc @@ -6742,7 +6742,7 @@ bool Table_map_log_event::write_data_body() write_data(cbuf, (size_t) (cbuf_end - cbuf)) || write_data(m_coltype, m_colcnt) || write_data(mbuf, (size_t) (mbuf_end - mbuf)) || - write_data(m_field_metadata, m_field_metadata_size), + write_data(m_field_metadata, m_field_metadata_size) || write_data(m_null_bits, (m_colcnt + 7) / 8) || write_data((const uchar*) m_metadata_buf.ptr(), m_metadata_buf.length());