rodrigc added a comment. Sorry for the late response. Please address @dim 's comment, if you have not addressed it already.
Also, removing the type-alias punned warnings is trickier than the other types of fixes that we have done to clean up the code. For this change, I would feel more comfortable if we actually tested ypbind with this code change. I know, this is a headcahe. :( But still, we have to do the right thing. Regarding importing ypbind from OpenBSD. I think this is a good idea, because it looks like they more actively maintain the code. However, this code in FreeBSD is *old*....almost 17 years old. Even if we import from OpenBSD, I don't know if there are changes to functionality, command-line options, etc. that will be introduced. This will require testing also. It depends on how much interest you have in ypbind. :) I don't have a lot of interest, and was just looking at doing minimal code changes to fix compiler warnings. Let me know what approach you want to take. I will support you. REPOSITORY rS FreeBSD src repository REVISION DETAIL https://reviews.freebsd.org/D2690 EMAIL PREFERENCES https://reviews.freebsd.org/settings/panel/emailpreferences/ To: rodrigc, bapt, emaste, dim, araujo Cc: imp, freebsd-toolchain-list _______________________________________________ freebsd-toolchain@freebsd.org mailing list http://lists.freebsd.org/mailman/listinfo/freebsd-toolchain To unsubscribe, send any mail to "freebsd-toolchain-unsubscr...@freebsd.org"
