All of lore.kernel.org
 help / color / mirror / Atom feed
From: Colin D Bennett <colin@gibibit.com>
To: grub-devel@gnu.org
Cc: Robert Millan <rmh@aybabtu.com>
Subject: Re: [RFC] New menu design based on MVC
Date: Thu, 25 Jun 2009 06:50:55 -0700	[thread overview]
Message-ID: <200906250650.56025.colin@gibibit.com> (raw)
In-Reply-To: <20090625123659.GA6874@thorin>

Robert Millan wrote on Thursday 25 June 2009:
> On Thu, Jun 25, 2009 at 12:07:18PM +0800, Bean wrote:
> > Hi,
> >
> > The current menu system is a little chaotic, the various components
> > are mixed together which make it difficult to maintain and extend.
> > Therefore, I propose a new design which seperates model, view and
> > controller.

I would definitely like to get my graphical menu stuff committed before you 
make such changes... :-)

> Isn't Colin already working on a redesign of the menu?

Yes, but I haven't drastically changed the design of the core menu system 
except to add the concept of a "menu viewer" so that the text menu and 
graphical menu viewers can be treated uniformly at a higher level.

In fact, I am nearly ready to commit my graphical menu changes.  I was hoping 
to track down an apparent memory leak that shows up when switching themes many 
times in a row, but I don't have to let that prevent me from committing the 
current state of things.  I know Vladimir (phcoder) has done some work to get 
an SDL video driver working for grub-emu; I am hoping that I can somehow use 
memory debugging tools to track down my memory leak.

Does anyone have an idea if a tool like Valgrind would work with grub-emu to 
find the source of memory leaks?  I guess it should if grub-emu is using 
libc's malloc() and free() functions, right?

Regards,
Colin



  reply	other threads:[~2009-06-25 13:51 UTC|newest]

Thread overview: 13+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2009-06-25  4:07 [RFC] New menu design based on MVC Bean
2009-06-25 12:37 ` Robert Millan
2009-06-25 13:50   ` Colin D Bennett [this message]
2009-06-25 14:05     ` Bean
2009-06-25 14:25     ` Robert Millan
2009-06-25 15:54       ` Bean
2009-06-25 16:27         ` Colin D Bennett
2009-06-25 16:34         ` Isaac Dupree
2009-06-25 17:48         ` Pavel Roskin
2009-06-25 16:47     ` Vladimir 'phcoder' Serbinenko
2009-06-26 14:31 ` Robert Millan
2009-06-26 15:25   ` Pavel Roskin
2009-06-28  8:30 ` Marco Gerards

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=200906250650.56025.colin@gibibit.com \
    --to=colin@gibibit.com \
    --cc=grub-devel@gnu.org \
    --cc=rmh@aybabtu.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.