<div dir="ltr">Oh right, I saw that pull request shortly after I sent my last message. It seems hpux735 also thought locale needed to be initialized. I will follow the process you mentioned in the future, if I have a similar issue.<div><br></div><div>Although, all that is unrelated to my latest question of getting some of the NSDateFormatter properties their proper default values?</div></div><div class="gmail_extra"><br><div class="gmail_quote">On Fri, Feb 26, 2016 at 6:10 PM, Philippe Hausler <span dir="ltr"><<a href="mailto:phausler@apple.com" target="_blank">phausler@apple.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div style="word-wrap:break-word"><div>NSLocale as nil infers the current locale so that might be what is missing. Since the Objective-C implementation’s behavior is the standard to judge against it is reasonable to check in tests that fail (preferably disabled so that CI passes and a bug associated with it if you are uncertain of the correct implementation).</div><div><br></div><div>There was recently a merged pull request that changed some of this iirc: <a href="https://github.com/apple/swift-corelibs-foundation/commit/89556789cf1003dfa137eec9eed43ed965f3ea74" target="_blank">https://github.com/apple/swift-corelibs-foundation/commit/89556789cf1003dfa137eec9eed43ed965f3ea74</a></div><div><div class="h5"><br><div><blockquote type="cite"><div>On Feb 26, 2016, at 3:58 PM, Taylor Franklin <<a href="mailto:taylorleefranklin@gmail.com" target="_blank">taylorleefranklin@gmail.com</a>> wrote:</div><br><div><div dir="ltr">Philippe,<div><br></div><div>Thanks for the response, in working on NSDateFormatter, I have noticed that using the <font face="monospace, monospace">_cfObject</font> will cause a runtime error if <font face="monospace, monospace">locale</font> is simply nil and has no default value. With that said, not sure how to prove in a test since it crashes.</div><div><br></div><div>I did have another question though, I am unsure as how or where some of the NSDateFormatter properties get their initial values.</div><div>For example, using the production version of Foundation and printing the <font face="monospace, monospace">.weekdaySymbols</font> property gives me:</div><div><font face="monospace, monospace">["Sunday", "Monday", "Tuesday", "Wednesday", "Thursday", "Friday", "Saturday"]</font></div><div><font face="monospace, monospace"><br></font></div><div><font face="arial, helvetica, sans-serif">While in the development version of Foundation, that property is never set. Do you have any insights on where those default weekday values come from given a user's current locale? I assume the answer would apply to other properties like </font><font face="monospace, monospace">longEraSymbols</font><font face="arial, helvetica, sans-serif">, </font><font face="monospace, monospace">veryShortMonthSymbols</font><font face="arial, helvetica, sans-serif">, etc. </font></div>
<div><br></div><div>Thanks,</div><div>Taylor</div><div><br></div></div><div class="gmail_extra"><br><div class="gmail_quote">On Mon, Feb 22, 2016 at 11:10 AM, Philippe Hausler <span dir="ltr"><<a href="mailto:phausler@apple.com" target="_blank">phausler@apple.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div style="word-wrap:break-word"><div>Some responses inline:</div><div><br></div><br><div><span><blockquote type="cite"><div>On Feb 21, 2016, at 10:22 PM, Taylor Franklin <<a href="mailto:taylorleefranklin@gmail.com" target="_blank">taylorleefranklin@gmail.com</a>> wrote:</div><br><div><div dir="ltr">Thanks Joe,<div><br></div><div>I will probably end up taking some of your work and execute Philippe's suggestion of testing with a dictionary. In the process of adding tests, hopefully I can improve NSDateFormatter over the next few weeks.</div></div></div></blockquote><div><br></div></span><div>Awesome, feel free to reach out if you have questions on the appropriate behaviors/edge cases.</div><span><br><blockquote type="cite"><div><div dir="ltr"><div><br></div><div>@Philippe, I did have a question:</div><div><br></div><div>Currently, many of the properties, such as locale, look like this:</div><div><br></div><div><font face="monospace, monospace">/*@NSCopying*/ public var locale: NSLocale! { willSet { _reset() } }</font></div></div></div></blockquote><div><br></div></span><div>This was changed to support the willSet pre-operation.</div><span><br><blockquote type="cite"><div><div dir="ltr"><div><font face="monospace, monospace"><br></font></div><div><font face="arial, helvetica, sans-serif">When previously, the looked like this:</font></div><div><font face="arial, helvetica, sans-serif"><br></font></div><div><div style="font-family:monospace,monospace">internal var _locale: NSLocale = NSLocale.currentLocale()</div><div style="font-family:monospace,monospace">/*@NSCopying*/ public var locale: NSLocale! {</div><div style="font-family:monospace,monospace"> get {</div><div style="font-family:monospace,monospace"> return _locale</div><div style="font-family:monospace,monospace"> }</div><div style="font-family:monospace,monospace"> set {</div><div style="font-family:monospace,monospace"> _reset()</div><div style="font-family:monospace,monospace"> _locale = newValue</div><div style="font-family:monospace,monospace"> }</div><div style="font-family:monospace,monospace">}</div><div style="font-family:monospace,monospace"><br></div><div><font face="arial, helvetica, sans-serif">Do you know a reason for this change? It seems mostly the same except the there is no default value set anymore.</font></div><div><font face="arial, helvetica, sans-serif">The pull request I'm referring to that made these changes is this: <a href="https://github.com/apple/swift-corelibs-foundation/pull/234" target="_blank">https://github.com/apple/swift-corelibs-foundation/pull/234</a></font></div></div></div></div></blockquote><div><br></div></span><div>The change should have been a non-functional change; if there are behavioral differences it is definitely a bug. The original version of that was written to be very close to the objective-c version (to the point that I actually inadvertently replicated a few bugs…)</div><div><br></div><div>If you can show via test that the behavior is different we can look into reverting/fixing date formatter.</div><div><div><br><blockquote type="cite"><div><div dir="ltr"><div><div><font face="arial, helvetica, sans-serif"><br></font></div><div><font face="arial, helvetica, sans-serif">Thanks!</font></div><div><font face="arial, helvetica, sans-serif">Taylor</font></div><div class="gmail_extra"><br><div class="gmail_quote">On Sun, Feb 21, 2016 at 7:17 AM, Joseph Bell <span dir="ltr"><<a href="mailto:joe@iachieved.it" target="_blank">joe@iachieved.it</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex"><div dir="ltr">Ah, I didn't even submit a pull request - that's how distracted I've been. Again, the code is all yours to rework or base off of if you so choose.</div><div><div><div class="gmail_extra"><br><div class="gmail_quote">On Sun, Feb 21, 2016 at 7:14 AM, Joseph Bell <span dir="ltr"><<a href="mailto:joe@iachieved.it" target="_blank">joe@iachieved.it</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex"><div dir="ltr">Thanks Taylor. Unfortunately I lost time and interest in taking the idea further, I just need to figure out how to withdraw the pull request. Feel free to take the implementation and rework per Philippe's suggestions!<div><br></div><div>Joe</div><div><br></div></div><div class="gmail_extra"><div><div><br><div class="gmail_quote">On Fri, Feb 19, 2016 at 6:35 PM, Philippe Hausler <span dir="ltr"><<a href="mailto:phausler@apple.com" target="_blank">phausler@apple.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex"><div style="word-wrap:break-word"><div>The problem with the change in that commit is that it is doing two different things: it is testing against python which is not the definition of Foundation’s expected output instead of testing against the expected output from the objective-c implementation. My suggestion in the pull request was to just encode a dictionary of dates to strings sampling a well known range before hand. This way it would be verifiable on both linux and Darwin as “correct” in accordance with the version of Foundation.framework that ships on Mac OS X and iOS. </div><div><br></div><div>The rest of the additions seemed pretty reasonable to me, except the case of calling popen to run a secondary script that may take extra execution time and add extra complexity other than just a dictionary of some good example date to string conversions (which actually could be utilized to reverse the test as well for scanning).</div><br><div><blockquote type="cite"><div><div><div>On Feb 19, 2016, at 4:13 PM, Taylor Franklin via swift-corelibs-dev <<a href="mailto:swift-corelibs-dev@swift.org" target="_blank">swift-corelibs-dev@swift.org</a>> wrote:</div><br></div></div><div><div><div><div dir="ltr"><div><div>Hello,</div><div><br></div><div>Are there any plans to integrate this commit into the main repo because I would love build off of the code within it, seeing that NSDateFormatter still seems incomplete. Additionally, the issue mentioned earlier, <a href="https://bugs.swift.org/browse/SR-208" target="_blank">https://bugs.swift.org/browse/SR-208</a>, is still valid with the latest code from master.</div><div><br></div><div>In fact, <font face="monospace, monospace">stringFromDate</font> doesn't seem to be doing much at all. Anyway, I would love to hear more from Joe or an admin on plans and progress for NSDateFormatter.</div><div><br></div><div>Thanks,</div><div>Taylor Franklin</div></div><div><br></div><div>> Howdy,</div><div>> </div><div>> A few weeks ago I opened <a href="https://bugs.swift.org/browse/SR-208" target="_blank">https://bugs.swift.org/browse/SR-208</a> as it appears</div><div>> setting the dateFormat property of NSDateFormatter has no effect. It's</div><div>> been open for a while and I thought I'd start looking at whether or not I</div><div>> could help, and decided to first start with getting NSDateFormatter</div><div>> included in TestFoundation.</div><div>> </div><div>> Before moving on further and issuing a PR, I would appreciate feedback on</div><div>> the approach that I'm taking here:</div><div>> </div><div>> <a href="https://github.com/iachievedit/swift-corelibs-foundation/commit/482d861127e8b78007ceaf15f6c905ac04b1e9a4" target="_blank">https://github.com/iachievedit/swift-corelibs-foundation/commit/482d861127e8b78007ceaf15f6c905ac04b1e9a4</a></div><div>> </div><div>> The first tests are only looking at the dateStyle property, and I've</div><div>> included tests for the various styles as they are rendered for the en_US</div><div>> locale. The intent is to add support for validating additional locales in</div><div>> the future.</div><div>> </div><div>> Since strptime doesn't appear to be available to the Glibc module I'm using</div><div>> a quick Python script included in Resources/ to take a format string and</div><div>> render a human-friendly date.</div><div>> </div><div>> At the moment I know there is a discrepancy between what NSDateFormatter</div><div>> and the python driver can emit, the python script is currently adjusting to</div><div>> my timezone and not using UTC, but that will be fixed before a PR is</div><div>> issued. I'll also add the timeStyle property and then continue to add</div><div>> tests.</div><div>> </div><div>> Thoughts and comments most welcome, and Happy New Year.</div><div>> </div><div>> Joe</div><div>> </div><div>> </div><div>> </div>
</div></div></div>
_______________________________________________<br>swift-corelibs-dev mailing list<br><a href="mailto:swift-corelibs-dev@swift.org" target="_blank">swift-corelibs-dev@swift.org</a><br><a href="https://lists.swift.org/mailman/listinfo/swift-corelibs-dev" target="_blank">https://lists.swift.org/mailman/listinfo/swift-corelibs-dev</a><br></div></blockquote></div><br></div></blockquote></div><br><br clear="all"><div><br></div></div></div><span><font color="#888888">-- <br><div><div dir="ltr">---<div><a href="http://dev.iachieved.it/iachievedit/" target="_blank">http://dev.iachieved.it/iachievedit/</a></div><div>@iachievedit</div></div></div>
</font></span></div>
</blockquote></div><br><br clear="all"><div><br></div>-- <br><div><div dir="ltr">---<div><a href="http://dev.iachieved.it/iachievedit/" target="_blank">http://dev.iachieved.it/iachievedit/</a></div><div>@iachievedit</div></div></div>
</div>
</div></div></blockquote></div><br><br clear="all"><div><br></div>-- <br><div><div dir="ltr"><div><div dir="ltr"><b><font face="arial, helvetica, sans-serif" size="2">Taylor Franklin</font></b><div>iOS Developer | IBM Mobile Innovation Lab</div><div><a href="tel:972-207-2051" value="+19722072051" target="_blank">972-207-2051</a> | <a href="mailto:taylorleefranklin@gmail.com" target="_blank">taylorleefranklin@gmail.com</a></div><div><i>Blog</i>: <a href="http://taylorfranklin.me/" style="color:rgb(17,85,204);font-size:small" target="_blank">http://taylorfranklin.me</a> | <i>LinkedIn</i>: <a href="https://www.linkedin.com/in/taylorfranklin" target="_blank">https://www.linkedin.com/in/taylorfranklin</a></div></div></div></div></div>
</div></div></div>
</div></blockquote></div></div></div><br></div></blockquote></div><br><br clear="all"><div><br></div>-- <br><div><div dir="ltr"><div><div dir="ltr"><b><font face="arial, helvetica, sans-serif" size="2">Taylor Franklin</font></b><div>iOS Developer | IBM Mobile Innovation Lab</div><div><a href="tel:972-207-2051" value="+19722072051" target="_blank">972-207-2051</a> | <a href="mailto:taylorleefranklin@gmail.com" target="_blank">taylorleefranklin@gmail.com</a></div><div><i>Blog</i>: <a href="http://taylorfranklin.me/" style="color:rgb(17,85,204);font-size:small" target="_blank">http://taylorfranklin.me</a> | <i>LinkedIn</i>: <a href="https://www.linkedin.com/in/taylorfranklin" target="_blank">https://www.linkedin.com/in/taylorfranklin</a></div></div></div></div></div>
</div>
</div></blockquote></div><br></div></div></div></blockquote></div><br><br clear="all"><div><br></div>-- <br><div class="gmail_signature"><div dir="ltr"><div><div dir="ltr"><b><font face="arial, helvetica, sans-serif" size="2">Taylor Franklin</font></b><div>iOS Developer | IBM Mobile Innovation Lab</div><div>972-207-2051 | <a href="mailto:taylorleefranklin@gmail.com" target="_blank">taylorleefranklin@gmail.com</a></div><div><i>Blog</i>: <a href="http://taylorfranklin.me/" style="color:rgb(17,85,204);font-size:small" target="_blank">http://taylorfranklin.me</a> | <i>LinkedIn</i>: <a href="https://www.linkedin.com/in/taylorfranklin" target="_blank">https://www.linkedin.com/in/taylorfranklin</a></div></div></div></div></div>
</div>