CoreParse icon indicating copy to clipboard operation
CoreParse copied to clipboard

A feature added, and gitignore file updated

Open V01dZer0 opened this issue 9 years ago • 4 comments

  1. Added shouldQuoteEscapeSequence for CPQuotedRecogniser.

This property is very useful when parse multiline string in source code. e.g. : a C string :

"Quick brown\
fox jumps over a lazy dog"

should be parsed into Quick brown fox jumps over a lazy dog, not Quick brown\ fox jumps over a lazy dog. And this work will be done only to set shouldQuoteEscapeSequence to YES.

  1. Removed shared project config files, and updated gitignore file.

V01dZer0 avatar Sep 01 '15 15:09 V01dZer0

As I commented in the other patch - I'm not sure this makes sense. Isn't this just an escapeRecogniser?

beelsebob avatar Sep 01 '15 23:09 beelsebob

@beelsebob I didn't find escapeRecogniser in CoreParse project, did you mean escapeReplacer ?

V01dZer0 avatar Sep 02 '15 13:09 V01dZer0

@OpenFibers its CPQuotedRecogniser https://github.com/beelsebob/CoreParse/blob/master/CoreParse/Tokenisation/Token%20Recognisers/CPQuotedRecogniser.h

siuying avatar Sep 02 '15 13:09 siuying

@beelsebob I see the escapeReplace covers the work shouldQuoteEscapeSequence can do, so forget shouldQuoteEscapeSequence, it's a bad idea.
But you may want to merge commits of .gitignore and remove local config files.

V01dZer0 avatar Sep 05 '15 14:09 V01dZer0