Ticket #227 (closed defect: fixed)

Opened 10 years ago

Last modified 10 years ago

Replace g_free() by mhl_mem_free()

Reported by: metux Owned by: metux
Priority: major Milestone: 4.7
Component: mc-core Version: 4.6.1
Keywords: vote-angel_il Cc:
Blocked By: Blocking:
Branch state: Votes for changeset:

Description (last modified by metux) (diff)

Now as we have mhl in mainline, replace g_free() by mhl_mem_free()

branch:227_replace_g_free
changeset:15d7d47c1556c154a0cf93ef887a3d0b9d962511

committed master
changeset:8b38cf44daee3b0f92591baa97f36d722be99280

Change History

comment:1 Changed 10 years ago by metux

  • Keywords review added
  • Status changed from new to accepted
  • Description modified (diff)

comment:2 Changed 10 years ago by slyfox

  • Keywords vote-slyfox added

git log --stat -p master..origin/227_replace_g_free does not fit into 1 page:

73 files changed, 1146 insertions(+), 1054 deletions(-)

But patch looks trivial (and shows what a mess we have in respect to memory allocation in general).

With that it will be easier to switch alloc/free implementation for the whole project.

Voted.

comment:3 Changed 10 years ago by winnie

  • Keywords vote-winnie approved added; review removed

looks fine

comment:4 Changed 10 years ago by metux

  • Status changed from accepted to testing
  • Keywords committed-master added; vote-slyfox vote-winnie approved removed
  • Resolution set to fixed
  • Description modified (diff)

comment:5 in reply to: ↑ description Changed 10 years ago by angel_il

  • Keywords vote-angel_il added; committed-master removed

Replying to metux:

Now as we have mhl in mainline, replace g_free() by mhl_mem_free()

it's ok

comment:6 Changed 10 years ago by winnie

  • Status changed from testing to closed
Note: See TracTickets for help on using tickets.