← Back to team overview

kicad-developers team mailing list archive

Re: [PATCH] Component and module search displays in which library they live

 

Hi Wayne,

I rebuilt the project using the latest revision (3987) and everything is
fine on my side. Please have a look at the new patch file in which I also
fixed a few issues regarding the coding style, namely spaces around
parentheses and blank lines before if statements and comments.

Thanks,

Carl

On Mon, Mar 4, 2013 at 12:51 PM, Wayne Stambaugh <stambaughw@xxxxxxxxxxx>wrote:

> On 3/2/2013 9:52 AM, Carl Poirier wrote:
> > Hi all,
> >
> > I have made a patch for one of the TODOs in the "Common" part, which
> > reads as follows:
> >
> >     * Component and module search displays in which library the module
> >     or component lives.
> >
> >
> >  I just want to mention that it's my first ever contribution to
> > open-source software, and so if I did something wrong, please kindly let
> > me know and I will fix what needs to. :) I followed the steps in the
> > "HOW_TO_CONTRIBUTE.txt" file, thus I provide you with a .patch file.
> > BTW, I branched from revision 3962.
> >
> > Ok, on to the actual code.
> >
> > The most important change is that I transformed the member m_listBox of
> > EDA_LIST_DIALOG_BASE in a wxListCtrl. This allows us to have more than
> > one column in there, so that we can display the component names and
> > their libraries. The EDA_LIST_DIALOG_BASE class has been edited through
> > wxFormsBuilder, which is what I believe had been done before, so the
> > .cpp and .h have been generated accordingly.
> >
> > The EDA_LIST_DIALOG_BASE class is used throughout kicad, therefore I
> > made it so that we can still push items in a single column to it.
> > However, I changed the type of that item list to a
> > std::vector<wxArrayString>& so that it's 2D. Vector elements are
> > components, for which every wxString in the array is a distinct column.
> > I did the needed few conversions from wxArrayString& to
> > std::vector<wxArrayString>& here and there.
> >
> > Also, the sorting methods have been changed to work on a vector and a
> > wxListCtrl. I also activated the sorting when the user searches for a
> > component in eeschema. Finally, I corrected a few typos in comments when
> > I happened to see them.
> >
> > This is it, so please let me know how it works out!
> >
> >
> > Carl Poirier
> >
> >
> > P.S. I first send this email earlier this week but I did not join the
> > developer team beforehand, so I believe it has been blocked. Sorry if
> > this is not the case.
> >
>
> Carl,
>
> I looked at this patch and I think it has potential to significantly
> improve the component/footprint list dialog.  However, the patch fails
> to apply against the testing branch version r3983.  The dialog box has
> changed since you created your patch.  Please update your source, fix
> any conflicts, and resubmit your patch.  However, before you resubmit
> you patch, please read the KiCad coding policy which can be found here:
>
>
> http://bazaar.launchpad.net/~kicad-testing-committers/kicad/testing/view/head:/Documentation/coding_style_policy.pdf
>
> and format your code accordingly.  Thank you for your interest in
> improving KiCad.
>
> Wayne
>
>
> _______________________________________________
> Mailing list: https://launchpad.net/~kicad-developers
> Post to     : kicad-developers@xxxxxxxxxxxxxxxxxxx
> Unsubscribe : https://launchpad.net/~kicad-developers
> More help   : https://help.launchpad.net/ListHelp
>

Attachment: cmp_and_mod_display_library_3.patch
Description: Binary data


Follow ups

References