public inbox for linux-bluetooth@vger.kernel.org
 help / color / mirror / Atom feed
* [Bluez-devel] [PATCH] Launch browser / email client when clicking on link in about dialogs
@ 2008-02-10 18:16 Jaap A. Haitsma
  2008-02-10 18:33 ` Marcel Holtmann
  0 siblings, 1 reply; 4+ messages in thread
From: Jaap A. Haitsma @ 2008-02-10 18:16 UTC (permalink / raw)
  To: BlueZ development

Hi Marcel

Another patch
http://bugzilla.gnome.org/show_bug.cgi?id=515581

Jaap

-------------------------------------------------------------------------
This SF.net email is sponsored by: Microsoft
Defy all challenges. Microsoft(R) Visual Studio 2008.
http://clk.atdmt.com/MRT/go/vse0120000070mrt/direct/01/
_______________________________________________
Bluez-devel mailing list
Bluez-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/bluez-devel

^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: [Bluez-devel] [PATCH] Launch browser / email client when clicking on link in about dialogs
  2008-02-10 18:16 [Bluez-devel] [PATCH] Launch browser / email client when clicking on link in about dialogs Jaap A. Haitsma
@ 2008-02-10 18:33 ` Marcel Holtmann
  2008-02-10 20:39   ` Marcel Holtmann
  0 siblings, 1 reply; 4+ messages in thread
From: Marcel Holtmann @ 2008-02-10 18:33 UTC (permalink / raw)
  To: BlueZ development

Hi Jaap,

> Another patch
> http://bugzilla.gnome.org/show_bug.cgi?id=515581

can you fix the coding style please. There are white spaces that
shouldn't be there.

Regards

Marcel



-------------------------------------------------------------------------
This SF.net email is sponsored by: Microsoft
Defy all challenges. Microsoft(R) Visual Studio 2008.
http://clk.atdmt.com/MRT/go/vse0120000070mrt/direct/01/
_______________________________________________
Bluez-devel mailing list
Bluez-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/bluez-devel

^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: [Bluez-devel] [PATCH] Launch browser / email client when clicking on link in about dialogs
  2008-02-10 18:33 ` Marcel Holtmann
@ 2008-02-10 20:39   ` Marcel Holtmann
  2008-02-10 21:45     ` Jaap A. Haitsma
  0 siblings, 1 reply; 4+ messages in thread
From: Marcel Holtmann @ 2008-02-10 20:39 UTC (permalink / raw)
  To: BlueZ development

Hi Jaap,

> > Another patch
> > http://bugzilla.gnome.org/show_bug.cgi?id=515581
> 
> can you fix the coding style please. There are white spaces that
> shouldn't be there.

never mind. Did it by myself. Just two minor things.

First, I am picky about the coding style. Always have been and always
will be. The code must look sane and not as some hackish crap. Also
having the coding style in place in the first place makes my life easier
and patching is a lot simpler.

Second, please don't put patches into Bugzilla. I stated before that I
don't care about the GNOME Bugzilla. If you wanna do that, fine, but
then send a copy to the mailing list (not some link). This is the place
where I pick them up. Otherwise the will go unseen and end up in
nirvana.

Regards

Marcel



-------------------------------------------------------------------------
This SF.net email is sponsored by: Microsoft
Defy all challenges. Microsoft(R) Visual Studio 2008.
http://clk.atdmt.com/MRT/go/vse0120000070mrt/direct/01/
_______________________________________________
Bluez-devel mailing list
Bluez-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/bluez-devel

^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: [Bluez-devel] [PATCH] Launch browser / email client when clicking on link in about dialogs
  2008-02-10 20:39   ` Marcel Holtmann
@ 2008-02-10 21:45     ` Jaap A. Haitsma
  0 siblings, 0 replies; 4+ messages in thread
From: Jaap A. Haitsma @ 2008-02-10 21:45 UTC (permalink / raw)
  To: BlueZ development

On Feb 10, 2008 9:39 PM, Marcel Holtmann <marcel@holtmann.org> wrote:
> Hi Jaap,
>
> > > Another patch
> > > http://bugzilla.gnome.org/show_bug.cgi?id=515581
> >
> > can you fix the coding style please. There are white spaces that
> > shouldn't be there.

Sorry. I try not to put spaces, but it's difficult to change a habit.
I'll try checking better when sending you a patch next time.


> never mind. Did it by myself. Just two minor things.
>
> First, I am picky about the coding style. Always have been and always
> will be. The code must look sane and not as some hackish crap. Also
> having the coding style in place in the first place makes my life easier
> and patching is a lot simpler.

Will do my best

> Second, please don't put patches into Bugzilla. I stated before that I
> don't care about the GNOME Bugzilla. If you wanna do that, fine, but
> then send a copy to the mailing list (not some link). This is the place
> where I pick them up. Otherwise the will go unseen and end up in
> nirvana.
>
I personally feel the other way around. I tend to loose track of
emails, so I prefer bugzilla.

But it's not a problem for me to also attach it, if you prefer that

Jaap

-------------------------------------------------------------------------
This SF.net email is sponsored by: Microsoft
Defy all challenges. Microsoft(R) Visual Studio 2008.
http://clk.atdmt.com/MRT/go/vse0120000070mrt/direct/01/
_______________________________________________
Bluez-devel mailing list
Bluez-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/bluez-devel

^ permalink raw reply	[flat|nested] 4+ messages in thread

end of thread, other threads:[~2008-02-10 21:45 UTC | newest]

Thread overview: 4+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2008-02-10 18:16 [Bluez-devel] [PATCH] Launch browser / email client when clicking on link in about dialogs Jaap A. Haitsma
2008-02-10 18:33 ` Marcel Holtmann
2008-02-10 20:39   ` Marcel Holtmann
2008-02-10 21:45     ` Jaap A. Haitsma

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox