Re: [Vm-dev] [OpenSmalltalk/opensmalltalk-vm] macos: Fix --trace for the PharoVM (#231)

zecke commented on this pull request.

@@ -166,7 +168,7 @@ - (int) parseArgument: (NSString *) argData peek: (char *) peek

/* Options with no arguments */

NS_DURING; - if ([argData compare: VMOPTIONOBJ("psn_") options: NSLiteralSearch range: NSMakeRange(0,5)] == NSOrderedSame) { + if ([argData compare: VMOPTIONOBJ("psn_") options: NSLiteralSearch range: NSMakeRange(0,VMOPTIONLEN(5))] == NSOrderedSame) {

Good point. OSX doesn't care about the number of dashes we have. I don't like the code. We should strip the dashes before parsing the arguments but that would have been a bigger change and outside the timebox I had. :(