On Oct 28, 2008, at 12:02 PM, J.B.C.Engelen@...1578... wrote:

Empty translation units are not a problem, otherwise we could never

write

#ifndef __FILE_DIALOGIMPL_H__

#define __FILE_DIALOGIMPL_H__

at the start of any header file!

Just a minor note, those defines are technically "naughty"

Starting with underscores like that is reserved.

Instead we've been switching files to

#ifndef FILE_DIALOGIMPL_H_SEEN

#define FILE_DIALOGIMPL_H_SEEN