Skip to content

Update new line handling #41

Closed
#44
Closed
Update new line handling#41
#44

Description

@Teatabler
Teatabler
opened on Oct 10, 2016

In line 560, in my opinion, Chr(10) is better than vbCrLf.

559      Case "n"
560           json_BufferAppend json_buffer, vbCrLf, json_BufferPosition, json_BufferLength
561           json_Index = json_Index + 1

Activity

  1. timhall commented on Dec 29, 2016

    @timhall
    timhall
    on Dec 29, 2016
    Member

    Related: VBA-tools/VBA-Web#270

  2. timhall
    changed the title [-]Chr(10) is better than vbCrLf[/-] [+]Update new line handling[/+] on Dec 29, 2016
  3. Sophist-UK
    added 2 commits that reference this issue on Jan 9, 2017

    Update new line handling

    c25b458

    Update new line handling

    b00c968

  4. omegastripes commented on Feb 19, 2017

    @omegastripes
    omegastripes
    on Feb 19, 2017

    I converted the sample {"text":"abc\r\ndef"} to object with ParseJson, then converted it to string with ConvertToJson, returned string is { "text": "abc\r\r\ndef" }
    Seems ConvertToJson duplicates \r within strings.

  5. timhall commented on Apr 6, 2018

    @timhall
    timhall
    on Apr 6, 2018
    Member

    A little more activity on #67 closing in favor of that.

  6. timhall
    closed this as completedon Apr 6, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Type

    No type

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions