Debug build broken on Linux

classic Classic list List threaded Threaded
3 messages Options
Reply | Threaded
Open this post in threaded view
|

Debug build broken on Linux

Stevethefiddle
void Meter::RestoreState(const State &state)
The cast from Meter* to int causes a fatal build error on Linux
(in the WxLogDebug line).

I don't understand casting from Meter* to int
What is that supposed to do?

Steve

------------------------------------------------------------------------------
Check out the vibrant tech community on one of the world's most
engaging tech sites, Slashdot.org! http://sdm.link/slashdot
_______________________________________________
audacity-devel mailing list
[hidden email]
https://lists.sourceforge.net/lists/listinfo/audacity-devel
Reply | Threaded
Open this post in threaded view
|

Re: Debug build broken on Linux

James Crook
Sorry.  Fixed in
https://github.com/audacity/audacity/commit/795cbe0c0ccbece944b10c75e75618de8c2b9574

That was a left over from some debugging where I was tracking down the
mysterious problem of meters not working.  The cast to int was a sloppy
way to see a pointer, on a 64 bit machine.  I should have used %p in the
print formatting.

On 4/16/2017 11:24 AM, Steve the Fiddle wrote:
> void Meter::RestoreState(const State &state)
> The cast from Meter* to int causes a fatal build error on Linux
> (in the WxLogDebug line).
>
> I don't understand casting from Meter* to int
> What is that supposed to do?
>
> Steve


------------------------------------------------------------------------------
Check out the vibrant tech community on one of the world's most
engaging tech sites, Slashdot.org! http://sdm.link/slashdot
_______________________________________________
audacity-devel mailing list
[hidden email]
https://lists.sourceforge.net/lists/listinfo/audacity-devel
Reply | Threaded
Open this post in threaded view
|

Re: Debug build broken on Linux

Stevethefiddle
On 16 April 2017 at 11:50, James Crook <[hidden email]> wrote:
> Sorry.  Fixed in
> https://github.com/audacity/audacity/commit/795cbe0c0ccbece944b10c75e75618de8c2b9574
>
> That was a left over from some debugging where I was tracking down the
> mysterious problem of meters not working.  The cast to int was a sloppy
> way to see a pointer, on a 64 bit machine.  I should have used %p in the
> print formatting.

OK, I see.
Debug build now works, thanks.

Steve

>
> On 4/16/2017 11:24 AM, Steve the Fiddle wrote:
>> void Meter::RestoreState(const State &state)
>> The cast from Meter* to int causes a fatal build error on Linux
>> (in the WxLogDebug line).
>>
>> I don't understand casting from Meter* to int
>> What is that supposed to do?
>>
>> Steve
>
>
> ------------------------------------------------------------------------------
> Check out the vibrant tech community on one of the world's most
> engaging tech sites, Slashdot.org! http://sdm.link/slashdot
> _______________________________________________
> audacity-devel mailing list
> [hidden email]
> https://lists.sourceforge.net/lists/listinfo/audacity-devel

------------------------------------------------------------------------------
Check out the vibrant tech community on one of the world's most
engaging tech sites, Slashdot.org! http://sdm.link/slashdot
_______________________________________________
audacity-devel mailing list
[hidden email]
https://lists.sourceforge.net/lists/listinfo/audacity-devel