Skip to content

Conversation

@chcg
Copy link
Contributor

@chcg chcg commented Nov 12, 2016

chcg added 3 commits November 12, 2016 20:24
 v3.1 and merged them with local ones

- adaptations for x64
- added appveyor.yml
- added gitignore
…mjson/files/mjson/mjson-1.7.0.tar.gz/download

- merge with json modifications from this repo
- removed unused json_helper.c
- corrected x86 -> win32 in vs solution
@kapilratnani kapilratnani merged commit 6742afb into NPP-JSONViewer:master Nov 14, 2016
@kapilratnani
Copy link
Collaborator

Hi,
Thanks for this mega commit. I see you have fixed lots of things. I made this plugin when i was a novice C++ programmer, still i am. I mostly work in Java and Python. Anyway i'll go ahead and merge it.

@chcg
Copy link
Contributor Author

chcg commented Nov 14, 2016

Biggest part was the merge\takeover of the N++ part. Maybe a little restructuring would make it easier to distinguish between the files just used directly from the template and the ones modified\ created by this project and the other thirdparty ones like the json files. But that is more interesting for outstanding guys like me who wan't to contibute.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants