Return-Path: Delivered-To: apmail-directory-dev-archive@www.apache.org Received: (qmail 57615 invoked from network); 25 Jan 2011 04:55:27 -0000 Received: from hermes.apache.org (HELO mail.apache.org) (140.211.11.3) by minotaur.apache.org with SMTP; 25 Jan 2011 04:55:27 -0000 Received: (qmail 1026 invoked by uid 500); 25 Jan 2011 04:55:27 -0000 Delivered-To: apmail-directory-dev-archive@directory.apache.org Received: (qmail 819 invoked by uid 500); 25 Jan 2011 04:55:24 -0000 Mailing-List: contact dev-help@directory.apache.org; run by ezmlm Precedence: bulk List-Help: List-Unsubscribe: List-Post: List-Id: Reply-To: "Apache Directory Developers List" Delivered-To: mailing list dev@directory.apache.org Received: (qmail 812 invoked by uid 99); 25 Jan 2011 04:55:24 -0000 Received: from athena.apache.org (HELO athena.apache.org) (140.211.11.136) by apache.org (qpsmtpd/0.29) with ESMTP; Tue, 25 Jan 2011 04:55:24 +0000 X-ASF-Spam-Status: No, hits=-0.7 required=10.0 tests=FREEMAIL_FROM,RCVD_IN_DNSWL_LOW,SPF_PASS X-Spam-Check-By: apache.org Received-SPF: pass (athena.apache.org: domain of akarasulu@gmail.com designates 209.85.216.50 as permitted sender) Received: from [209.85.216.50] (HELO mail-qw0-f50.google.com) (209.85.216.50) by apache.org (qpsmtpd/0.29) with ESMTP; Tue, 25 Jan 2011 04:55:19 +0000 Received: by qwd6 with SMTP id 6so5041247qwd.37 for ; Mon, 24 Jan 2011 20:54:58 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=gamma; h=domainkey-signature:mime-version:sender:date:x-google-sender-auth :message-id:subject:from:to:content-type; bh=rW/YRsnpUnj2+NYbublrPazaZt8wrAKkQCnjZpQkT3Q=; b=f22abSyvJ28lg/nAs6KMtEqJLoaRwAiMX7NpBKDfmsYTQmn5PhIK0Iv3JVgXY2ddIZ aUKqx8sMCtqXlV9vx8ZHqFyKe+cHUssaxHTPP+UrovPCUw1cAWhAlVk+ijvS+fS76nkQ 1d/jAi0PYMnvnf+ElvuXMC7LkHm6FoBWzYS1w= DomainKey-Signature: a=rsa-sha1; c=nofws; d=gmail.com; s=gamma; h=mime-version:sender:date:x-google-sender-auth:message-id:subject :from:to:content-type; b=L2Q6wEmkXwW9usoZ6s0aRfIFZnA/OA9ZAGP3PCQur0p82QA/8cxLfiLiCv2o1uWHUa LWyiHeyS9VraoynwBw2u0xeudC191QIQuO3ZVM929b0GGl5JMuIRQds3uV+iXYTXfZMW /lu16WCbrSEw4d4Le62zdgbSM2uMwfQFdigAc= MIME-Version: 1.0 Received: by 10.229.229.209 with SMTP id jj17mr4409484qcb.273.1295931298374; Mon, 24 Jan 2011 20:54:58 -0800 (PST) Sender: akarasulu@gmail.com Received: by 10.229.246.148 with HTTP; Mon, 24 Jan 2011 20:54:58 -0800 (PST) Date: Tue, 25 Jan 2011 06:54:58 +0200 X-Google-Sender-Auth: plm_2hDE5i9jgtMbBtmDFx2mfMo Message-ID: Subject: [Codec] Refactoring status From: Alex Karasulu To: Apache Directory Developers List Content-Type: text/plain; charset=ISO-8859-1 Although I was going to apply the encoding decorator pattern last (STEP 5 in previous status) I started discussing the approach with Emmanuel last night and that kicked off the step ahead of the others. And it's not so bad to do this first. Now I have a few issues. First I am running out of time, need to crash but could not fix two failing test cases: Failed tests: testDecodeDelResponseSuccessWithControls(org.apache.directory.shared.ldap.codec.del.DelResponseTest) testDecodeDelResponseSuccess(org.apache.directory.shared.ldap.codec.del.DelResponseTest) This regression appeared after creation and check in of the LdapResultDecorator. Emmanuel if you could take a look at this for me tomorrow morning this would be great. Now the other bigger issue. There's no way I could finish the SearchRequestImpl decoupling. I created the decoder and applied it. It seems to be working just fine but the problem is SearchRequestImpl has several codec dependencies. Seems there's a whole Filter subsystem for the codec doing what the ExprNode does for LDAP filter creation in the model. I'm hoping Emmanuel you can lend a hand here since you're most familiar with this code. If we can get this knocked out then I think we can bud ldap-model out right after into it's own maven module as an OSGi bundle. Thanks, -- Alex Karasulu My Blog :: http://www.jroller.com/akarasulu/ Apache Directory Server :: http://directory.apache.org Apache MINA :: http://mina.apache.org To set up a meeting with me: http://tungle.me/AlexKarasulu