<div dir="ltr"><div dir="ltr"><br></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Tue, May 14, 2019 at 12:23 AM Vijay Kiran Kamuju <<a href="mailto:infyquest@gmail.com">infyquest@gmail.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">On Mon, May 13, 2019 at 1:23 PM Alexandre Julliard <<a href="mailto:julliard@winehq.org" target="_blank">julliard@winehq.org</a>> wrote:<br>
><br>
> Vijay Kiran Kamuju <<a href="mailto:infyquest@gmail.com" target="_blank">infyquest@gmail.com</a>> writes:<br>
><br>
> > @@ -32,6 +32,7 @@ C_SRCS = \<br>
> >       shelldispatch.c \<br>
> >       shellitem.c \<br>
> >       shelllink.c \<br>
> > +     shellnew.c \<br>
><br>
> Please use a better name, or preferably put this into some existing file<br>
> like shlmenu.c.<br>
This cannot be put into existing files like shlmenu.c or<br>
shlview_cmenu.c as it we need separate implementations of<br>
IContextMenu3 and IShellExtInit.<br>
Please let me know if the filename like shlview_nmenu.c or<br>
shlview_newmenu.c is ok or not.<br></blockquote><div><br></div><div>Maybe it's possible to unify some of that. Shell32 has enough duplication already. If not, nothing prevents from having two objects implementing same interface in the same file.</div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
---<br>
Vijay<br>
><br>
> --<br>
> Alexandre Julliard<br>
> <a href="mailto:julliard@winehq.org" target="_blank">julliard@winehq.org</a><br>
<br>
<br>
</blockquote></div></div>