Page 2 of 2

Re: t.version

Posted: Thu Sep 22, 2011 6:29 am
by miko
hryx wrote:I have opened this up as ticket #299 on the issue tracker.

Miko, I referred to your parsing statement. But I took off the final "%.", which is superfluous.
Except in cases like 0.8.0-BETA6 ;) OK, maybe this is not relevant. Anyways, thanks for creating the ticket.

Re: t.version

Posted: Sat Nov 05, 2011 7:58 am
by hryx
Sorry for digging up this somewhat old thread. I'm not very regularly active on forums.

Anyway, just to update about the ticket: Bart marked it invalid, but I don't quite understand why. It seems like either I'm misunderstanding something or he is. Or maybe my explanation was unclear.

Can anyone chime in on this topic, either in this thread or on the Bitbucket ticket? I don't want to annoy the devs by just opening it up again without a convincing reason...

Re: t.version

Posted: Sat Nov 05, 2011 12:02 pm
by bartbes
t.version was never used, and as such no code existed treating it either way. Saying it was a number or a string was simply a thing with documentation, not with the code.

Re: t.version

Posted: Sat Nov 05, 2011 8:36 pm
by hryx
Right, but will it be used in future versions of Love? What I was proposing is not a matter of how it should be documented, but how it should be treated by Love. The fact that it isn't used at all yet makes this the perfect time to figure out how it could/should be handled.

So, I was making a suggestion for how Love should handle it. If there are no plans for it, or it's just decorative, then I don't believe it belongs in love.conf.

Re: t.version

Posted: Sat Nov 05, 2011 9:02 pm
by Robin
Actually, in the 0.8.0 boot.lua shows that it is being used, in the way proposed in the issue report.

(By the way, I think there should be a break after compat = true.)

Re: t.version

Posted: Sat Nov 05, 2011 9:06 pm
by bartbes
But that was after. At the time it was a non-issue. (See what I did there?)

Re: t.version

Posted: Sat Nov 05, 2011 10:34 pm
by hryx
bartbes wrote:But that was after. At the time it was a non-issue. (See what I did there?)
Very clever. :|

Thanks for pointing out the boot.lua, Robin.

I updated the wiki page.