<div dir="ltr">I see, you're proposing to change Parser::ParseVersionTuple() and maybe VersionTuple::tryParse(). Sounds fine to me.</div><div class="gmail_extra"><br><div class="gmail_quote">On Tue, Sep 30, 2014 at 4:52 PM, jahanian <span dir="ltr"><<a href="mailto:fjahanian@apple.com" target="_blank">fjahanian@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">Sure. Our primary interest is in their use in availability/deprecated attributes. <div><br><div>An example would be:</div><div><div style="margin:0px;font-size:11px;font-family:Menlo">@protocol P</div><div style="margin:0px;font-size:11px;font-family:Menlo">- (void)proto_method __attribute__((availability(macosx,introduced=10_1_0,deprecated=10_2)));</div><div style="margin:0px;font-size:11px;font-family:Menlo">@end</div><div style="margin:0px;font-size:11px;font-family:Menlo"><br></div><div style="margin:0px;font-size:11px;font-family:Menlo">Instead of:</div><div style="margin:0px;font-size:11px;font-family:Menlo"><div style="margin:0px">@protocol P</div><div style="margin:0px">- (void)proto_method __attribute__((availability(macosx,introduced=10.1.0,deprecated=10.2)));</div><div style="margin:0px">@end</div><div style="margin:0px"><br></div><div style="margin:0px">But, 'major[.minor[.subminor]]’ are parsed and ‘.’ are checked for, irrespective of their use. </div><div style="margin:0px">So, rather than special case for what we need, I am proposing to allow them everywhere.</div><div style="margin:0px"><br></div><div style="margin:0px">- Fariborz</div><div style="margin:0px"><br></div></div><div><div class="h5"><div><div>On Sep 30, 2014, at 4:45 PM, Reid Kleckner <<a href="mailto:rnk@google.com" target="_blank">rnk@google.com</a>> wrote:</div><br><blockquote type="cite"><div dir="ltr">Can you elaborate on what exactly you're planning to change? Just the output of -help, or do we parse LLVM's version somewhere?</div><div class="gmail_extra"><br><div class="gmail_quote">On Tue, Sep 30, 2014 at 3:47 PM, jahanian <span dir="ltr"><<a href="mailto:fjahanian@apple.com" target="_blank">fjahanian@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">I would like to move on with this if there is no objections.<div><br><div>- Fariborz</div><div><br><div><div><div><div>On Sep 29, 2014, at 4:41 PM, jahanian <<a href="mailto:fjahanian@apple.com" target="_blank">fjahanian@apple.com</a>> wrote:</div><br></div></div><blockquote type="cite"><div><div><div style="word-wrap:break-word"><div><div><br></div><blockquote type="cite"><div style="word-wrap:break-word">Hi All,<div><div>We have internal request to allow ‘_’ in addition to ‘.’ as version tuple separators.</div><div>So, in addition to <font face="Menlo"><span style="font-size:11px">'major[.minor[.subminor]]’, proposal is to allow ‘major[_minor[_subminor]]’</span></font></div><div><font face="Menlo"><span style="font-size:11px">as well. Is there a reason we shouldn’t do this?</span></font></div><div><font face="Menlo"><span style="font-size:11px"><br></span></font></div><div><font face="Menlo"><span style="font-size:11px">- Thanks, Fariborz</span></font></div><div><font face="Menlo"><span style="font-size:11px"><br></span></font></div></div></div></blockquote></div><br></div></div></div>_______________________________________________<br>cfe-dev mailing list<br><a href="mailto:cfe-dev@cs.uiuc.edu" target="_blank">cfe-dev@cs.uiuc.edu</a><br><a href="http://lists.cs.uiuc.edu/mailman/listinfo/cfe-dev" target="_blank">http://lists.cs.uiuc.edu/mailman/listinfo/cfe-dev</a><br></blockquote></div><br></div></div></div><br>_______________________________________________<br>
cfe-dev mailing list<br>
<a href="mailto:cfe-dev@cs.uiuc.edu" target="_blank">cfe-dev@cs.uiuc.edu</a><br>
<a href="http://lists.cs.uiuc.edu/mailman/listinfo/cfe-dev" target="_blank">http://lists.cs.uiuc.edu/mailman/listinfo/cfe-dev</a><br>
<br></blockquote></div><br></div>
</blockquote></div><br></div></div></div></div></div></blockquote></div><br></div>