<div dir="auto"><div><div class="gmail_quote"><div dir="ltr">Am Fr., 9. Nov. 2018, 03:44 hat Ryan Joseph <<a href="mailto:ryan@thealchemistguild.com">ryan@thealchemistguild.com</a>> geschrieben:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><br>
<br>
> On Nov 9, 2018, at 4:28 AM, Florian Klämpfl <<a href="mailto:florian@freepascal.org" target="_blank" rel="noreferrer">florian@freepascal.org</a>> wrote:<br>
> <br>
> I like the idea of const in generics, but it needs serious cleanup when it's working:<br>
> - commit messages like "first commit" are useless<br>
<br>
Those are for github so I could share but I need to learn SVN (again) eventually.<br></blockquote></div></div><div dir="auto"><br></div><div dir="auto">You can also use git svn. At least I myself use that for my own work (I think Florian does, too). Of course you need to set up syncing with your GitHub Repo as well, but I haven't done that for a SVN Repo yet... </div><div dir="auto"><br></div><div dir="auto"><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
> - do not commit meta files like .gitignore with a functional commit<br>
> - follow the indention style of the surrounding code<br>
<br>
Is the compiler 2 spaces for indent? There’s already enough inconsistency but I’m trying to figure it out.<br></blockquote></div></div><div dir="auto"><br></div><div dir="auto">Yes, two spaces. And "begin... end" is indented and its content is indented again. </div><div dir="auto"><br></div><div dir="auto"><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
> - the compiler contains already a type called tgenericdef<br>
<br>
I think I changed that name in the last commit. Btw I made all those types because I didn’t want to populate the main types like ttypesym with extra bytes but I don’t know what’s best practice in the codebase.<br></blockquote></div></div><div dir="auto"><br></div><div dir="auto">When something belongs in a main type then it belongs there. But as I already wrote it should not be a ttypesym for constant generic parameters anyway.</div><div dir="auto"><br></div><div dir="auto">Regards, </div><div dir="auto">Sven </div></div>