Re: Interfacing Gtk+ and Threaded Design



Ah, that ;) I usually tend to slip some of these out, as I don't give
much importance to them. But I will correct all of them to a consistent
style. Thank you for your observation!

On Mon, 2004-08-02 at 10:37, Thomas Vander Stichele wrote:
> On Mon, 2004-08-02 at 12:33, Tiago Cogumbreiro wrote:
> > Thank you, very much :)
> > 
> > Could you be more explicit on the coding style issue? What's wrong?
> 
> The thing that stuck out to me was spacing after function names.
> 
> You have
> 
> gtk_widget_destroy (dialog);
> 
> vs.
> 
> g_idle_add(lengthy_func_done, NULL);
> 
> Thomas
> 
> > On Mon, 2004-08-02 at 09:06, Thomas Vander Stichele wrote:
> > > Hi,
> > > 
> > > On Fri, 2004-07-30 at 22:12, Tiago Cogumbreiro wrote:
> > > > Hello all in the list, I have written a small article on how to
> > > > interface Gtk+ programming and threaded development side by side.
> > > > 
> > > > I would like your comments and critics about it's content. Remember this
> > > > is just a draft.
> > > > 
> > > > Here is the link:
> > > > 
> > > > http://tiago.blogdns.org/gtk_threads.html
> > > 
> > > Nice work.  Only one comment - PLEASE use a consistent coding style. 
> > > Tutorials on any concept should implicitly teach people good programming
> > > style as well.  Mixing coding styles or having an inconsistent coding
> > > style confuses your target audience.
> > > 
> > > Thomas
> > > 
> > > 
> > > Dave/Dina : future TV today ! - http://www.davedina.org/
> > > <-*- thomas (dot) apestaart (dot) org -*->
> > > If I could talk I'd tell you
> > > If I could smile I'd let you know
> > > You are far and away my most imaginary friend
> > > <-*- thomas (at) apestaart (dot) org -*->
> > > URGent, best radio on the net - 24/7 ! - http://urgent.fm/
> > > 
> 
> Dave/Dina : future TV today ! - http://www.davedina.org/
> <-*- thomas (dot) apestaart (dot) org -*->
> Say what you mean
> or you won't mean a thing to me
> <-*- thomas (at) apestaart (dot) org -*->
> URGent, best radio on the net - 24/7 ! - http://urgent.fm/
> 




[Date Prev][Date Next]   [Thread Prev][Thread Next]   [Thread Index] [Date Index] [Author Index]