Am 11.02.2016 um 23:33 schrieb Colomban Wendling:<br>
>     Now, I didn't include the code/script to generate the<br>
>     scintilla_object_* methods out of sci_* in this PR (it's another<br>
>     small python script, and uses *attribute*((alias)) as I've failed to<br>
>     find a better solution yet). Therefore the scintilla_object_*<br>
>     functions are missing too.<br>
><br>
> It'd be nice it it was possible to user |rename-to| annotation, but<br>
> apparently that only works if the target name does exist…<br>
<br>
Yes, rename-to isn't suitable. I've tried...<br>
<br>
><br>
>     […] I eventually plan to fully implement the scintilla_object_*<br>
>     stuff […]<br>
><br>
> what do you mean?<br>
<br>
I plan to find an acceptable solution to provide sci_ as methods for <br>
ScintillaObject (as far as gir is concerned), and implement it so<br>
that Geany can ship these methods.<br>
<br>
><br>
>     (A separate header in order to generate a separate<br>
>     GeanyScintilla-1.0.gir and GeanyScintilla-1.0.typelib. So that you<br>
>     import GeanyScintilla without importing Geany)<br>
><br>
> Hum… why do you want this separation?<br>
><br>
<br>
IMO it makes sense to split out GeanyScintilla from a design pov. <br>
scintilla adds so many symbols which can be saved if you don't need <br>
scintilla stuff (think of "from gi.repository import Geany" not pulling <br>
lots of scintilla symbols).<br>
<br>
Also it allows scintilla to be used without geany parts. I expect that <br>
could become common for peasy sub-plugins, which want to use the <br>
GeanyScintilla module (assuming they mess with edited text), but not <br>
Geany because Peasy provides the preffered API wrappers.<br>
<br>
At the end of the day it doesn't make a huge difference, but it's my <br>
personal preference to do the split.<br>
<br>


<p style="font-size:small;-webkit-text-size-adjust:none;color:#666;">—<br>Reply to this email directly or <a href="https://github.com/geany/geany/pull/890#issuecomment-183978981">view it on GitHub</a>.<img alt="" height="1" src="https://github.com/notifications/beacon/ABDrJ2Bvo2lT3MtnFtf-6UOKTwPajTG1ks5pkOaDgaJpZM4HM9VD.gif" width="1" /></p>
<div itemscope itemtype="http://schema.org/EmailMessage">
<div itemprop="action" itemscope itemtype="http://schema.org/ViewAction">
  <link itemprop="url" href="https://github.com/geany/geany/pull/890#issuecomment-183978981"></link>
  <meta itemprop="name" content="View Pull Request"></meta>
</div>
<meta itemprop="description" content="View this Pull Request on GitHub"></meta>
</div>