Fix audio and video capabilities

Fix setting capabilities when Tp returns a null set of capabilities

Merge branch 'master' into fixes3

Conflicts:
folks/individual.vala

Merge remote branch 'pwith/favourites'

(Fixed)
Conflicts:
folks/individual.vala
folks/persona.vala

Don't declare IndividualAggregator's constructor as throwing an error, since it doesn't.

Merge remote branch 'pwith/docs2'

Fixed the documented return value for Presence.typecmp from docs2.

(Fixed)
Conflicts:
folks/individual-aggregator.vala
folks/individual.vala

Make create_personas_from_channel_handles_async actually async.

Bump the required version of GLib so we can take advantage of automatic ref counting for various GLib classes.

Use get_object_for_name_owner() rather than just get_object()

This makes the Logger proxy more robust against the Logger service going
away.

Make Individual.avatar.set private

Avatars are provided by the contacts; we can't set them ourselves.

Implement a FIXME about checking for all-whitespace aliases

Add capabilities support to TpfPersona

Catch some unhandled errors

Split the Logger interface out into a separate file

Split the LoggerIface we use to wrap the telepathy-logger D-Bus service out
into a separate file, and implement it in a new Logger class which shares the
same D-Bus object between all users of the class.

First round of favourites review cleanups

Change IndividualAggregator.members to IndividualAggregator.individuals for consistency with PersonaStore, etc.

Rephrase explanation of Individual:removed signal.

Add documentation for every public signal, property and member

Remove an outdated comment

Add Favourites support

This adds a Favourite interface, implemented by Individual and Persona, and
backed by a D-Bus connection to the telepathy-logger service in
TpfPersonaStore.

The D-Bus logger proxy is a custom-written one in Vala, as the logger proxy
doesn't exist in telepathy-glib at the moment.

Whenever the is-favourite property of an Individual is changed, its new value
is propagated to _all_ of the individual's Personas. Conversely, if a single
Persona has is-favourite == true, the Individual containing that Persona will
also have is-favourite == true.

Merge remote branch 'pwith/fixes2'

(Fixed)
Conflicts:
backends/telepathy/tpf-persona-store.vala

Merge remote branch 'pwith/docs'

(Fixed)
Conflicts:
folks/individual-aggregator.vala
folks/persona-store.vala

Micro version bump for new functionality.

Re-add the stored list, as it's necessary to properly remove contacts.

Support contact removal

Rename internal-only 'remove' functions in Tpf.PersonaStore so public 'remove' functions consistently mean 'remove permanently from the server'.

Add a comment about the usage of IndividiualAggregator.add_persona_from_details().parent

Cut out 'stored' contact list support; if a contact is in neither 'publish' nor 'subscribe', we don't want to present them.

Remove personas for Telepathy contacts removed from both the subscribe and publish contact lists.

Filter incoming group channel members through the standard contact list channels

Add support for the 'stored' standard contact list channel.

Don't be excessively-explicit about the HashMap hash/equal functions, since they're determined by the generic types.

Avoid another potential group/non-group contact list channel hash collision.

Handle the standard non-group contact list channels separately from each other and the group channels, as they need to be.

Change the key for channel_group_personas_map to Tp.Channel to avoid potential collisions between the ContactList and Group channels.

Change the key for channel_group_incoming_adds to Tp.Channel to avoid potential collisions between the ContactList and Group channels.

Use a literal GError when proxying errors in the TpfPersonaStore

Note about a Vala bug when chaining constructors which affects us

Make BackendStore.load_backends() async

It does I/O, so should be async so that it doesn't block. Consequently, it's
no longer guaranteed that all backends/personas/individuals are loaded by the
time IndividualAggegator's constructor returns.

Documentation fixes

Remove the redundant documentation of enum members and improve the
documentation of a few classes.

Clean up the distinction between the standard 'subscribe', 'publish' contact list channels and arbitrary group channels.

Add the ability to add new personas to a persona store.

Have each Individual use the presence message for one of its most-online personas rather than the first message found.

Cut idea for unncessary signal.

Merge remote branch 'pwith/fixes'

Add documentation support and stub documentation to Folks

Add valadoc documentation support in the build system, and add stub
documentation to the top level of all the classes in the Folks namespace, but
not the Folks.Backends namespace.

The documentation is generated in HTML format in docs/folks-0.1/ but isn't
distributed in the tarball.

gcc warning fixes in tp-lowlevel.[ch]

Ensure the individual is removed if all its persona list property is emptied

This was already the case when existing personas were removed from the
individual due to their source going away. This commit fixes it for when the
persona list property is changed directly.

Use a persona's UID as an individual's alias if no personas have an alias

Make Individual.get_presence_message() return an unowned string

This prevents a completely unnecessary string copy.

Remove system m4 macros from the m4 directory

Add them to a .gitignore file instead.

Add some comments to the code

Relocate folks_tp_lowlevel_connection_open_contact_list_channel_finish() next to its _async() function

Use a hash table for tracking new incoming TpfPersonas and only add them if they're new.

Clean up Telepathy PersonaStore removal and addition. This fixes a regression where a Telepathy persona store wouldn't emit 'removed' even if its account were disabled

Use consistent naming and signatures for the Tpf.Persona property-notification-passing helper functions.

Generalize overly-specific 'using' statement

Trivial clean-up.

Support avatars (generally, and for the Telepathy backend)

Fix the location of libfolks-telepathy in its -uninstalled pkgconfig file.

Bump the version number for the major changes of the last few commits.

Add .deps files for the public libraries.

Rename the folks-telepathy files, since they're in a different namespace than the folks-backend-telepathy files

Add proper namespaces to the Telepathy backend and support libraries

Rename liblowlevel to libtp-lowlevel for clarity

Split the telepathy backend into a support library and a thin backend
wrapper (so applications can use the support library without linking against the
backend)

Factor out the Telepathy backend into a separate, loadable module.

Add tp-glib Vala bindings dependency to the .pc file and add a -uninstalled.pc file

Support writing contact group changes back to Telepathy accounts.

Don't try to process accounts which are currently being disconnected.

Set a proper internal ID for TpPersonas and throw an error if its basis TpContact has an invalid UID.

Add group notification/read support for Telepathy personas

Avoid extraneous property notifications.

Cut out debugging default value.

Forward along TpPersona presence changes through their parent Individual.

Cut outdated comment.

Support account deletion

Cut outdated comments

Actually add the presence source.

Update for the more accurate signature for Tp.Account.status_changed.

Add stub Presence interface.

Add a stub for the Capabilities interface.

Fix the way we prepare Telepathy accounts and connections (including automatically handling ones that were already open)

Update for symbol changes in the latest tp-glib bindings.

Split PersonaStore into an abstract class and the TpPersonaStore derived class.

Rename from libtelepathy-people to libfolks.

Strip off the extraneous 'tpp-' prefix to the lowlevel files.

Organize classes in the following sections in order:

  * private (enum, class, signal, plain data, property)
  * public (interface members, signal, properties), one block per interface
  * public (enum, signal, plain data, property)
  * constructor
  * constructor's dependent functions, in order of reference
  * destructor
  * destructor's dependent functions
  * public functions, followed by their dependent functions in order of ref

Additional style changes.

  * use 'var' instead of an explicit type whenever possible. It's still
    type-safe yet reduces these types of redundancy (which would be common):

        Foo foo = new Foo ();
        Foo<Bar>[] foo_bar_array = new Foo<Bar>[size];

        ------------------->

        var foo = new Foo ();
        var foo_bar_array = new Foo<Bar>[size];

  * declare variables as closely to their first use as possible. This is to make
    all the relevant code as tight as possible, to minimize scrolling of all
    types. Exceptions include: class data members, blocks of code where there
    would be more than two variable definitions in a row; in this case, add a
    blank line between the last definition and the next line of code (to avoid
    making the code unreadably-dense); Eg, do this:

    {
      var manager = AccountManager.dup ();
      unowned GLib.List<Account> accounts = manager.get_valid_accounts ();
      this.stores = new HashMap<string, PersonaStore> ();

      foreach (Account account in accounts)
        {
          var store = new PersonaStore (account);
          this.stores.set (account.get_object_path (account), store);
        }
      ...
    }

Switch the Vala source to Telepathy style, with minor clarifications:

  * properties that use the default get() and set() functions should be on a
    single line. Eg,

        public string alias { get; set; default = "(No one of consequence)"; }

  * custom one-line get() and set() functions should be on a single line. Eg,

        get { return this._personas; }

  * lambda blocks should be formatted as if they were regular blocks (with the {
    indented once on the next line, another indent for the block content,
    outdent once for the ending }). Any arguments following the lambda
    definition in a function call should be continued on the next line (this
    hasn't come up yet). Eg,

        this._personas.foreach ((persona) =>
          {
            Persona p = (Persona) persona;

            if (alias == null)
              {
                alias = p.alias;
              }
          });

Change the namespace of the telepathy-people code to Tpp, to avoid ambiguity with telepathy-glib and its Vala bindings.

Create Individuals from new Personas and notify of their creation.

Change the personas_added type to a GLib.List from an array. This way we can consistently use GLists in the C API (since they're more conventional).

Add the Alias interface

Reduce the noise of our debug output.

Signal the addition of personas to the persona store.

Create the abstract class Tp.Persona and the Tp.TpPersona wrapper of TpContacts

Retrieve the TpContacts for the discovered contact handles

Make a trivial comment change

Use debug statements instead of standard output for the various debug output