Opened 4 years ago

Closed 4 years ago

#52564 closed defect (fixed)

podofo @0.9.4 +universal: non-const lvalue reference to type 'pdf_long' (aka 'int') cannot bind to a value of unrelated type 'long'

Reported by: ryandesign (Ryan Schmidt) Owned by: dbevans (David B. Evans)
Priority: Normal Milestone:
Component: ports Version: 2.3.4
Keywords: Cc:
Port: podofo

Description (last modified by ryandesign (Ryan Schmidt))

test/unit/EncryptTest.cpp:211:76: error: non-const lvalue reference to type 'pdf_long' (aka 'int') cannot bind to a value of unrelated type 'long'
        pEncrypt->Decrypt( pEncryptedBuffer, nOutputLen, pDecryptedBuffer, m_lLen );
                                                                           ^~~~~~
src/base/PdfEncrypt.h:371:59: note: passing argument to parameter 'outLen' here
                         unsigned char* outStr, pdf_long &outLen) const = 0;
                                                          ^
1 error generated.

Attachments (2)

main.log.bz2 (8.9 KB) - added by ryandesign (Ryan Schmidt) 4 years ago.
main.log.2.bz2 (8.8 KB) - added by ryandesign (Ryan Schmidt) 4 years ago.

Download all attachments as: .zip

Change History (7)

Changed 4 years ago by ryandesign (Ryan Schmidt)

Attachment: main.log.bz2 added

comment:1 Changed 4 years ago by ryandesign (Ryan Schmidt)

Description: modified (diff)

Builds fine when not using the universal variant.

comment:2 Changed 4 years ago by dbevans (David B. Evans)

Status: newassigned

Bummer, I was really feeling good about finally fixing the "can't build with old installed copy" problem. But I'll take a look. My experience is that cmake and +universal don't always work well together. Too much going on silently (and often undocumented) behind the scenes with cmake. Thanks for catching this.

comment:3 in reply to:  2 ; Changed 4 years ago by ryandesign (Ryan Schmidt)

Replying to devans@…:

Bummer, I was really feeling good about finally fixing the "can't build with old installed copy" problem.

That's not it: this happens even if podofo is not already installed.

My experience is that cmake and +universal don't always work well together.

This particular problem isn't related to universal; it's related to building 32-bit. podofo @0.9.4 doesn't build single-arch i386 either; log attached.

Changed 4 years ago by ryandesign (Ryan Schmidt)

Attachment: main.log.2.bz2 added

comment:4 in reply to:  3 Changed 4 years ago by dbevans (David B. Evans)

Replying to ryandesign@…:

Replying to devans@…: This particular problem isn't related to universal; it's related to building 32-bit. podofo @0.9.4 doesn't build single-arch i386 either; log attached.

I was just coming around to that way of thinking myself. Thanks for the logfile. There may be hope yet.

comment:5 Changed 4 years ago by dbevans (David B. Evans)

Resolution: fixed
Status: assignedclosed

Fix committed in r153780.

Note: See TracTickets for help on using tickets.