[Bug default/25998] Support incomplete enum types

["dodji at redhat dot com" <sourceware-bugzilla@sourceware.org>]

https://sourceware.org/bugzilla/show_bug.cgi?id=25998

dodji at redhat dot com changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
   Last reconfirmed|                            |2020-05-19
             Status|UNCONFIRMED                 |NEW
     Ever confirmed|0                           |1

--- Comment #1 from dodji at redhat dot com ---
Hello,

Thank you for filling this enhancement request and for working on its
underlying feature.

I agree with you that the incomplete enum support is needed and I am glad that
you started working on it.  I want to help get it in, so I am looking at it.

As for the way to go, I tend to not be a fan of early optimization.  So really,
at first, I wouldn't spend to much time (and I'd rather not spent too much
review time) on trying to remove duplicated code etc.

I'd rather want to have the feature in and make it work solidly first.  Of
course, if the design is elegant enough to integrate with the code base in a
manner that avoids duplication, then fine.  But I don't think we should go out
of our way to try and squeeze all the juice out of avoiding duplication.

Incidentally, I have posted a comment on the WIP types patches at
https://sourceware.org/pipermail/libabigail/2020q2/002262.html.

I'll be looking at the C++ enum type specification again, as I don't have the
details in mind at the moment.

Thanks again!

-- 
You are receiving this mail because:
You are on the CC list for the bug.