Add WideString test for Issue #756

[BillyDonahue]

Testing shows no problems on the jsoncpp side.

[cdunn2001]

Hmm. This fails in AppVeyor for VS2012. Worth addressing?

[hjmjohnson]

@BillyDonahue Is this worth rebase on the current master branch?

[BillyGoto]

I’ll look into it

On Thu, Jan 17, 2019 at 8:29 PM Hans Johnson ***@***.***> wrote: @BillyDonahue <https://github.com/BillyDonahue> Is this worth rebase on the current master branch? — You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub <#781 (comment)>, or mute the thread <https://github.com/notifications/unsubscribe-auth/AHFzPLWjO2Ix-Z0aVGv-tEOrBR5LO1yoks5vESOTgaJpZM4UQbEx> .

-- ǝnɥɐuop ʎllıq

[baylesj]

Is this still relevant? I would like to merge or close all the outstanding PRs... especially the old ones like this one.

[baylesj]

Looked into this, VS 2012 and 2013 don't support the u8 operator. Considering it's now 2019, I'd be open to rolling the dependency to VS 2015 or something more modern, which would resolve this issue.

[baylesj]

I'm going to roll our Travis and Appveyor dependencies, then hopefully this should just be plug and play.

#955

[baylesj]

Running tests in a separate PR since I couldn't figure out how to kick Travis here:

#956

If they pass, I'll merge this and feel good about it.

[baylesj]

All checks passed. Merging!