Jacek Caban : ieframe: Moved freeing travellog entry into a helper function .

Alexandre Julliard julliard at winehq.org
Tue Apr 2 13:38:37 CDT 2013


Module: wine
Branch: master
Commit: f1517b15719e489d5d35d01e68c1cfc18787c86b
URL:    http://source.winehq.org/git/wine.git/?a=commit;h=f1517b15719e489d5d35d01e68c1cfc18787c86b

Author: Jacek Caban <jacek at codeweavers.com>
Date:   Tue Apr  2 17:39:34 2013 +0200

ieframe: Moved freeing travellog entry into a helper function.

---

 dlls/ieframe/dochost.c |    9 +++++++--
 1 files changed, 7 insertions(+), 2 deletions(-)

diff --git a/dlls/ieframe/dochost.c b/dlls/ieframe/dochost.c
index 0788132..e1a897a 100644
--- a/dlls/ieframe/dochost.c
+++ b/dlls/ieframe/dochost.c
@@ -333,6 +333,11 @@ static LRESULT WINAPI doc_view_proc(HWND hwnd, UINT msg, WPARAM wParam, LPARAM l
     return DefWindowProcW(hwnd, msg, wParam, lParam);
 }
 
+static void free_travellog_entry(travellog_entry_t *entry)
+{
+    heap_free(entry->url);
+}
+
 static void update_travellog(DocHost *This)
 {
     travellog_entry_t *new_entry;
@@ -357,7 +362,7 @@ static void update_travellog(DocHost *This)
         }
 
         while(This->travellog.length > This->travellog.position)
-            heap_free(This->travellog.log[--This->travellog.length].url);
+            free_travellog_entry(This->travellog.log + --This->travellog.length);
     }
 
     new_entry = This->travellog.log + This->travellog.position;
@@ -1013,7 +1018,7 @@ void DocHost_Release(DocHost *This)
     ConnectionPointContainer_Destroy(&This->cps);
 
     while(This->travellog.length)
-        heap_free(This->travellog.log[--This->travellog.length].url);
+        free_travellog_entry(This->travellog.log + --This->travellog.length);
     heap_free(This->travellog.log);
 
     heap_free(This->url);




More information about the wine-cvs mailing list