On Thu, 29 Jan 2015, silvioprog wrote:
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?
Yes. I usually prefer late allocation on an as-needed basis.
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.
ProcessFCGIRecord is usually called only once or maybe twice if there are
really a lot of parameters.
So I seriously doubt it will make a noticeable difference in speed :)
Michael.
_______________________________________________
fpc-pascal maillist - fpc-pascal@lists.freepascal.org
http://lists.freepascal.org/cgi-bin/mailman/listinfo/fpc-pascal