| Age | Commit message (Collapse) | Author |
|---|
|
It's currently impractical to use out-of-tree plugins that want to be
distributed via distribution channels.
Poezio now looks for every entry point registered in the
`poezio_plugins` entry group, and will use the first matching module
with the specified name.
This also helps specifically for the OMEMO plugin that has a conflicting
name (omemo / omemo) with the backend library.
Thanks jonas for pointing this out.
Signed-off-by: Maxime “pep” Buquet <pep@bouah.net>
|
|
And do not rely on the "module name" to index the plugins, use the
intended plugin name instead.
|
|
Introduce the concept of priority for event handlers, instead of the
position parameter.
The new `priority` parameter replacing `position` should be an integer
between 0 and 100. It defaults to 50.
The previous `position` parameter was only used to insert at a certain
position in the list of handlers (for this particular event). No
reference of it was kept, which means that it was not possible to ensure
an event was called in the position is was supposed to be.
I am now using per-event dicts, containing priority buckets (lists) of
handlers. I am using OrderedDicts to make it easier to loop through all
of the handlers, as insertion happens less often than reading.
I was also suggested using bisect with a simple list of tuples
(priority, handler) per event, but bisect tries to compare bound
methods, which is obviously not possible. Maybe it would be interesting
to find a way use this method instead of OrderedDict as that might be
less resource consuming.
This said, I don't think this part of poezio is a bottleneck at all, so
maybe this is just fine as is.
Signed-off-by: Maxime “pep” Buquet <pep@bouah.net>
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
Which were being filtered by the information popup filter
|
|
unused variables, imports, name shadowing
|
|
Use a placeholder object that can run it afterwards, so that we don’t
have side effects inside those functions.
|
|
|
|
|
|
|
|
|
|
|
|
|
