New feature: serverside syntax highlighting

[ra4king]

What is the font used in that image? I really like it! I’m getting tired of Courier New…

[sproingie]

Looks like Terminal to me.

I’m a big fan of DejaVu Sans Mono. Use it for emacs and all my terminals.

[Evil-Devil]

@Riven

Derailing my own thread…

@Evil-Devil

      // fill fourCC with whitespaces if length is shorter than 4
      if (fourCC.length() < 4) {
         for (int i=0; i<(4-fourCC.length()); i++) { // <--------------- this is a bug!
            /* fourCC = */ fourCC.concat(" "); // <--------------- this is a bug!
         }
      }

1. fourCC will not be modified (strings are immutable), you’re basically doing nothing.
2. if they were, it still wouldn’t work, so adding the assignment wouldn’t help (much)
   You can see that if you pass an empty string, it will stop once it reaches a length of 2, instead of 4.

Trivial fix:

      // fill fourCC with whitespaces if length is shorter than 4
      while (fourCC.length() < 4) {
         fourCC = fourCC.concat(" ");
      }

Besides that, shoving arbitrary bytes in a byte[4] into a String(…) is extremely dangerous, as you get a conversation to a char[] using the default system dependent charset which is not at all guaranteed to result into a char[4], and even when it’s a char[4] within the String, that doesn’t mean each char holds the value each original byte had.

Spooky nicely colored code

Erm, well…i only knew the code worked like intented back in 2005/2006 and using the systems default charset was out of scope as the FourCC was part of the fileheader. Which was IIRC lower endian …though concat returns a new string object

[pitbuller]

Blue background brings up good memories from QBasic.

[Riven]

@Evil-Devil

Erm, well…i only knew the code worked like intented back in 2005/2006

It only works correctly if the argument String length is already 4

[appel]

Can you color code and change the font of what I say so it matches my mood and feelings?

[aazimon]

The code color works much better. Thanks for the effort.

[Riven]

There are still some bugs in it, but well, that’s mostly caused by SMF html-encoding a lot of characters which messes up the tokenizer.

What I forgot to mention though that since a few weeks, you can use the ‘@@’ prefix to highlight lines:

for(int i=-13; i<13; i++) {
@@   if(i==1) { // ORLY!
      System.out.println("One is my lucky number!");
   }
}

[nsigma]

@Riven

What I forgot to mention though that since a few weeks, you can use the ‘@@’ prefix to highlight lines:

For a moment I was like, wtf is he on about? Then I squinted and tilted my laptop screen … and I can just about make it out. Maybe my screen’s set up badly, but a little bit darker, please!

[Riven]

Hm, it’s very clear on both my monitors.

If your monitor displays “#ffc” as “#fff” you’re really going to experience the inter tubes differently.

I adjusted it to “#ffa”. Better now?

[nsigma]

@Riven

Hm, it’s very clear on both my monitors.

Better now?

hmm … feels like being at the optician! Actually, doesn’t look much if any different. If no-one else pipes up, leave it and I’ll furkle with my screen instead.

[Riven]

Having “#ffa” look like “#fff”… That must be a really bad configured monitor.

[nsigma]

@Riven

Having “#ffa” look like “#fff”… That must be a really bad configured monitor.

Actually, it does look better now. Did you change it before or after you edited your post? Maybe some caching going on - I did try and force a refresh.

You might be right about my laptop screen, though. It’s a good job that the web platform stuff I do is backend and not client facing. Otherwise we might have lurid mid-90’s throwbacks and I wouldn’t even notice. anyone??? ;D

[Riven]

Finally fixed the last ‘known bug’…

The ’ character had 3 variants in the database: ', ’ and ’ :cranky:

[Riven]

And the line numbers are back, somehow forgot about them in the rewrite :persecutioncomplex:

[Mike]

Yuck, line numbers! If I want to copy something (using without asking isn’t stealing, the pirate bay taught me that) I now have to manually remove the line numbers? Maybe make text select ignore them somehow or add a button to turn them off so we can copy code out of the code block?

Ignore me, I forgot that Riven was awesome

Mike

[Riven]

@Mickelukas

Yuck, line numbers! If I want to copy something (using without asking isn’t stealing, the pirate bay taught me that) I now have to manually remove the line numbers? Maybe make text select ignore them somehow or add a button to turn them off so we can copy code out of the code block?

Did you try it? :persecutioncomplex:

[Mike]

I did, promise! Tested again though without moving the mouse cursor over the line number :point: I edited the above post as you posted a reply

[Riven]

I hate it too when the line numbers are part of the code. I look at you, pastebin…

[ra4king]

That’s why you copy the code from the text box BELOW the highlighted syntax in pastebin