Attachment 57393 Details for Bug 46384 – Avoid forward-declaring Call classes

[patch] Avoid forward-declaring Call classes

0001-Avoid-forward-declaring-Call-classes.patch (text/plain), 2.75 KB, created by Simon McVittie on 2012-02-21 06:00:57 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Simon McVittie

Created: 2012-02-21 06:00:57 UTC

Size: 2.75 KB

patch

obsolete

>From a26adf87d4985834a64d165ba5c6ee19ffd7a989 Mon Sep 17 00:00:00 2001
>From: Simon McVittie <simon.mcvittie@collabora.co.uk>
>Date: Tue, 21 Feb 2012 13:59:05 +0000
>Subject: [PATCH] Avoid forward-declaring Call classes
>
>Having more than one typedef for the same name is an error.
>
>If forward declarations are needed to break cycles, the typedef must
>be *moved* (not copied) from the header it'd normally appear in to the
>header with the forward declaration: for instance, TpCallContent
>(forward-declared in call-channel.h) gets this right. In extreme cases,
>a global "types.h" header can forward-declare everything, although that
>often leads to unnecessary compilation, so we've avoided it in
>telepathy-glib.
>---
> telepathy-glib/base-call-content.h  |    3 ---
> telepathy-glib/base-call-internal.h |   18 ++++++++----------
> 2 files changed, 8 insertions(+), 13 deletions(-)
>
>diff --git a/telepathy-glib/base-call-content.h b/telepathy-glib/base-call-content.h
>index 56245f0..ea15e08 100644
>--- a/telepathy-glib/base-call-content.h
>+++ b/telepathy-glib/base-call-content.h
>@@ -31,9 +31,6 @@
> 
> G_BEGIN_DECLS
> 
>-/* forward declaration */
>-typedef struct _TpBaseCallChannel TpBaseCallChannel;
>-
> typedef struct _TpBaseCallContent TpBaseCallContent;
> typedef struct _TpBaseCallContentPrivate TpBaseCallContentPrivate;
> typedef struct _TpBaseCallContentClass TpBaseCallContentClass;
>diff --git a/telepathy-glib/base-call-internal.h b/telepathy-glib/base-call-internal.h
>index aae5e17..9e40547 100644
>--- a/telepathy-glib/base-call-internal.h
>+++ b/telepathy-glib/base-call-internal.h
>@@ -22,20 +22,18 @@
> #ifndef __TP_BASE_CALL_INTERNAL_H__
> #define __TP_BASE_CALL_INTERNAL_H__
> 
>+#include <telepathy-glib/base-call-channel.h>
>+#include <telepathy-glib/base-call-content.h>
>+#include <telepathy-glib/base-call-stream.h>
> #include <telepathy-glib/base-connection.h>
>+#include <telepathy-glib/base-media-call-channel.h>
>+#include <telepathy-glib/base-media-call-content.h>
>+#include <telepathy-glib/base-media-call-stream.h>
>+#include <telepathy-glib/call-content-media-description.h>
>+#include <telepathy-glib/call-stream-endpoint.h>
> 
> G_BEGIN_DECLS
> 
>-/* Forward declaration */
>-typedef struct _TpBaseCallChannel TpBaseCallChannel;
>-typedef struct _TpBaseCallContent TpBaseCallContent;
>-typedef struct _TpBaseCallStream  TpBaseCallStream;
>-typedef struct _TpCallContentMediaDescription  TpCallContentMediaDescription;
>-typedef struct _TpBaseMediaCallChannel TpBaseMediaCallChannel;
>-typedef struct _TpBaseMediaCallContent TpBaseMediaCallContent;
>-typedef struct _TpBaseMediaCallStream TpBaseMediaCallStream;
>-typedef struct _TpCallStreamEndpoint TpCallStreamEndpoint;
>-
> /* Implemented in base-call-content.c */
> void _tp_base_call_content_set_channel (TpBaseCallContent *self,
>     TpBaseCallChannel *channel);
>-- 
>1.7.9.1
>

Actions: View | Splinter Review

Attachments on bug 46384: 57393 | 57399 | 57400

Use of freedesktop.org services, including Bugzilla, is subject to our Code of Conduct. How we collect and use information is described in our Privacy Policy.