content-hub click hook takes forever

On amd64 running the content hub hook is very rapid:
$ /usr/lib/x86_64-linux-gnu/content-hub/content-hub-peer-hook
int main(int, char**)
void Hook::run()
Failed to install peer ""
virtual void Registry::enumerate_known_peers(const std::function<void(const com::ubuntu::content::Peer&)>&)
virtual void Registry::enumerate_known_peers(const std::function<void(const com::ubuntu::content::Peer&)>&) "music"
virtual void Registry::enumerate_known_peers(const std::function<void(const com::ubuntu::content::Peer&)>&) "pictures"
virtual void Registry::enumerate_known_peers(const std::function<void(const com::ubuntu::content::Peer&)>&) "documents"

On armhf/emulator it's much much slower...

valgrind --tool=callgrind callgraph.

Larsu, could you take a look at the content-hub hook in the content-hub? it's not doing much, yet seems to be very slow (e.g. especially under ubuntu emulator)

I'm confused as to what this hook is suppose to do. A declared pattern is Pattern: ${home}/.local/share/content-hub/${id}, yet there are no clicks that hook into that, which i can find ...

It's sitting in a poll loop - I don't think it's inefficiency, I think it's waiting for something that just isn't there.

On 4 March 2014 18:01, Colin Watson <email address hidden> wrote:
> It's sitting in a poll loop - I don't think it's inefficiency, I think
> it's waiting for something that just isn't there.
>

Hm, it does use a oneshot QTimer and slangasek was reporting that in
the arm-emulator, system clock is not always ticking. =/

--
Regards,

Dimitri.

This bug was fixed in the package content-hub - 0.0+14.04.20140402-0ubuntu1

---------------
content-hub (0.0+14.04.20140402-0ubuntu1) trusty; urgency=low

  [ Loïc Minier ]
  * Downgrade content-hub Recommends to a Suggests as we don't want
    content-hub pulled via ubuntu-sdk on developer desktops.

  [ Ken VanDine ]
  * protect against queries for Type::unknown (LP: #1293463)
  * Replace a couple missed qDebug lines for logging
  * Don't unref the g_icon, the value belongs to app
  * Ensure m_defaultSources is a StringList before attempting to use it
    as a list
  * ContentPeerPicker: Make headers translatable and use a
    ResponsiveGridView for the devices grid to match the apps grid.
    ContentPeerPicker: hide the devices section until we have a way to
    populate it

  [ Javier Collado ]
  * Added README file (LP: #1274899)

  [ Dimitri John Ledkov ]
  * Exit quicker, if there is nothing to do. (LP: #1287674) (LP:
    #1287674)

  [ Michael Sheldon ]
  * Fixes updating of the ContentPeerModel when using Loader and
    changing ContentType or ContentHandler dynamically and ensures that
    ContentPeerModel Loader doesn't begin loading the model until the
    peer picker becomes visible (avoiding delaying app start-up time).
  * Make qdoc based documentation compatible with the Qt 5.2 version of
    qdoc.
  * Fixes version number of import statements in QML documentation.
 -- Ubuntu daily release <email address hidden> Wed, 02 Apr 2014 14:46:54 +0000