On Thu, Jan 29, 2015 at 7:26 PM, Michael Van Canneyt <mich...@freepascal.org
> wrote:
>
>
> On Thu, 29 Jan 2015, silvioprog wrote:
>
> Hello,
>>
>> Please see this patch in attached. Can I send it to bugtracker?
>>
>
> I don't see the point. Why do you need this ?
Yes, after a while I saw that my patch seems redundant. But please notice
this code below:
function TFCGIRequest.ProcessFCGIRecord(AFCGIRecord: PFCGI_Header): boolean;
...
else
begin
if not assigned(FCGIParams) then
FCGIParams := TStringList.Create;
GetNameValuePairsFromContentRecord(PFCGI_ContentRecord(AFCGIRecord),FCGIParams);
end;
It seems to offer the programmer to create an own cgiparams. Is this "if"
just to create this field when the request need to use it?
What do you think about to create the FCGIParams field in the create of the
TFCGIRequest class and remove the "if" from the ProcessFCGIRecord method?
I think that the ProcessFCGIRecord method is more triggered than the
constructor of the TFCGIRequest class.
--
Silvio Clécio
My public projects - github.com/silvioprog
_______________________________________________
fpc-pascal maillist - fpc-pascal@lists.freepascal.org
http://lists.freepascal.org/cgi-bin/mailman/listinfo/fpc-pascal
