[prev in list] [next in list] [prev in thread] [next in thread] 

List:       gtk-devel
Subject:    Re: RFC: String cleanup
From:       Matthias Clasen <matthias.clasen () gmail ! com>
Date:       2005-12-16 15:17:33
Message-ID: cbccc63c0512160717n7d59c699w9366fadb07e073c3 () mail ! gmail ! com
[Download RAW message or body]

[Attachment #2 (multipart/alternative)]

[Attachment #4 (text/plain)]

On 12/16/05, Benoît Dejean <benoit@placenet.org> wrote:
>
> Le vendredi 16 décembre 2005 à 10:40 +0100, Daniel Marjamäki a écrit :
> > Hello friends!
> >
> > I'm new to GTK, and would like to contribute. I have an idea for what I
> can do...
> >
> > Changing from:
> >   [const] char *str
> > to:
> >   [const] char str[]
>
>
Many places where this matters have already been fixed. But by all means, if
you find more,
submit patches.

(The benefit is that the latter is placed in rodata...)

Matthias

[Attachment #5 (text/html)]

On 12/16/05, <b class="gmail_sendername">Benoît Dejean</b> &lt;<a \
href="mailto:benoit@placenet.org">benoit@placenet.org</a>&gt; wrote:<div><span \
class="gmail_quote"></span><blockquote class="gmail_quote" style="border-left: 1px \
solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;"> Le vendredi \
16 décembre 2005 à 10:40 +0100, Daniel Marjamäki a écrit :<br>&gt; Hello \
friends!<br>&gt;<br>&gt; I'm new to GTK, and would like to contribute. I have an idea \
for what I can do...<br>&gt;<br>&gt; Changing from: <br>&gt;&nbsp;&nbsp; [const] char \
*str<br>&gt; to:<br>&gt;&nbsp;&nbsp; [const] char \
str[]<br><br></blockquote><div><br>Many places where this matters have already been \
fixed. But by all means, if you find more,<br>submit patches. <br><br>(The benefit is \
that the latter is placed in rodata...) <br><br>Matthias<br></div><br></div><br>



_______________________________________________
gtk-devel-list mailing list
gtk-devel-list@gnome.org
http://mail.gnome.org/mailman/listinfo/gtk-devel-list


[prev in list] [next in list] [prev in thread] [next in thread] 

Configure | About | News | Add a list | Sponsored by KoreLogic