Modify ↓

Ticket #97 (closed enhancement: wontfix)

Opened 5 years ago

Last modified 4 years ago

Reader can be an instance variable in Scanner

Reported by: py4fun@… Owned by: xi
Priority: normal Component: pyyaml
Severity: normal Keywords: Reader, Scanner
Cc:

Description

Hi, thank you for PyYAML. Unfortunately I need to exchange YAML between Java and Python and Java does not have such a good library as PyYAML. There is a port of PyYAML (JvYAML) but it is dead.
I would like to develop the Java port further that it reaches the quality of PyYAML. To simplify maintenance I would like to be as close to the PyYAML API as possible.
May I propose to make Reader an instance variable in Scanner?
Can you please apply the attached patch to achieve the following:

  • improve encapsulation. Scanner and Reader can be instantiated and tested separately
  • Java can use the same approach (Java does not have multiple inheritance)

All the tests succeed.

Thank you,

Andrey

Attachments

reader.patch Download (47.4 KB) - added by py4fun@… 5 years ago.

Change History

Changed 5 years ago by py4fun@…

comment:1 Changed 4 years ago by py4fun@…

  • Status changed from new to closed
  • Resolution set to wontfix

It is better to keep it as it is.

View ↑

Add a comment

Modify Ticket

Change Properties
<Author field>
Action
as closed
The resolution will be deleted. Next status will be 'reopened'
Author


E-mail address and user name can be saved in the Preferences.

Attachments ↑
 
Note: See TracTickets for help on using tickets.