stsievert / swix

Swift Matrix Library
http://docs.stsievert.com/swix/
MIT License
593 stars 54 forks source link

Migration to newer constructors #3

Closed donald-pinckney closed 10 years ago

donald-pinckney commented 10 years ago

With Swift still changing, factor methods are being replaced by constructors. Not sure what timeline is best for this, but at some point code such as:

var x = String.stringWithContentsOfFile(prefix+"../"+filename, encoding: NSUTF8StringEncoding, error: nil)

needs to be changed to:

var x = String(contentsOfFile: prefix+"../"+filename, encoding: NSUTF8StringEncoding, error: nil)

to continue building (thats necessary as of Xcode 6.1 beta 2)

ghost commented 10 years ago

Donald, just want to mention I think you’re doing a great thing with Swix. There’s a lot of silence out there of people on the fulcrum of doing what you’re doing and I think it’s going to gain serious momentum.

Thanks again for your great lib.

On Sep 16, 2014, at 7:51 PM, Donald Pinckney notifications@github.com wrote:

With Swift still changing, factor methods are being replaced by constructors. Not sure what timeline is best for this, but at some point code such as:

var x = String.stringWithContentsOfFile(prefix+"../"+filename, encoding: NSUTF8StringEncoding, error: nil) needs to be changed to:

var x = String(contentsOfFile: prefix+"../"+filename, encoding: NSUTF8StringEncoding, error: nil) to continue building (thats necessary as of Xcode 6.1 beta 2)

— Reply to this email directly or view it on GitHub https://github.com/scottsievert/swix/issues/3.

donald-pinckney commented 10 years ago

You mean Scott, not I. But yes, I would say the same thing for this library.

stsievert commented 10 years ago

Thanks guys! This library was built out of pure need; I wouldn't be surprised if it was needed elsewhere as well.

Oh, and it fully compiles under Xcode 6.1b2 now.