Re: Google backend
- From: Joe Shaw <joe ximian com>
- To: Don Smith <donsmith77 hotpop com>
- Cc: "dashboard-hackers gnome org" <dashboard-hackers gnome org>
- Subject: Re: Google backend
- Date: Mon, 16 Feb 2004 19:15:52 -0500
My comments, mostly stylistic:
> + string google_api_key="INSERT DEVTAG HERE";
Use spaces around the assignment, ie:
string google_api_key = "INSERT DEVTAG HERE";
> + if (google_api_key == "INSERT DEVTAG HERE")
> return false;
Probably want to print something out to the Console so people notice
this and fix it. We'll need to do something about this at some point...
not sure what.
> + Match GoogleMatch = new Match("WebLink", clue);
> + GoogleMatch ["title"] = gsr.resultElements[i].title;
> + GoogleMatch ["url"] = gsr.resultElements[i].URL;
> + GoogleMatch ["icon"] = "internal:bookmark.png";
> + results.Add(GoogleMatch);
Spaces are needed around the parentheses in method calls, and the square
brackets in indexers, ie:
Function (args);
and
Array [index];
Joe
[
Date Prev][
Date Next] [
Thread Prev][
Thread Next]
[
Thread Index]
[
Date Index]
[
Author Index]