Linux bluetooth development
 help / color / mirror / Atom feed
From: Johan Hedberg <johan.hedberg@gmail.com>
To: Luiz Augusto von Dentz <luiz.dentz@gmail.com>
Cc: linux-bluetooth@vger.kernel.org
Subject: Re: [PATCH 1/2] Add conf file support
Date: Fri, 20 May 2011 11:04:01 -0700	[thread overview]
Message-ID: <20110520180401.GA23399@dell.amr.corp.intel.com> (raw)
In-Reply-To: <1305620264-22304-1-git-send-email-luiz.dentz@gmail.com>

Hi Luiz,

On Tue, May 17, 2011, Luiz Augusto von Dentz wrote:
> This should reduce the amount of options which normally don't change each
> time obexd is executed.
> ---
>  Makefile.am    |    2 +-
>  plugins/ftp.c  |    8 ++-
>  src/main.c     |  176 +++++++++++++++++++++++++++++++++++++++-----------------
>  src/obexd.conf |   19 ++++++
>  4 files changed, 149 insertions(+), 56 deletions(-)
>  create mode 100644 src/obexd.conf

This doesn't compile with gcc 4.6:

src/main.c: In function ‘main’:
src/main.c:267:12: error: variable ‘config’ set but not used [-Werror=unused-but-set-variable]

Please fix and resend.

Johan

  parent reply	other threads:[~2011-05-20 18:04 UTC|newest]

Thread overview: 10+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2011-05-17  8:17 [PATCH 1/2] Add conf file support Luiz Augusto von Dentz
2011-05-17  8:17 ` [PATCH 2/2] Simplify options for including/excluding plugins Luiz Augusto von Dentz
2011-05-20 18:04 ` Johan Hedberg [this message]
2011-05-20 18:37   ` [PATCH 1/2] Add conf file support Marcel Holtmann
2011-05-20 19:17     ` Luiz Augusto von Dentz
  -- strict thread matches above, loose matches on Subject: below --
2011-05-24 16:12 Luiz Augusto von Dentz
2011-05-25  0:39 ` Marcel Holtmann
2011-05-25  6:14   ` Luiz Augusto von Dentz
2011-05-25 19:48     ` Marcel Holtmann
2011-05-25 19:58       ` Luiz Augusto von Dentz

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20110520180401.GA23399@dell.amr.corp.intel.com \
    --to=johan.hedberg@gmail.com \
    --cc=linux-bluetooth@vger.kernel.org \
    --cc=luiz.dentz@gmail.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body. 
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox