Ticket #12 (new defect)

Opened 2 months ago

missing public in front of import

Reported by: lindquist Assigned to: keinfarbton
Priority: major Milestone:
Component: component1 Version:
Keywords: Cc:

Description

Hi Frank

I'm very close to being able to build DWT with LLVMDC now, the immensely slow compile time issue has been solved as well as a few other things.

There's one more case of protection which DMD doesn't catch as shown in this patch. Modules expect Math.* to be available, but imports are private by default:

[tomas@myhost dwt-linux]$ hg diff diff -r ffb904674b52 dwt/DWT.d --- a/dwt/DWT.d Sat Sep 27 15:58:22 2008 +0200 +++ b/dwt/DWT.d Wed Oct 01 22:36:02 2008 +0200 @@ -19,7 +19,7 @@

import dwt.DWTError; import dwt.DWTException;

-import dwt.dwthelper.utils; +public import dwt.dwthelper.utils;

import tango.core.Exception; version( build ){

With this applied there's a single issue left and we should be able to test a run of HelloWorld?1.d

Display.d(442): Error: function dwt.widgets.Display.Display.initDeviceFinder is static and cannot access nested class dwt.widgets.Display.Display.initDeviceFinder.anonclass61

This however seems to be a bug in LLVMDC, from what I can see the code does not need a context and should be fine, Christian will look into this and hopefully we'll have DWT working with LLVMDC soon :) This will be great and a good showcase that LLVMDC is very much useful, so expect it to work soon ;)

Attachments

llvmdc.diff (309 bytes) - added by lindquist on 10/01/08 16:42:02.
attached diff as I forgot to format it properly…

Change History

10/01/08 16:42:02 changed by lindquist

  • attachment llvmdc.diff added.

attached diff as I forgot to format it properly...