[prev in list] [next in list] [prev in thread] [next in thread] 

List:       forgerock-opendj-dev
Subject:    [Opendj-dev] OPENDJ-82
From:       nemanja.lukic () profiq ! cz (=?UTF-8?Q?Nemanja_Luki=C4=87?=)
Date:       2012-01-27 10:57:28
Message-ID: CAPu2gq5GEQwr9j_EQ-M2rEWA+0psqONKAdbXc=vV+r4PeqcgDw () mail ! gmail ! com
[Download RAW message or body]

    Hello everyone,

  Regarding the proposed solution for OPENDJ-82 (
https://bugster.forgerock.org/jira/browse/OPENDJ-82). Modifying the
existing methods for tab based formatting was quite straightforward,
however refactoring them to use common output-generating code seems to be a
bit tricky. That is, IMHO, encapsulating the part of the common code would
not bring much benefit since each method does a lot of specific processing
and separating common parts might not be of much value.
  I am attaching here the patch - someone might see something I didn't.

Kind regards,
Nemanja Luki?
senior software engineer
http://www.profiq.cz
-------------- next part --------------
An HTML attachment was scrubbed...
URL: http://lists.forgerock.org/pipermail/opendj-dev/attachments/20120127/2327f446/attachment-0001.html 
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: image/jpeg
Size: 31274 bytes
Desc: not available
Url : http://lists.forgerock.org/pipermail/opendj-dev/attachments/20120127/2327f446/attachment-0001.jpe 
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: image/png
Size: 27511 bytes
Desc: not available
Url : http://lists.forgerock.org/pipermail/opendj-dev/attachments/20120127/2327f446/attachment-0001.png 
-------------- next part --------------
A non-text attachment was scrubbed...
Name: opendj-82.patch
Type: application/octet-stream
Size: 17238 bytes
Desc: not available
Url : http://lists.forgerock.org/pipermail/opendj-dev/attachments/20120127/2327f446/attachment-0001.obj 

[prev in list] [next in list] [prev in thread] [next in thread] 

Configure | About | News | Add a list | Sponsored by KoreLogic