svn commit: samba r3737 - in branches/SAMBA_4_0/source: auth build/pidl build/smb_build lib lib/dcom lib/dcom/common lib/registry lib/registry/common lib/registry/reg_backend_dir lib/registry/reg_backend_gconf lib/registry/reg_backend_ldb lib/registry/reg_backend_nt4 lib/registry/reg_backend_rpc lib/registry/reg_backend_w95 lib/tdb libcli/auth librpc/rpc ntvfs ntvfs/cifs ntvfs/ipc ntvfs/nbench ntvfs/posix ntvfs/print ntvfs/simple ntvfs/unixuid rpc_server rpc_server/dcom rpc_server/remote smbd

[Stefan (metze) Metzmacher]

Jelmer Vernooij schrieb:

> Stefan (metze) Metzmacher wrote:
> 
>> jelmer at samba.org schrieb:
>>
>>> Author: jelmer
>>> Date: 2004-11-14 22:23:23 +0000 (Sun, 14 Nov 2004)
>>> New Revision: 3737
>>>
>>> WebSVN: 
>>> http://websvn.samba.org/cgi-bin/viewcvs.cgi?view=rev&root=samba&rev=3737
>>>
>>> Log:
>>> - Get rid of the register_subsystem() and register_backend() functions. 
>>
>>
>>
>> why? there was a very good reason for adding them.
> 
> 
> Originally, yes. However, they have a few issues:
> 
> - Registration of a subsystem has to happen before registration of a module
I don't see how using subsystem specific function solve that

> - No type checking 
that's noe really a reason I think

> - Need to recompile the files containing the _static_init_modules macros 
> after the configration changes.
I don't see how using subsystem specific function solve that

> 
> I'll replace this later (when introducing support for shared modules and 
> actual loading of shared modules) with an improved version (using a 
> subsystem_supported() function, perhaps).

but please let us discus before a commit:-)



-- 
metze

Stefan Metzmacher <metze at samba.org> www.samba.org