This is the mail archive of the binutils@sourceware.org mailing list for the binutils project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

gold patch committed: Update comment


In last night's patch I forgot to remove a FIXME which no longer
applies.  Committed.

Ian


2008-07-24  Ian Lance Taylor  <iant@google.com>

	* dynobj.cc (Versions::Versions): Update comment.


Index: dynobj.cc
===================================================================
RCS file: /cvs/src/src/gold/dynobj.cc,v
retrieving revision 1.41
diff -u -r1.41 dynobj.cc
--- dynobj.cc	24 Jul 2008 07:26:27 -0000	1.41
+++ dynobj.cc	24 Jul 2008 21:20:33 -0000
@@ -1305,12 +1305,9 @@
   : defs_(), needs_(), version_table_(),
     is_finalized_(false), version_script_(version_script)
 {
-  // We always need a base version, so define that first. Nothing
+  // We always need a base version, so define that first.  Nothing
   // explicitly declares itself as part of base, so it doesn't need to
   // be in version_table_.
-  // FIXME: Should use soname here when creating a shared object. Is
-  // this fixme still valid? It looks like it's doing the right thing
-  // to me.
   if (parameters->options().shared())
     {
       const char* name = parameters->options().soname();

Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]