r21207 - gnucash/trunk/src/gnome-utils - Missed a free in main_window_update_page_name

John Ralls jralls at ceridwen.us
Mon Aug 22 19:49:59 EDT 2011


On Aug 22, 2011, at 1:05 PM, Geert Janssens wrote:

> On maandag 22 augustus 2011, John Ralls wrote:
>> Author: jralls
>> Date: 2011-08-22 14:18:53 -0400 (Mon, 22 Aug 2011)
>> New Revision: 21207
>> Trac: http://svn.gnucash.org/trac/changeset/21207
>> 
>> Modified:
>>   gnucash/trunk/src/gnome-utils/gnc-main-window.c
>> Log:
>> Missed a free in main_window_update_page_name
>> 
>> Modified: gnucash/trunk/src/gnome-utils/gnc-main-window.c
>> ===================================================================
>> --- gnucash/trunk/src/gnome-utils/gnc-main-window.c	2011-08-22 07:12:37 
> UTC
>> (rev 21206) +++ gnucash/trunk/src/gnome-utils/gnc-main-window.c	2011-08-22
>> 18:18:53 UTC (rev 21207) @@ -1929,7 +1929,7 @@
>>     {
>>         g_free(old_page_name);
>>         g_free(old_page_long_name);
>> -        g_free(name);
>> +	g_free(name);
>>         LEAVE("no window widget available");
>>         return;
>>     }
>> 
> 
> John,
> 
> This commit seems to undo the source code reindentation Christian just did. 
> Perhaps you forgot to run a git update before dcommitting ?
> 

Geert,

Ah, no, I had a conflict when I ran git-update and picked the wrong one. I thought that it was a git-svn glitch, but it was a tab instead of spaces for the indentation. I've reverted that with r21209. Thanks for pointing it out.

Now to fix my emacs settings so I'm not sticking tabs into source files.

Regards,
John Ralls


More information about the gnucash-devel mailing list