kicad-developers team mailing list archive
-
kicad-developers team
-
Mailing list archive
-
Message #35511
Re: [PATCH 4/4] Convert line endings in trace_helpers.{cpp, h} to Unix
-
To:
kicad-developers@xxxxxxxxxxxxxxxxxxx
-
From:
Wayne Stambaugh <stambaughw@xxxxxxxxx>
-
Date:
Tue, 17 Apr 2018 10:18:22 -0400
-
Autocrypt:
addr=stambaughw@xxxxxxxxx; prefer-encrypt=mutual; keydata= xsFNBFrV/VIBEACwjXP6sfRzMzy5SREVwJgVYHPVYKsDaeMZmAGV2QzSjYpGmE4RifKQYxQr 2n1RhBO8cHFVRGZdACAIgXqDW0mqvVkQKvchSotn9ZKItZiwBo1OpBrnVA4fQ7g8buM6H94S h7JOo73grpI7JwZY2uCTTQKCofoDMMWu6EPki3nkulRDT+00prvAxhF5JxKg0tmoI7vD9MpC JFS4LKxgFV7j1a4JRmiS0rJ/P32P3gxEhD2QRcPCOep3yyJ1q9mbcaPcUKY9W2DqIwYrOzbN JN8znFq3acNd39A917y1AaIdSsOLu+ru/2OFjJNB2LAyzer4toKyfRVl09xLzmg1B5QlZsKv xpg3norvkbvRVjqrmbuaMRSMGcfj41+RkksHwqVVmF/1ZFEAXUTuEFFdl/iySuFtd93xecXi RcYuUpXgkeWhXnjLjgrYuE58rARLnHYuuFQL1q3b5lIKrxBK2y25KBeafmwmamNNCXlj8ZN9 HYUPhR7UsaVzLRiIcN6bH6il+CPOBAFH/UmGyMngqkOv6ymDk5rSUBOGlOareZcrb6H3f3Gm 7Ql7Bdjpx+e5Ep//FNYHd09d/jdfLUbgMy0Q6VrolUomPUY74jk6O07XAHYwvVldphHbRPSz HAtKCFqi8kK35ITYrGRvM8Qzu1ARkMM7kUxxgxwO7sBMyLJa9wARAQABzSZXYXluZSBTdGFt YmF1Z2ggPHN0YW1iYXVnaHdAZ21haWwuY29tPsLBlAQTAQgAPhYhBDIvZUIeIpa0UdrdRRtS nXVnOciYBQJa1f1SAhsjBQkJZgGABQsJCAcCBhUKCQgLAgQWAgMBAh4BAheAAAoJEBtSnXVn OciYGdUQAJQbLb2bsaDa2hLovIBlaKON4s3KQnHhfJjXoghj+Q84VvxaSWCXO9GgWeK53NVD fz+439pSXJcRsr7j3DajLqjRnJN60ManFhNKRjEZQds/kXOMw3F9l4gJ8QYQveZ5xBHjmXux mvPf9E6GP25AwGoD+SKr6ICWVeGKXDrdiuuhRrAfxhCwuQcLCJmBXOjgOQE+l6IB+AOfwu22 Cg9ySWLM4G562Xy42RD6YvDrdDGwjnQyFb6GOcMt21zI7pKn60fYSSrm+xuUNhIIEZWMEScZ 8TfOjPLcgRbPzHrXDc8pjdVQ04IJfstKGaJV+a80egfS4B+WBhJAGvTXmXucJzZqgEGZQY0z vbG5pCo0x4zAy0tIue10GCJwOTUsUbsvgWeMGJRmJ0HqIETwdURnjDcQclLruNgi797VmUFd Zdduz5mBWt9OKXIyMMkRAig+Wsw/MogTl+SraXSNVsZELbRCGxxUbqO2JLxoTQqtakfTgIW1 0vQN0CWBsW84ulDB56WYeDHjSxF/f/oGAQYAy7I7u6suPtjJltPGKrz8+0QUyk2eU+YblwEb 5w31Joooo2QVpUaUav/3Eqr5zmAnePxeALeKDkwvNm8K3Wj6Ko1vGNkDLIvYt8fK4UPQsv7x CkHAf6havgur0aZcAnkQdLQFglYajvswCXMB1htEr7IYzsFNBFrV/VIBEADEfeL6wMxduIDn OMCSBUZSJLbhbnJYRGcjorQFsJ1Gf2fMpOkvSr3x0VM2SgDzhet/jhEZeInekWLTbWfqQhh3 UhxbZMswsLV5f9FZx6WHTBRE0sO+NXGT1NRasL73XK96L1FdQWrB5auwZxEeWadeLQ9i/72N J4k7ySsMhzpFEEhMUun9gxWzR1EVCAYzjNPijk5ucJLsHHTKyhmDs0dHe7HVV69mYcVSWlK0 P9ZveEqiZzNFjKJdx6wWy1J4u9Db5siX5g76CYQ1hrLZdHxka3BTHg8WW3YBO9hd1MOyGodH K3wnxnZPDuaYyX/8bilw9nz3571tRzxCv6gRLQHUZQSpGzPu5p/rhEk8C7LaAUr9yshLhImy R76JNXm0KtJ/k6LWAxBsVoq0k/ld0dOVKg0bjgihIbg7rJoC1VnfvYVcv89I+zqpyVrWbN8A dvRf2MabDoNMEZL2E1wntpPK/Lysczea/z0EAn87C2WvKNDrXw4a0DPlLMRuHPUUtHfBH7KO Y9IwwQhYRjT+PbdVAMjfQT7VKS0jBPlxP8OQ6gv4X3ktE/SB+dId/4UfLW+MZhaTqt9RgaKt GGFN4GlhDjtiPpa0TVn7wVz+Z/pMYozHOrhANLJmUfevd4GF8qUBpffIFnQp8CorjsSr4e4b PO3Kt5KEG2tptU+c5M2XMwARAQABwsF8BBgBCAAmFiEEMi9lQh4ilrRR2t1FG1KddWc5yJgF AlrV/VICGwwFCQlmAYAACgkQG1KddWc5yJjbtw//SYgKlu72eSwlpsEE1LXSvq2QwWkkZAm0 ZXCLar2Nj5qSXGvDlX13WwV8c0zlkjqV0LxOHoUOeqSMRMndk+EqCFsxHcfW9ZEUSdX9N62Q VX7wNyL7h53xKLRwXVjoG4CibE0dp0r2nm+/fgMT8RMaK2eRkQsre44196DFrqtxmT4ky7x6 rQfCZc64IhSAxNJcXoJKUrIua+4ZuLul6ITuPI6tHVbOQdQfVdTwDRZwMJvaFJ/WQDwfmtWa U03gVGaMxsUzODHpxhMRndCWVvATQi4YmVJDhW8AJbA+gmhcJDjuv1IcdQpnSsZ9DD32tGFd iFl6QUM7vhUYEyLnr4ojybkx4rbQsJz2PT9h1Wp8S7lkMljPmRPjoiJq7gkVL+2NcmHF2iOY lCY7+OcHwUcTCAzKRmtMaFI4MZPp/m2HPbkb8x0Mg8wWeQ47ufaOzaAJqz+K6TtwqQlf9zVa igXz+SFvUMz88gAKZ3lRL+/+RsEKXVCb0s0RW02XTFaVGyLdTF8GI998q4QkOahf6SynbiHF wUoQcnetxR2SQF/O+x5DSkBSvt5L54wpxiiJaJC1pwjWXAEIbtxMB/FHHr2GoMsPBXfM4UyV bQvZx6hHx8CBPKbhoFWPdhwh9Y0OQXJ1vBQppotDoiSwCymSAc0MoMcyWh5PTxu/41c48fBp 41o=
-
In-reply-to:
<910BE73C-BB8B-436A-848E-E31D6D2295F8@rokeby.ie>
-
Openpgp:
preference=signencrypt
-
User-agent:
Mozilla/5.0 (Windows NT 10.0; WOW64; rv:52.0) Gecko/20100101 Thunderbird/52.7.0
Go ahead and merge the changes. I didn't see anything that was a show
stopper. It was just some housekeeping on my part before tagging rc2
that Simon found some minor issues. Sorry I didn't reply sooner but I'm
in the process of migrating to my new work computer. Hopefully by the
end of the day, things will be back to normal.
Thanks,
Wayne
On 4/17/2018 10:15 AM, Jeff Young wrote:
> Wayne’s change is in master, so it’s going to end up in 5.0.
>
> I’ll go ahead and merge your patches there too. Thanks!
>
>
>> On 17 Apr 2018, at 14:58, Simon Richter <Simon.Richter@xxxxxxxxxx> wrote:
>>
>> Hi,
>>
>> On 17.04.2018 15:54, Simon Richter wrote:
>>
>>> IMO, these are trivial and should also go into 5.0. The code as is
>>> compiles because gcc doesn't detect the error. MSVC encodes the type of
>>> data objects and the return types of functions in the mangled symbol, so
>>> mismatching declaration and definition cause undefined symbols there.
>>
>> Ah wait, no. These make no sense in 5.0, because they are fixes for
>> 81843c37, which isn't in 5.0.
>>
>> Simon
>>
>> _______________________________________________
>> Mailing list: https://launchpad.net/~kicad-developers
>> Post to : kicad-developers@xxxxxxxxxxxxxxxxxxx
>> Unsubscribe : https://launchpad.net/~kicad-developers
>> More help : https://help.launchpad.net/ListHelp
>
>
> _______________________________________________
> Mailing list: https://launchpad.net/~kicad-developers
> Post to : kicad-developers@xxxxxxxxxxxxxxxxxxx
> Unsubscribe : https://launchpad.net/~kicad-developers
> More help : https://help.launchpad.net/ListHelp
>
References
-
[PATCH 1/4] Make declaration/definition consistent
From: Simon Richter, 2018-04-17
-
[PATCH 4/4] Convert line endings in trace_helpers.{cpp, h} to Unix
From: Simon Richter, 2018-04-17
-
Re: [PATCH 4/4] Convert line endings in trace_helpers.{cpp, h} to Unix
From: Jeff Young, 2018-04-17
-
Re: [PATCH 4/4] Convert line endings in trace_helpers.{cpp, h} to Unix
From: Simon Richter, 2018-04-17
-
Re: [PATCH 4/4] Convert line endings in trace_helpers.{cpp, h} to Unix
From: Simon Richter, 2018-04-17
-
Re: [PATCH 4/4] Convert line endings in trace_helpers.{cpp, h} to Unix
From: Jeff Young, 2018-04-17