r3284: - don't use a enum for bit-fields. It isn't legal C (as C is limited to

  signed integer values for enum), and it is also very confusing to read.

  Also, please align defines nicely. The value of SV_TYPE_DOMAIN_ENUM
  was defined incorrectly, which became obvious when you align the
  defines, but wasn't at all obvious in the original code.

- removed redundent defines in smb.h
(This used to be commit fe042b8e5ca15dfd986bc0c05d464214a51cc0cf)

1 files changed, 32 insertions, 38 deletions

diff --git a/source4/librpc/idl/svcctl.idl b/source4/librpc/idl/svcctl.idl
index 5003d407f1..c293298499 100644
--- a/source4/librpc/idl/svcctl.idl
+++ b/source4/librpc/idl/svcctl.idl
@@ -33,44 +33,38 @@
 		SERVICE_STATUS status;
 	} ENUM_SERVICE_STATUS;
 
-	typedef enum {
-		SERVICE_TYPE_KERNEL_DRIVER=0x01,
-		SERVICE_TYPE_FS_DRIVER=0x02,
-		SERVICE_TYPE_ADAPTER=0x04,
-		SERVICE_TYPE_RECOGNIZER_DRIVER=0x08,
-		SERVICE_TYPE_DRIVER=SERVICE_TYPE_KERNEL_DRIVER|SERVICE_TYPE_FS_DRIVER|SERVICE_TYPE_RECOGNIZER_DRIVER,
-		SERVICE_TYPE_WIN32_OWN_PROCESS=0x10,
-		SERVICE_TYPE_WIN32_SHARE_PROCESS=0x20,
-		SERVICE_TYPE_WIN32=SERVICE_TYPE_WIN32_OWN_PROCESS|SERVICE_TYPE_WIN32_SHARE_PROCESS
-	} ServiceType;
-
-	typedef enum {
-		SERVICE_STATE_ACTIVE=0x01,
-		SERVICE_STATE_INACTIVE=0x02,
-		SERVICE_STATE_ALL=0x03
-	} ServiceState;
-
-	typedef enum {
-		SV_TYPE_WORKSTATION =0x00000001,
-		SV_TYPE_SERVER =0x00000002,
-		SV_TYPE_DOMAIN_CTRL =0x00000008,
-		SV_TYPE_DOMAIN_BAKCTRL =0x00000010,
-		SV_TYPE_TIME_SOURCE =0x00000020,
-		SV_TYPE_AFP =0x00000040,
-		SV_TYPE_DOMAIN_MEMBER =0x00000100,
-		SV_TYPE_PRINTQ_SERVER =0x00000200,
-		SV_TYPE_DIALIN_SERVER =0x00000400,
-		SV_TYPE_XENIX_SERVER =0x00000800,
-		SV_TYPE_SERVER_UNIX =0x00000800,
-		SV_TYPE_NT =0x00001000,
-		SV_TYPE_WFW =0x00002000,
-		SV_TYPE_POTENTIAL_BROWSER =0x00010000,
-		SV_TYPE_BACKUP_BROWSER =0x00020000,
-		SV_TYPE_MASTER_BROWSER =0x00040000,
-		SV_TYPE_DOMAIN_MASTER =0x00080000,
-		SV_TYPE_LOCAL_LIST_ONLY =0x40000000,
-		SV_TYPE_DOMAIN_ENUM =0x80000000
-	} SERVICE_BITS;
+	const int SERVICE_TYPE_KERNEL_DRIVER       = 0x01;
+	const int SERVICE_TYPE_FS_DRIVER           = 0x02;
+	const int SERVICE_TYPE_ADAPTER             = 0x04;
+	const int SERVICE_TYPE_RECOGNIZER_DRIVER   = 0x08;
+	const int SERVICE_TYPE_DRIVER=SERVICE_TYPE_KERNEL_DRIVER|SERVICE_TYPE_FS_DRIVER|SERVICE_TYPE_RECOGNIZER_DRIVER;
+	const int SERVICE_TYPE_WIN32_OWN_PROCESS   = 0x10;
+	const int SERVICE_TYPE_WIN32_SHARE_PROCESS = 0x20;
+	const int SERVICE_TYPE_WIN32=SERVICE_TYPE_WIN32_OWN_PROCESS|SERVICE_TYPE_WIN32_SHARE_PROCES;
+
+	const int SERVICE_STATE_ACTIVE   = 0x01;
+	const int SERVICE_STATE_INACTIVE = 0x02;
+	const int SERVICE_STATE_ALL      = 0x00;
+
+	const int SV_TYPE_WORKSTATION       = 0x00000001;
+	const int SV_TYPE_SERVER            = 0x00000002;
+	const int SV_TYPE_DOMAIN_CTRL       = 0x00000008;
+	const int SV_TYPE_DOMAIN_BAKCTRL    = 0x00000010;
+	const int SV_TYPE_TIME_SOURCE       = 0x00000020;
+	const int SV_TYPE_AFP               = 0x00000040;
+	const int SV_TYPE_DOMAIN_MEMBER     = 0x00000100;
+	const int SV_TYPE_PRINTQ_SERVER     = 0x00000200;
+	const int SV_TYPE_DIALIN_SERVER     = 0x00000400;
+	const int SV_TYPE_XENIX_SERVER      = 0x00000800;
+	const int SV_TYPE_SERVER_UNIX       = 0x00000800;
+	const int SV_TYPE_NT                = 0x00001000;
+	const int SV_TYPE_WFW               = 0x00002000;
+	const int SV_TYPE_POTENTIAL_BROWSER = 0x00010000;
+	const int SV_TYPE_BACKUP_BROWSER    = 0x00020000;
+	const int SV_TYPE_MASTER_BROWSER    = 0x00040000;
+	const int SV_TYPE_DOMAIN_MASTER     = 0x00080000;
+	const int SV_TYPE_LOCAL_LIST_ONLY   = 0x40000000;
+	const int SV_TYPE_DOMAIN_ENUM       = 0x80000000;
 	
 	/*****************/
 	/* Function 0x00 */