FLTK logo

Re: [fltk.coredev] Fl_Terminal selection_text() methods

FLTK matrix user chat room
(using Element browser app)   FLTK gitter user chat room   GitHub FLTK Project   FLTK News RSS Feed  
  FLTK Apps      FLTK Library      Forums      Links     Login 
 All Forums  |  Back to fltk.coredev  ]
 
Previous Message ]New Message | Reply ]Next Message ]

Re: Fl_Terminal selection_text() methods Greg Ercolano 12:50 Mar 12  
 

On 3/12/24 06:28, 'Albrecht Schlosser' via fltk.coredev wrote:

should the methods Fl_Terminal::selection_text_len() and Fl_Terminal::selection_text() be public rather than protected?
Currently they are protected. Was this intentional?

    While intentional, I can see making it public, if for no other purpose to be consistent
    with the other widgets that allow access to selections publicly.

    My thinking was that manipulation of this kind of thing would be primarily in the
    context of a derived class (such as implementing a popup menu to provide Copy/Paste),
    and therefore protected. Also, I would think someone doing this might also want access
    to the color/attribute information, which starts getting into the internals, as there's no
    "standard" way to access color information that I'd think would be safe to be public.

    Anyway, I'll change it to be public, as these methods work with simple text,
    devoid of color/attribute information. Commit forthcoming.

--
You received this message because you are subscribed to the Google Groups "fltk.coredev" group.
To unsubscribe from this group and stop receiving emails from it, send an email to fltkcoredev+unsubscribe@googlegroups.com.
To view this discussion on the web visit https://groups.google.com/d/msgid/fltkcoredev/3b1dd7f3-457c-4e6f-bb4b-73403c0de8b8%40seriss.com.
Direct Link to Message ]
 
     
Previous Message ]New Message | Reply ]Next Message ]
 
 

Comments are owned by the poster. All other content is copyright 1998-2024 by Bill Spitzak and others. This project is hosted by The FLTK Team. Please report site problems to 'erco@seriss.com'.