MKDataScanner icon indicating copy to clipboard operation
MKDataScanner copied to clipboard

isAtEnd implementation is the inverse of how NSScanner works

Open FergalMohan opened this issue 9 years ago • 5 comments

I think that isAtEnd should really be opposite of the way it is implemented to conform with how NSScanner is implemented i.e.

  • (BOOL)isAtEnd { return (self.offset >= self.data.length); }

FergalMohan avatar Oct 11 '15 08:10 FergalMohan

Pull Request with some tests?

krzyzanowskim avatar Oct 11 '15 08:10 krzyzanowskim

Hi Marcin, I'm not familiar with exactly how ? Do I make a Pull Request and then make the change with a corresponding Unit Test so that you can review ? Fergal

FergalMohan avatar Oct 11 '15 09:10 FergalMohan

yes, please

krzyzanowskim avatar Oct 11 '15 17:10 krzyzanowskim

Hi Marcin, I did that just now, sorry about the delay.Fergal From: Marcin Krzyzanowski [email protected] To: krzyzanowskim/MKDataScanner [email protected] Cc: FergalMohan [email protected] Sent: Sunday, October 11, 2015 6:31 PM Subject: Re: [MKDataScanner] isAtEnd implementation is the inverse of how NSScanner works (#2)

yes, please— Reply to this email directly or view it on GitHub.

FergalMohan avatar Oct 12 '15 19:10 FergalMohan

Hi Marcin, is there something that needs to be done to the PodSpec to make these latest changes available when a "pod update" is done ? Fergal

FergalMohan avatar Oct 14 '15 22:10 FergalMohan