Hi Xiao-Feng,

On Fri, Jul 18, 2008 at 7:05 AM, Xiao-Feng Li <xiaofeng.li@gmail.com> wrote:

> On Fri, Jul 18, 2008 at 5:52 AM, Senaka Fernando <senakafdo@gmail.com>
> wrote:
> > Hi all,
> >
> > I have few questions regarding localizing the GC.
> >
> > 1. Isn't this buggy (working_vm/vm/gc_gen/src/common/large_pages.cpp:129)
> >
> >     if (proc_huge_pages_total * proc_huge_page_size >= required_size) {
> >      WARN(("GC large_page: Not enough free large pages, some of reserved
> > space is already busy."));
> >     } else {
> >      WARN(("GC large_page: Not enough free large pages, some of reserved
> > space is already busy."));
> >     }
>
>

>
> What's the bug you are mentioning?


In here, both inside the if and inside the else we have the same code. Why
do we need a if-else here? or is there some mistake made in the warning
message?

>
>
> > 2. What's WARN2, and DIE2? where are they defined?
>
> Please try to find their definitions yourself.


I couldn't find them defined in any header. And, they lie in code portions
that won't be compiled, or commented source.

>
>
> Thanks,
> xiaofeng


Regards,
Senaka