Feature request: suggestion for cPlugin

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Martin Wache wrote:
> In my opinion the method CallPluginExtension() is not needed, the same
> can easily done by just calling
> cPluginManager::GetPlugin("name")->ExtensionInterface().

Why didn't I see that before? Obviously, this call also misses an
important NULL test, so your way blows up to:

cPlugin *p=cPluginManager::GetPlugin("name");
if (p) p->ExtensionInterface(...);

In the end, these functions are just convenient helper functions. All
these functions including GetPlugin("Name") can be substituted just by
using GetPlugin(int) and a few more lines of code.

Cheers,

Udo


[Index of Archives]     [Linux Media]     [Asterisk]     [DCCP]     [Netdev]     [Xorg]     [Util Linux NG]     [Xfree86]     [Big List of Linux Books]     [Fedora Users]     [Fedora Women]     [ALSA Devel]     [Linux USB]

  Powered by Linux