search

zserge / jsmn

Jsmn is a world fastest JSON parser/tokenizer. This is the official repo replacing the old one at Bitbucket
MIT License
3.65k stars 778 forks source link

fatal bug #173

Closed MyTDT-Mysoft closed 4 years ago

MyTDT-Mysoft commented 4 years ago

i couldnt even started using it... when i found the worst bug of al...

THERES FUCKING CODE ON A .H

fix it IMEDIATELLY please... that is totally unaceptable

pt300 commented 4 years ago

That was a choice made some time ago. You can read more in #149 and the README. If you have constructive criticism of that decision you are more than welcome to share it with everyone.

MyTDT-Mysoft commented 4 years ago

no i will go back to breah gasoline, while putting melon juice on my car, while finish my catalog of all the types of air that i'm selling...

MyTDT-Mysoft commented 4 years ago

its very retarded you name standards that it can be compiled when you broke the rule #1 no code on .h.. they are .headers not .code, so if you will do stupid things then why even bother naming it .h? name it library.jsmn directly... since you made the extension be meaningless...

pt300 commented 4 years ago

So doing it the way it has been done here is wrong just because, despite the problems it actually helped solve?

MyTDT-Mysoft commented 4 years ago

having it named as .c instead of .h would not cause any problems...