kicad-developers team mailing list archive
-
kicad-developers team
-
Mailing list archive
-
Message #27449
Re: Eeschema one_field_edit dialog setselection patch
Hi,
On 31.01.2017 20:13, Chris Pavlina wrote:
> /home/cmp/git/kicad/eeschema/dialogs/dialog_edit_one_field.cpp: In member function ‘virtual bool DIALOG_EDIT_ONE_FIELD::TransferDataToWindow()’:
> /home/cmp/git/kicad/eeschema/dialogs/dialog_edit_one_field.cpp:174:42: warning: comparison between signed and unsigned integer expressions [-Wsign-compare]
> if ( m_text.find_first_of( '?' ) != -1 )
> ~~~~~~~~~~~~~~~~~~~~~~~~~~~~^~~~~
> wxString::find_first_of returns size_t, so that comparison will always
> be false.
Hmm, probably all instances of -1 should be wxString::npos instead?
Simon
Attachment:
signature.asc
Description: OpenPGP digital signature
References