qpid-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Alan Conway" <acon...@redhat.com>
Subject Re: Review Request: WIP: enable -fvisibility=hidden -fvisibility-inlines-hidden
Date Tue, 22 Mar 2011 15:10:48 GMT

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/520/
-----------------------------------------------------------

(Updated 2011-03-22 08:10:48.607304)


Review request for qpid and Andrew Stitcher.


Changes
-------

Initial stab at enabling symbol visibility, runs into warnings


Summary (updated)
-------

WIP: enable -fvisibility=hidden -fvisibility-inlines-hidden 

Failing with warnings: warning: lowering visibility of ... to match its type 
Google suggests warning is ok, but no apparent way to disable it. 
Try build with warnings off to see if the results are good.


Diffs
-----

  trunk/qpid/cpp/configure.ac 1084053 
  trunk/qpid/cpp/include/qmf/engine/QmfEngineImportExport.h 1084053 
  trunk/qpid/cpp/include/qpid/CommonImportExport.h 1084053 
  trunk/qpid/cpp/include/qpid/ImportExportDefs.h PRE-CREATION 
  trunk/qpid/cpp/include/qpid/agent/QmfAgentImportExport.h 1084053 
  trunk/qpid/cpp/include/qpid/client/ClientImportExport.h 1084053 
  trunk/qpid/cpp/include/qpid/console/ConsoleImportExport.h 1084053 
  trunk/qpid/cpp/include/qpid/messaging/ImportExport.h 1084053 
  trunk/qpid/cpp/src/Makefile.am 1084053 
  trunk/qpid/cpp/src/qpid/broker/BrokerImportExport.h 1084053 

Diff: https://reviews.apache.org/r/520/diff


Testing
-------


Thanks,

Alan


Mime
  • Unnamed multipart/alternative (inline, None, 0 bytes)
View raw message