Re: [PATCH 0/1] MR2773: include: Add xamlom.idl file.

[Zebediah Figura (＠zfigura)]

Zebediah Figura (@zfigura) commented about include/xamlom.idl:
> -
> - but WITHOUT ANY WARRANTY; without even the implied warranty of
> -
> - MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the GNU
> -
> - Lesser General Public License for more details.
> -
> -
> -
> - You should have received a copy of the GNU Lesser General Public
> -
> - License along with this library; if not, write to the Free Software
> -
> - Foundation, Inc., 51 Franklin St, Fifth Floor, Boston, MA 02110-1301, USA
> - */
> -
>
> +import "oaidl.idl";
> +import "ocidl.idl";
> +import "inspectable.idl";
> +import "dxgi1_2.idl";
> -
>
> +cpp_quote("#define E_NOTFOUND HRESULT_FROM_WIN32(ERROR_NOT_FOUND)")
> +cpp_quote("#define E_UNKNOWNTYPE MAKE_HRESULT(SEVERITY_ERROR, FACILITY_XAML, 40L)")

Can we define this as the expanded value instead? It makes it easier to search for.