Re: memory lead while using Glib regexp

[Bernhard Schuster <schuster bernhard googlemail com>]

No it does, not I already cross-read it and it looks fine, did you
give it a shot with valgrind?

Regards

Bernhard Schuster

Am 5. MÃrz 2012 15:43 schrieb Olivier Sessink <oliviersessink gmail com>:
> On 03/02/2012 04:00 AM, Xi Yang wrote:
> > Hi, everyone!
> >
> >
> > I use Glib's regular expression in a cycle, and find a rapid memory
> > consumption increase. Did I forget to release anything?
>
>
> since nobody replied I'll give it a shot (without any C++ knowledge):
>
> does seq.header.c_str() return a newly allocated string?
>
>
> Olivier
>
>
> > Thanks a lot!
> >
> >
> > ================= code below ======================
> >
> >
> > GRegex* regex_illumina = g_regex_new(
> > Â Â "@(.+):(\\d+):(\\d+):(\\d+):(\\d+)#(\\d+)/([12])",
> > Â Â G_REGEX_OPTIMIZE,
> > Â Â GRegexMatchFlags(0),
> > Â Â NULL
> > );
> >
> >
> > GRegex* regex_casava_1p8 = g_regex_new(
> > Â Â "@(.+):(.+):(.+):(\\d+):(\\d+):(\\d+):(\\d+)
> > ([12]):([YN]):\\d+:(\\w*)",
> > Â Â G_REGEX_OPTIMIZE,
> > Â Â GRegexMatchFlags(0),
> > Â Â NULL
> > );
> >
> >
> > // inside cycle body
> > Â Â Â Â gchar* tile_str;
> > Â Â Â Â int tile;
> > Â Â Â Â GMatchInfo* what;
> >
> > Â Â Â Â // match and fetch something
> > Â Â Â Â if ( g_regex_match(regex_illumina, seq.header.c_str(),
> > GRegexMatchFlags(0),&what) ) {
> > Â Â Â Â Â Â tile_str = g_match_info_fetch(what,3);
> > Â Â Â Â }
> > Â Â Â Â else if ( g_regex_match(regex_casava_1p8, seq.header.c_str(),
> > GRegexMatchFlags(0),&what) ) {
> >
> > Â Â Â Â Â Â tile_str = g_match_info_fetch(what,5);
> > Â Â Â Â }
> > Â Â Â Â else {
> > Â Â Â Â Â Â throw runtime_error("failed to parse header
> > line:\n"+seq.header);
> > Â Â Â Â }
> >
> >
> > Â Â Â Â // parse a number
> > Â Â Â Â if (sscanf(tile_str,"%d",&tile)!=1)
> > Â Â Â Â Â Â throw runtime_error("failed to fetch tile from "+seq.header);
> >
> >
> > Â Â Â Â // release match info and fetched string
> > Â Â Â Â g_match_info_free(what);
> > Â Â Â Â g_free(tile_str);
> > // end of cycle body
> > _______________________________________________
> > gtk-app-devel-list mailing list
> > gtk-app-devel-list gnome org
> > http://mail.gnome.org/mailman/listinfo/gtk-app-devel-list
>
>
> _______________________________________________
> gtk-app-devel-list mailing list
> gtk-app-devel-list gnome org
> http://mail.gnome.org/mailman/listinfo/gtk-app-devel-list