[SCM] Samba Shared Repository - branch master updated - release-4-0-0alpha8-337-g73e9693

boyang boyang at samba.org
Sun Aug 2 23:12:57 MDT 2009


boyang wrote:
> Hi, Jeremy:
>      I have finished the patch as your suggestion. Using poll-style
> method to do this, and it can work. I didn't add the user share
> directory to lp_file_list_changed, the reason is if only usershare was
> changed, call reload_services() with lp_file_list_changed() evaluated to
> false is expensive. It will reload all services, including printers,
> somehow slow.
>
>      Once the team decide to add reload_services() to child's main loop
> [smbd_serever_connection_loop_once()], we can call
> load_usershare_shares(true) within reload_services() to include
> usershare. load_usershare_shares(false) is the original
> load_usershare_shares(void).
>
>      I didn't do the registry part.  The message register part  is easy.
> I have to figure out  where  should we send  the message from.
> I am not familiar with that part. :-) need to browse the code.
>
>      Please review the patch in the attachment. If it is ok, I'll clean
> comments and log messages and then commit. Which won't bother you
> reverting it, again. :-) heihei
>   
Hi, Jeremy:
     Is the patch functionally ok? If it is, I'll clean up the comments
and fix the code style, and then commit it.
     Thanks for reviewing it! ;-)
>      Thanks!
>   


-- 
Bo Yang, Software Engineer, Suse Labs
GPG-key-ID   538C4C1A
Samba Team   boyang at samba.org    http://www.samba.org/
SUSE Linux   boyang at suse.de      http://www.novell.com/



More information about the samba-technical mailing list