Opened 6 years ago

Closed 3 years ago

#3450 closed enhancement (complete)

Extend JSON parser with C, C++ comments, include files

Reported by: tomek Owned by: tomek
Priority: medium Milestone: Kea1.2
Component: remote-management Version: git
Keywords: Cc:
CVSS Scoring: Parent Tickets:
Sensitive: no Defect Severity: N/A
Sub-Project: DHCP Feature Depending on Ticket:
Estimated Difficulty: 0 Add Hours to Ticket: 0
Total Hours: 0 Internal?: no

Description (last modified by tomek)

After #3400 work, the current JSON parser is able to omit
bash-style (# in the first line) comments. It would be useful
to add C (/* ... */) and C++ ( ...) style comments.
Also, starting # comments somewhere in the middle would be
nice, too.

We can consider adding a capability to include one
file from another.

Finally, Tom proposed some improvement ideas in #3400:comment:9.

Subtickets

Change History (10)

comment:1 Changed 6 years ago by tomek

  • Milestone changed from Kea-proposed to Kea1.0

comment:2 Changed 5 years ago by tomek

  • Version set to git

#3960 has been created for the # comments in the middle of a line.

Last edited 5 years ago by tomek (previous) (diff)

comment:3 Changed 5 years ago by marcin

  • Milestone changed from Kea1.0 to Kea1.1

Deferring from 1.0 as per 1.0 tickets scrub.

comment:4 Changed 4 years ago by tomek

  • Milestone changed from Kea1.1 to DHCP Outstanding Tasks

comment:5 Changed 4 years ago by tomek

  • Milestone changed from DHCP Outstanding Tasks to Outstanding Tasks

Milestone renamed

comment:6 Changed 4 years ago by tomek

  • Description modified (diff)
  • Milestone changed from Outstanding Tasks to Kea1.2

comment:7 Changed 3 years ago by tomek

  • Component changed from configuration to remote-management

comment:8 Changed 3 years ago by fdupont

If you believe there is more to do please give me the list (e.g., recursive include unit test) and the ticket. If not please close.

comment:9 Changed 3 years ago by tomek

  • Owner set to tomek
  • Status changed from new to reviewing

This is implemented as part of #5017 (v4, in review) and #5036 (v6, merged). Moving to review.

comment:10 Changed 3 years ago by tomek

  • Resolution set to complete
  • Status changed from reviewing to closed

This is now implemented and on master, closing ticket.

Note: See TracTickets for help on using tickets.