Blackbox does not compile, diff in here
Kubilay Kocak
koobs at FreeBSD.org
Thu Aug 9 05:22:20 UTC 2018
On 9/08/2018 3:14 pm, Kubilay Kocak wrote:
> On 9/08/2018 3:05 pm, Erich Dollansky wrote:
>> Hi,
>>
>> On Thu, 9 Aug 2018 14:34:17 +1000
>> Kubilay Kocak <koobs at FreeBSD.org> wrote:
>>
>>> On 9/08/2018 12:29 pm, Erich Dollansky wrote:
>>>>
>>>> I do not know if somebody spotted this already. Some casting was
>>>> done to the wrong types.
>>>
>>> This looks like
>>> https://bugs.freebsd.org/bugzilla/show_bug.cgi?id=226708
>>>
>> yes, it is the same. But why could I compile blackbox without a problem
>> up to one or two months ago? Did the compiler to comprehend the problem
>> before?
>
> The bug report doesn't contain system information (freebsd version/arch,
> date) so it's not immediately at this stage, but I have reproduced the
> error on 12-CURRENT (r336056) and am QA'ing other versions/archs as we
> speak.
Looks like it's Clang 6, so I'm guessing all reporters are running
12-CURRENT post clang 6 import.
See Also:
https://clang.debian.net/status.php?version=6.0&key=CXX11_NARROWING
> What's the uname -a of the system you are seeing this issue on? Could
> you add this information in the bug?
>
>
>> Erich
>>
>>
>>> The first and third chunks below appear to be
>>> noise/spurious/unnecessary though.
>>>
>>>> Erich
>>>>
>>>> PS I am not experienced in creating diffs for ports. This one was
>>>> done in the working directory's lib entry:
>>>>
>>>>
>>>> --- EWMH.cc.180809 2005-01-24
>>>> 15:50:56.000000000 +0800 +++ EWMH.cc 2018-08-09
>>>> 10:10:05.630203000 +0800 @@ -204,7 +204,7 @@
>>>> bool bt::EWMH::readNumberOfDesktops(Window target,
>>>> unsigned int* number) const {
>>>> unsigned char* data = 0;
>>>> - if (getProperty(target, XA_CARDINAL, net_number_of_desktops,
>>>> &data)) {
>>>> + if (getProperty (target, XA_CARDINAL, net_number_of_desktops,
>>>> &data)) { *number =
>>>> static_cast<unsigned int>(*(reinterpret_cast<unsigned long
>>>> *>(data)));
>>>> @@ -246,8 +246,11 @@
>>>>
>>>>
>>>> void bt::EWMH::setDesktopViewport(Window target, int x, int y)
>>>> const {
>>>> + //
>>>> + // 09.08.18 ed: the following statement was modified.
>>>> + //
>>>> const unsigned long viewport[] =
>>>> - { static_cast<long>(x), static_cast<long>(y) };
>>>> + { static_cast<unsigned long>(x), static_cast<unsigned
>>>> long>(y) }; setProperty(target, XA_CARDINAL, net_desktop_viewport,
>>>> reinterpret_cast<const unsigned char *>(viewport),
>>>> 2); }
>>>> @@ -644,8 +647,10 @@
>>>> }
>>>>
>>>>
>>>> -bool bt::EWMH::getProperty(Window target, Atom type, Atom property,
>>>> - unsigned char** data) const {
>>>> +bool bt::EWMH::getProperty (Window target, Atom type, Atom
>>>> property,
>>>> + unsigned char ** data)
>>>> const { Atom atom_return;
>>>> int size;
>>>> unsigned long nitems, bytes_left;
>>>> _______________________________________________
>>>> freebsd-ports at freebsd.org mailing list
>>>> https://lists.freebsd.org/mailman/listinfo/freebsd-ports
>>>> To unsubscribe, send any mail to
>>>> "freebsd-ports-unsubscribe at freebsd.org"
>>>
>>
>
More information about the freebsd-ports
mailing list