Fix bug in plugin section ordering infrastructure when layout is deferred.

Cary Coutant ccoutant@google.com
Thu Sep 29 18:26:00 GMT 2011


> All done. Patch attached.

In gold.cc...
+      for (Layout::Section_list::const_iterator p =
layout->section_list().begin();

This line is too long.

+      && !parameters->options().section_ordering_file ())

Sorry, I missed a space-before-paren in the last review.

In plugin.cc...
+  std::map<Section_id, unsigned int>* order_map
+    = layout->get_section_order_map ();

Another one.

This looks good to me with those fixes. (Still needs Ian's OK, though.)

-cary



More information about the Binutils mailing list