Linux bluetooth development
 help / color / mirror / Atom feed
From: Marcel Holtmann <marcel@holtmann.org>
To: Szymon Janc <szymon.janc@tieto.com>
Cc: "linux-bluetooth@vger.kernel.org" <linux-bluetooth@vger.kernel.org>
Subject: Re: [PATCH 2/9] Pass confirm name as gboolean value
Date: Wed, 10 Oct 2012 12:04:11 +0200	[thread overview]
Message-ID: <1349863451.27233.116.camel@aeonflux> (raw)
In-Reply-To: <1505997.4TGEV3eNyH@uw000953>

Hi Szymon,

> > > confirm_name is a boolean value and can be pass as such instead of
> > > uint8_t.
> > 
> > actually long term I want to move to bool usage. So maybe this is a good
> > start to get this going.
> 
> OK, will send v2 with bool.
> Do you also want to stop introducing other glib types like gchar, gint etc?
> (at least for code that don't inherit this due to glib calls)

yes, please no more gchar and gint. These are horrible. We were young
and naive when we thought they were a good idea ;)

Regards

Marcel



  reply	other threads:[~2012-10-10 10:04 UTC|newest]

Thread overview: 17+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2012-10-09 12:11 [PATCH 0/9] Remove some old code Szymon Janc
2012-10-09 12:11 ` [PATCH 1/9] Simplify checking if found device is legacy Szymon Janc
2012-10-09 12:11 ` [PATCH 2/9] Pass confirm name as gboolean value Szymon Janc
2012-10-09 14:51   ` Marcel Holtmann
2012-10-09 15:49     ` Lucas De Marchi
2012-10-09 16:12       ` Marcel Holtmann
2012-10-10  9:57     ` Szymon Janc
2012-10-10 10:04       ` Marcel Holtmann [this message]
2012-10-09 12:11 ` [PATCH 3/9] event: Don't store EIR in btd_event_device_found Szymon Janc
2012-10-09 12:11 ` [PATCH 4/9] storage: Remove not used read_remote_eir Szymon Janc
2012-10-09 12:11 ` [PATCH 5/9] storage: Remove not used write_version_info Szymon Janc
2012-10-09 12:11 ` [PATCH 6/9] storage: Remove not used write_features_info Szymon Janc
2012-10-09 12:12 ` [PATCH 7/9] storage: Remove not used read_remote_features Szymon Janc
2012-10-09 12:12 ` [PATCH 8/9] adapter: Remove not used btd_adapter_switch_{online,offline} functions Szymon Janc
2012-10-09 12:12 ` [PATCH 9/9] device: Remove not used device_set_class function Szymon Janc
2012-10-09 14:30 ` [PATCH 0/9] Remove some old code Vinicius Costa Gomes
2012-10-09 16:39 ` Johan Hedberg

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=1349863451.27233.116.camel@aeonflux \
    --to=marcel@holtmann.org \
    --cc=linux-bluetooth@vger.kernel.org \
    --cc=szymon.janc@tieto.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