[ 
https://issues.apache.org/jira/browse/DERBY-3936?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Kristian Waagan updated DERBY-3936:
-----------------------------------

    Attachment: derby-3936-1a-CharacterStreamDescriptor.diff

'derby-3936-1a-CharacterStreamDescriptor.diff' is the first revision of the 
CharacterStreamDescriptor class.

There is no code showing how it will be used yet, but I plan to pass it to 
UTF8Reader so that it can properly configure itself.
Patch ready for review.

What you people think of the builder pattern used?
The reasons I tried it are to avoid a constructor with a load of arguments, 
avoid having a set of many constructors and a wish to keep the descriptor 
object immutable.

> Add CharacterStreamDescriptor
> -----------------------------
>
>                 Key: DERBY-3936
>                 URL: https://issues.apache.org/jira/browse/DERBY-3936
>             Project: Derby
>          Issue Type: Sub-task
>          Components: JDBC
>    Affects Versions: 10.5.0.0
>            Reporter: Kristian Waagan
>            Assignee: Kristian Waagan
>            Priority: Minor
>         Attachments: derby-3936-1a-CharacterStreamDescriptor.diff
>
>
> A set of state information is required about byte streams representing 
> characters to be able to handle them properly.
> The basic pieces of information are:
>  - is bufferable
>  - is position aware
>  - data offset
>  - current byte position
>  - current char position
>  - byte length
>  - char length
> Additional info:
>  - encoding
>  - max allowed length
>  - bytes per char information
> The information is chosen with UTF8Reader in mind, and the plan is to pass a 
> CharacterStreamDescriptor instance to the constructor to allow the reader to 
> configure itself appropriately. The information is expected to be valid only 
> at a specific point in time, as some of it will change as soon as the reader 
> starts working with the underlying stream.

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.

Reply via email to