On Sat, 3 Aug 2019, Ondrej Pokorny wrote:
On 13.07.2019 21:26, Michael Van Canneyt wrote:
I think all sides have now been reviewed to the point of boring or
annoying each other almost to death, and we finally need to decide on
whether the patch is applied, and if so, which parts of it.
Tha
Am 02.08.2019 um 21:27 schrieb bla...@blaise.ru:
On 02.08.2019 21:36, bla...@blaise.ru wrote:
embed a copy of the body of insert_struct_hidden_paras into
TVMTBuilder.generate_vmt, then merge those two procdef-member
traversals into one (hey, performance!)
Would you guys oppose such a change?
Before the main course, I offer the attached refactorings for
trecorddef.create_global_internal:
1) streamline insertions into the symtable;
2) avoid shadow-copying of the parameter "n", which is now const
--
βþ
# HG changeset patch
# User Blaise
# Date 1564833600 -10800
# Sat Aug 03 15:00:
On 03.08.2019 15:01, Sven Barth via fpc-devel wrote:
In principle one could do that, though more often than not inside the compiler
maintainability beats performance. I'd prefer an opinion of Florian and/or
Jonas on this though...
Leaving the issue of current_filepos for a moment, the change