Re: [LyX/master] Add support for IM attribute specification over multiple segments

2026/06/14 19:59、Scott Kostyshak <[email protected]>のメール:

On Sun, Jun 14, 2026 at 11:13:31AM +0200, Kornel Benko wrote:
Am Sun, 14 Jun 2026 05:04:06 +0000
schrieb Koji Yokota <[email protected]>:

commit f198a3847c685e29565ffb952caf7870ba45bc60
Author: Koji Yokota <[email protected]>
Date:   Sun Jun 14 13:57:45 2026 +0900

   Add support for IM attribute specification over multiple segments

   Although QInputMethodEvent documents that there should be at most one
   text format attribute specified on each segment, the de-facto standard
   seems different. LyX allowed duplicate attribute specification segment
   by segment so far. This fix extends it to allow overwrapped
   specification over segments.

   Eg. existing attribute  [   seg   ] [charFormat for a segment] [seg]
       incoming attribute  [ <------ charFormat over segments ------> ]
---
src/frontends/qt/GuiInputMethod.cpp | 192 ++++++++++++++++++++++++++++--------
src/frontends/qt/GuiInputMethod.h   |   3 +-
src/frontends/qt/GuiPainter.cpp     |   4 +-
3 files changed, 155 insertions(+), 44 deletions(-)
...

Now I get the attached.

Interesting, I can't reproduce. Maybe a difference in versions?

$ g++ --version
g++ (Ubuntu 15-20250404-0ubuntu1) 15.0.1 20250404 (experimental) [master r15-9193-g08e803aa9be]

$ clang++ --version
Ubuntu clang version 20.1.2 (0ubuntu1)

Thanks. This seems a mistake anyway. Variable cf will not be a nullptr.

Should be fixed with commit 7d40274. As I also cannot get the error, please notify me if it is not fixed.

Koji

-- 
lyx-devel mailing list
[email protected]
https://lists.lyx.org/mailman/listinfo/lyx-devel