Funny Comments in NI VI code

[bbean]

I found this commment in the following NI VI - LabVIEW 8.0\vi.lib\_oldvers\_oldvers.llb\compatReadText.vi:

"this may need work, see note below"

Makes you wonder.

Has anyone found other interesting comments left accidentally in NI's or your own code?

[PaulG.]

Uh, umm, he he ... I've made similar comments myself.

Don't be too critical ... some of us are STILL cleaning up code written by FORMER employees who knew just enough LV to be dangerous. A sense of humor goes a long way in these situations.

[Aristos Queue]

Uh, umm, he he ... I've made similar comments myself.

Don't be too critical ... some of us are STILL cleaning up code written by FORMER employees who knew just enough LV to be dangerous. A sense of humor goes a long way in these situations.

Now there's a signature... "I know enough LabVIEW to be dangerous." Maybe Michael could use that for the new LAVA users. Instead of "2 more posts to go" it could be "Enough LAVA to be dangerous."

[bbean]

That comment is not as bad as the ones ppl found in windows source code :

http://www.kuro5hin.org/story/2004/2/15/71552/7795

* !!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!

* !!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!

* !!!!!!!IF YOU CHANGE TABS TO SPACES, YOU WILL BE KILLED!!!!!!!

* !!!!!!!!!!!!!!DOING SO F*&^s THE BUILD PROCESS!!!!!!!!!!!!!!!!

* !!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!

* !!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!

[Mike Ashe]

I've seen much worse, in government LabVIEW code, but I can't repeat it here ...

On the NI comments, I'm actually surprised that we haven't seen more of this over the years. Lots of people I know put in temporary comments, and on a large project and deadlines they often do not get cleaned up.

My current client (Lockheed) uses color coding on temporary comments, so I have been thinking of making a scripting tool to go searching through a VI hierarchy for all text comments of a certain color(s).

[bsvingen]

I saw a strange help file some weeks ago. Actually several of these variant VIs have this help file text even though they are completely different.

[Yair]

I saw a strange help file some weeks ago. Actually several of these variant VIs have this help file text even though they are completely different.

Which presumably means that they were all copied from an existing VI and someone forgot to change their documentation (what, no checklist at NI?).

[Aristos Queue]

Which presumably means that they were all copied from an existing VI and someone forgot to change their documentation (what, no checklist at NI?).

If it isn't in the palettes, it doesn't always get checked. Only real zealots go digging into all the subVIs in vi.lib. Not that I'm referring to anyone on this forum or anything.

[Mike Ashe]

If it isn't in the palettes, it doesn't always get checked. Only real zealots go digging into all the subVIs in vi.lib. Not that I'm referring to anyone on this forum or anything.

We're not zealots, we're spelunkers and archeologists ...

[bbean]

We're not zealots, we're spelunkers and archeologists ...

OK Just found a new tool - Google Code Search. I did a search on labview and found this in Perl code:

"

$self->tone(@$_) if ($_->[0] > 0);

usleep($_->[1]*15000);

}

# Now how many square meters of LabView code

# would you need to do this?

}

"

http://www.google.com/codesearch?q=+labvie...ent/SR780.pm#a0

Sometimes I feel like that.

[Aurora]

"

# Now how many square meters of LabView code

# would you need to do this?

"

Hey that's what those funny little CIN things are for - them and their cousins. Other than that I like to see the bigger picture rather than deciphering some crypric shorthand

[Jacemdom]

OK Just found a new tool - Google Code Search. I did a search on labview and found this in Perl code:

"

$self->tone(@$_) if ($_->[0] > 0);

usleep($_->[1]*15000);

}

# Now how many square meters of LabView code

# would you need to do this?

}

"

http://www.google.com/codesearch?q=+labvie...ent/SR780.pm#a0

Sometimes I feel like that.

Could someone translate this in english?

[Darren]

Actually, I'd prefer if somebody translated it into LabVIEW...I could probably understand a square meter of LabVIEW code better than that text-based nonsense.

-D

[jaegen]

This brings up an interesting point - how is Google searching through all of the open source LabVIEW stuff?

Jaegen

[jpdrolet]

Perl really looks like an explosion in an ASCII factory... (slashdot quote)

[i2dx]

Perl really looks like an explosion in an ASCII factory... (slashdot quote)

ROFL

yea, thats true - but I really like Perl and use it often.

Last example?

3 days ago I got a Phishig email and - WHOW - the website was still active.

So I wrote a Perl Script which is until now filling the Phisher's database with randomly generated PINs and TANs - I'd really like to see the face of "da BoYz"*, if they realize, their "really successfull draft of phish" is not more than random numbers in the correct format ...

conculusion: Perl is fun

*BTW: in the Perl Cookbook there is an example, which manipulates a text string in a WaY tHaT eAcH 2nD ChArAcTeR is set to uppercase. This is extremely helpfull, if you have to communicate with "WaReZ DuDeZ"

[Dan Bookwalter]

Perl really looks like an explosion in an ASCII factory... (slashdot quote)

Nothing against PERL , but , that quote is hilarious !!!!!

Dan

[Adam Kemp]

sub play_song {	my $self=shift;	for (([4,20],[5,20],[6,20],[7,20],[8,40],[ 8,40],		  [9,20],[9,20],[9,20],[9,20],[8,40],[-1,20],		  [9,20],[9,20],[9,20],[9,20],[8,40],[-1,20],		  [7,20],[7,20],[7,20],[7,20],[6,40],[ 6,40],		  [5,20],[5,20],[5,20],[5,20],[4,40])) {		$self->tone(@$_) if ($_->[0] > 0);		usleep($_->[1]*15000);	}	# Now how many square meters of LabView code	# would you need to do this?}

I see nothing in that code that would be complicated to implement in LabVIEW...

It says "for each of these sets of 2 values, pass the set to the tone function if the first element in the set is greater than 0, then sleep for a bit". Or, put more simply, each bracketed pair of numbers is a tone and a duration. It plays each tone for the given duration, and -1 means silence.

You would implement it in LabVIEW using an array of clusters, a for loop with auto-indexing, and a case structure. Pretty simple.

That guy must know just enough LabVIEW to be dangerous.

[rpursley]

On a related note. I get this error description in the lvfailurelog whenever I get a BadLinkerObjs error in LV 8.20 (which happens more often than I would wish).

.\editor\BadLinkerObjs.cpp(195) : DWarn: [LinkIdentity "Laser.lvclass:Laser Selection.ctl" [ My Computer] is NOT a bad subObj! Why are we propagating the myth that it is bad?

$Id: //labview/branches/Europa/dev/source/editor/BadLinkerObjs.cpp#49 $

[jpdrolet]

Look what I found looking into the file lvapp.rsc (LabVIEW 7):

[crelf]

Look what I found looking into the file lvapp.rsc (LabVIEW 7):

:thumbup: Now that's cool!

[Justin Goeres]

Look what I found looking into the file lvapp.rsc (LabVIEW 7):

<snip>

I wonder if it is needed for something actually useful...

It might be an image bitmap. Anybody care to convert those ASCII values to an 8-bit picture control and see what colors we get?

[crelf]

It might be an image bitmap. Anybody care to convert those ASCII values to an 8-bit picture control and see what colors we get?

:laugh: I don't think you'll see much - it'll be recognisable, but the ASCII equivalents are way up there in the basic page (all between F0 and FF if I'm not mistaken?)