registry.cpp not #if wxUSE_CONFIG wrapped?

metalogic · Post by **metalogic** » Sun May 08, 2005 10:52 pm

I defined wxUSE_CONFIG 0 but it was still compiling the registry class.

If this is a bug (can anyone confirm?) how do I report it?

Details
VC++ 2003
wxWidgets 2.6.0
Windows 2000

Cursor · Post by **Cursor** » Mon May 09, 2005 5:49 am

IMHO it is not.
wxUSE_CONFIG is for the wxConfig classes but you can use the registry (windows only) without the wxConfig classes.

No preprocessor definition is present to disable the registry class. But is is not a problem because wxRegistry is support only on windows and all windows plateform support it.

metalogic · Post by **metalogic** » Mon May 09, 2005 3:47 pm

Well, it is a problem if you are trying to minimize the size of the library. I don't need the registry and I would rather not have it compiled in.

If it shouldn't be tied to wxUSE_CONFIG I think it should have it's own #define.

Thanks for replying.

- Santiago

ABX · Post by **ABX** » Mon May 09, 2005 3:58 pm

metalogic wrote:Well, it is a problem if you are trying to minimize the size of the library. I don't need the registry and I would rather not have it compiled in.

You don't need, but wxWidgets functionality needs it for reading settings of OS.

metalogic wrote:If it shouldn't be tied to wxUSE_CONFIG I think it should have it's own #define.

Sources are open, editor in your hands and patch tracker available

If you know how to modify rest of the wxWidgets code to work without wxRegKey class you can share it.

ABX

metalogic · Post by **metalogic** » Mon May 09, 2005 4:04 pm

Thanks ABX for your answer.

Well, I simply removed registry.cpp from the library and everything compiled fine so I'm assuming there weren't any dependencies. I have undefined a lot of things since I am trying to get a very, very lean library so maybe in the process I undefined everything that depends on the registry.

I guess I could do a test by taking the plain vanilla library and just removing registry.cpp and see how that goes.

Might try that tonight.