Sign in Join

Live chat for Free Pascal, Lazarus, Delphi and every Pascal programmer.

Mailing lists / lazarus / 2003-08

FindInFiles Changes(findinfilesdlg.pas, ideprocs.pp,uniteditor.pp, dirsel.pas)

7 messages · 3 people ·

Reply by email Subscribe to this list Your subscription

Read-only archive. Use j and k to move between messages.

Mattias Gaertner
to Bob Wingard: Includes all changes from this morning, Including the patch that would not load.…
Show quoted text 16 lines
On Sun, 3 Aug 2003 10:50:08 +0200 Mattias Gaertner wrote: > On Sat, 02 Aug 2003 16:02:30 -0400 > Bob Wingard wrote: > > > Summary, > > Includes all changes from this morning, Including the patch that > > > > would not load. > > Thx. Applied. > > Mattias Almost perfect, but now the FindFirst is missing in recursive directory search in FindMatchingTextFiles.

Where exactly?

Mattias

Reply by email

Bob Wingard
to Bob Wingard: Includes all changes from this morning, Including the patch that would not load.…

Vincent Snijders wrote:

<snip>

Show quoted text 11 lines
My apolegies, I got some conflicts on the ideproc.pp file, because of editing it myself. When resolving the conflict I must have cut out the line with the find first. I got a completely new version from CVS, which looks good. Sorry for the misunderstanding. Vincent.

<snip>

No problem, I will take all the help I can get. :-)

Bob

Reply by email

Bob Wingard

Summary,
Includes all changes from this morning, Including the patch that would not load.

findinfilesdlg.pas
Modified the Directory options group box to be disabled if search in directories option is not selected.
Add new procedure WhereRadioGroupClick.
Implemented recomendations from Vincent.

ideprocs.pp
Add SynRegExpr to Uses in the implementaton section. Mattias, I am sure that you will change this :-)
Add Regular Expression search to FindInFiles Procedure. Not throughly tested! Finds ^procedure just fine.

uniteditor.pp
Added Try-Except block around the search to catch ERegExpr error thrown by TRegExpr object.

dirsel.pas
Changes recommended by Vincent this morning (my time).

Bob

-------------- next part --------------
A non-text attachment was scrubbed...
Name: zip00022.zip
Type: application/octet-stream
Size: 10617 bytes
Desc: "findinfiles.zip"
Url : http://localhost/pipermail/lazarus/attachments/20030802/c9646422/zip00022.obj

Reply by email

Vincent Snijders
to Bob Wingard
Show quoted text 11 lines
On Sat, 02 Aug 2003 16:02:30 -0400 Bob Wingard wrote: > Summary, > Includes all changes from this morning, Including the patch that > > would not load. Thx. Applied. Mattias

Almost perfect, but now the FindFirst is missing in recursive directory search in FindMatchingTextFiles.

Vincent.

Reply by email

Mattias Gaertner
to Bob Wingard: Includes all changes from this morning, Including the patch that would not load.…
Summary, Includes all changes from this morning, Including the patch that would not load.

Thx. Applied.

Mattias

Show quoted text 22 lines
findinfilesdlg.pas Modified the Directory options group box to be disabled if search in directories option is not selected. Add new procedure WhereRadioGroupClick. Implemented recomendations from Vincent. ideprocs.pp Add SynRegExpr to Uses in the implementaton section. Mattias, I am sure that you will change this :-) Add Regular Expression search to FindInFiles Procedure. Not throughly tested! Finds ^procedure just fine. uniteditor.pp Added Try-Except block around the search to catch ERegExpr error thrown by TRegExpr object. dirsel.pas Changes recommended by Vincent this morning (my time). Bob

Reply by email

Vincent Snijders
to Bob Wingard: Includes all changes from this morning, Including the patch that would not load.…
Show quoted text 15 lines
Vincent Snijders wrote: <snip> >Almost perfect, but now the FindFirst is missing in recursive >directory search in FindMatchingTextFiles. > >Vincent. > > <snip> What line, do not see it. Bob

My apolegies, I got some conflicts on the ideproc.pp file, because of editing it myself. When resolving the conflict I must have cut out the line with the find first. I got a completely new version from CVS, which looks good.

Sorry for the misunderstanding.

Vincent.

Reply by email