AssetsManagerDelegateProtocol is needed by previous design. (AssetsManager::setDelegate) It provides just a closure-like thin wrapper.

2013. 9. 6., 오후 3:45, minggo <notifications@github.com> 작성:

> In extensions/AssetsManager/AssetsManager.cpp:
> 
> > +    class DelegateProtocolImpl : public AssetsManagerDelegateProtocol 
> > +    {
> > +    public :
> > +        DelegateProtocolImpl(ErrorCallback errorCallback, ProgressCallback progressCallback, SuccessCallback successCallback)
> > +        : errorCallback(errorCallback), progressCallback(progressCallback), successCallback(successCallback)
> > +        {}
> > +
> > +        virtual void onError(AssetsManager::ErrorCode errorCode) { errorCallback(int(errorCode)); }
> > +        virtual void onProgress(int percent) { progressCallback(percent); }
> > +        virtual void onSuccess() { successCallback(); }
> > +
> > +    private :
> > +        ErrorCallback errorCallback;
> > +        ProgressCallback progressCallback;
> > +        SuccessCallback successCallback;
> > +    };
> It is a smart design, so AssetsManagerDelegateProtocol is not needed.
> Right?
> 
> —
> Reply to this email directly or view it on GitHub.
>