Differential D6219

Add tests for multi-line strings and fix them.
ClosedPublic
Actions

Authored by ematirov on Jun 13 2017, 5:05 PM.

Details

Reviewers
apol
brauch
Commits
R59:7d576419f3e8: Fix multiline strings behavior
Summary

Add tests for multi-line strings
Fix multi-line strings behavior




The "" string may not be multi-line (cannot contain new line char "\n") .

The `` should be correctly handled in multi-line case (can contain new line char "\n").
Test Plan
Tests pass
Diff Detail
Repository 
R59 KDevelop Go 
Lint 
Automatic diff as part of commit; lint not applicable.
 
Unit 
Automatic diff as part of commit; unit tests not applicable.
 
ematirov created this revision.Jun 13 2017, 5:05 PM
Restricted Application added a subscriber: kdevelop-devel.  ·  View Herald TranscriptJun 13 2017, 5:05 PM
apol accepted this revision.Jun 14 2017, 9:56 PM
FWIW it's probably not necessary that you post everything for review, especially if it's accompanied by a Unit Test.
This revision is now accepted and ready to land.Jun 14 2017, 9:56 PM
Closed by commit R59:7d576419f3e8: Fix multiline strings behavior (authored by ematirov).  ·  Explain WhyJun 17 2017, 4:19 PM
This revision was automatically updated to reflect the committed changes.
Revision Contents
Changeset List
PathPackages
Mparser/go.g (10 lines)
Diff 15528
View Options
parser/go.g
Loading...