Sure, I'l make this change. But I'm curious about the motivation. From the automake handbook:

"However when the header actually belongs to a single convenience library or program, we recommend listing it in the program’s or library’s _SOURCES variable (see Program Sources) instead of in noinst_HEADERS."

To @liewegas 's comment, the headers will still show up in the tarball after make dist.