MouseSap linking broken (MSW, 32 bits)

[Miguel Gimenez]

Last revision to MouseSap (r12170) breaks linking in MSW 32 bits because MouseSap_wx31.cbp needs adding MouseSapCfg.cpp and MouseSapCfg.h

[Pecan]

Thanks
Fixed: rev. 12172

[Miguel Gimenez]

The four CBP files added in r12172 were removed some time ago, because their associated wxWidgets versions (wx2.8 and wx3.0 in MSW, wx2.8 in unix) are not supported anymore.

[Pecan]

Thanks
Hopefully corrected this time. Rev: 12173

[oBFusCATed]

Do we actually need this plugin for Linux? Does it do something useful there?

[Pecan]

Do we actually need this plugin for Linux? Does it do something useful there?

This plugin is actually not very useful to either Linux or Windows 10 anymore.
It adds the following:

If selected text & shift-middleMouse,                         paste selected text at current cursor position(like Linux).
If selected text & shift-middleMouse inside selection,  copy to clipboard(like ctrl-c).
If selected text & shift-switch editor,                          copy selection to clipboard.
If selected text & shift-ctrl-middleMouse inside selection,  overwrite selection with clipboard data.
If no selected text & Shift-middleMouse,                    paste clipboard data at cursor position(like ctrl-v).

It was very useful before windows 10 added the primary paste feature (like Linux).
If it's not needed anymore, and if it's permissible, I vote to remove it.

[oBFusCATed]

It was very useful before windows 10 added the primary paste feature (like Linux).

Really? They've done that everywhere? In all apps?

[Pecan]

It was very useful before windows 10 added the primary paste feature (like Linux).

Really? They've done that everywhere? In all apps?

no, I can't get it to work in notepad.
Edit: I must be wrong about this. I can't get Atom to middle-mouse paste either.

[oBFusCATed]

We have code in scintilla which does middle click handling, but it should be gtk only.
Does it work in textctrls and similar native controls in Code::Blocks?

[Pecan]

No, middleMouse (primary) paste does not work in wxTextCtrls on MSWindows.

I've made a very ignorant assumption.
I did not know about the addition of middleMouse code in wxScintilla.

But middleMouse pasting does work in CB on MSWindows using wxWidgets/wxScintilla.

[oBFusCATed]

This is rather strange.
The code in ScintillaWX::ClaimSelection and ScintillaWX::DoMiddleButtonUp is gtk only.
Can you test if this works in scite and the stc wxWidgets sample?

[Pecan]

This is rather strange.
The code in ScintillaWX::ClaimSelection and ScintillaWX::DoMiddleButtonUp is gtk only.
Can you test if this works in scite and the stc wxWidgets sample?

Pasting with middleMouse does NOT work in scite 4.4.3 nor does it work in wx3.1.3 stc sample (compiled with CB 12173).

But it does work within the CB editors (CB 12173) with all contrib plugins disabled.

I hope you're not considering disabling it in Windows. I've become addicted to it.

[oBFusCATed]

I hope you're not considering disabling it in Windows. I've become addicted to it.

As a first step we need to find which code does this, so we don't break it when we do an update to the scintilla code.
Could you try to use a debugger to find why it does work?

[Pecan]

As a first step we need to find which code does this, so we don't break it when we do an update to the scintilla code.
Could you try to use a debugger to find why it does work?

It's implemented in:
void cbStyledTextCtrl::OnMouseMiddleDown(wxMouseEvent& event)

[oBFusCATed]

Good, it is an option and it is off by default.

So what is the plan for the mouse sap plugin? Is it needed any more?