-----------------------------------------------------------
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

Reply via email to