[prev in list] [next in list] [prev in thread] [next in thread] 

List:       kwrite-devel
Subject:    Re: Review Request 128905: Add syntax highlighting support for Praat
From:       José Joaquín Atria <jjatria () gmail ! com>
Date:       2016-09-18 23:27:34
Message-ID: CA+CQHC+48c++K6yEqk6byE25O=GaZHFBqsJNWcf8=MJ0C5f6CQ () mail ! gmail ! com
[Download RAW message or body]

Sure. We could use the file I use for testing the Rouge lexer, which I also
wrote.

You can see it here:
https://raw.githubusercontent.com/jneen/rouge/master/spec/visual/samples/pr=
aat

That's a fairly complete visual test set. If you were thinking of a smaller
representative one, it could e the language demo, also from Rouge:
https://raw.githubusercontent.com/jneen/rouge/master/lib/rouge/demos/praat

How should I add it? And how do those unit tests work? Should I also add
some reference that says how they should be highlighted?

Best,


Jos=C3=A9 Joaqu=C3=ADn Atria

www.pinguinorodriguez.cl

On Sun, Sep 18, 2016 at 1:40 PM, Volker Krause <vkrause@kde.org> wrote:

> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/128905/
>
> Could you provide a Praat file we can use for automated regression testin=
g? Thanks!
>
>
> - Volker Krause
>
> On September 14th, 2016, 11:27 a.m. UTC, Jos=C3=A9 Joaqu=C3=ADn Atria wro=
te:
> Review request for Kate.
> By Jos=C3=A9 Joaqu=C3=ADn Atria.
>
> *Updated Sept. 14, 2016, 11:27 a.m.*
> *Repository: * ktexteditor
> Description
>
> Praat (praat.org) is an acoustic analysis software extensively used in Ph=
onetics research. This commit adds syntax highlighting for its internal scr=
ipting language.
>
> Testing
>
> $ ./validatehl.sh praat.xml
> praat.xml validates
>
> Diffs
>
>    - src/syntax/data/praat.xml (PRE-CREATION)
>
> View Diff <https://git.reviewboard.kde.org/r/128905/diff/>
>

[Attachment #3 (text/html)]

<div dir="ltr"><div><div>Sure. We could use the file I use for testing the Rouge \
lexer, which I also wrote.<br><br></div>You can see it here: <a \
href="https://raw.githubusercontent.com/jneen/rouge/master/spec/visual/samples/praat"> \
https://raw.githubusercontent.com/jneen/rouge/master/spec/visual/samples/praat</a><br><br></div><div>That&#39;s \
a fairly complete visual test set. If you were thinking of a smaller representative \
one, it could e the language demo, also from Rouge: <a \
href="https://raw.githubusercontent.com/jneen/rouge/master/lib/rouge/demos/praat">http \
s://raw.githubusercontent.com/jneen/rouge/master/lib/rouge/demos/praat</a><br></div><div><br></div>How \
should I add it? And how do those unit tests work? Should I also add some reference \
that says how they should be \
highlighted?<br><div><br></div><div>Best,<br><br></div></div><div \
class="gmail_extra"><br clear="all"><div><div class="gmail_signature" \
data-smartmail="gmail_signature">José Joaquín Atria<br><br><a \
href="http://www.pinguinorodriguez.cl" \
target="_blank">www.pinguinorodriguez.cl</a></div></div> <br><div \
class="gmail_quote">On Sun, Sep 18, 2016 at 1:40 PM, Volker Krause <span \
dir="ltr">&lt;<a href="mailto:vkrause@kde.org" \
target="_blank">vkrause@kde.org</a>&gt;</span> wrote:<br><blockquote \
class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc \
solid;padding-left:1ex">



 <div>
  <div style="font-family:Verdana,Arial,Helvetica,Sans-Serif"><span class="">
   <table style="border:1px #c9c399 solid;border-radius:6px" cellpadding="12" \
bgcolor="#f9f3c9" width="100%">  <tbody><tr>
     <td>
      This is an automatically generated e-mail. To reply, visit:
      <a href="https://git.reviewboard.kde.org/r/128905/" \
target="_blank">https://git.reviewboard.kde.<wbr>org/r/128905/</a>  </td>
    </tr>
   </tbody></table>
   <br>





 </span><pre style="white-space:pre-wrap;white-space:-moz-pre-wrap;white-space:-pre-wrap;white-space:-o-pre-wrap;word-wrap:break-word"><p \
style="padding:0;margin:0;line-height:inherit;white-space:inherit">Could you provide \
a Praat file we can use for automated regression testing? Thanks!</p></pre>  <br>









<p>- Volker Krause</p><span class="">


<br>
<p>On September 14th, 2016, 11:27 a.m. UTC, José Joaquín Atria wrote:</p>








</span><table style="border:1px #888a85 solid;border-radius:6px" cellpadding="12" \
cellspacing="0" bgcolor="#fefadf" width="100%">  <tbody><tr>
  <td><span class="">

<div>Review request for Kate.</div>
<div>By José Joaquín Atria.</div>


</span><p style="color:grey"><i>Updated Sept. 14, 2016, 11:27 a.m.</i></p>









<div style="margin-top:1.5em">
 <b style="color:#575012;font-size:10pt">Repository: </b>
ktexteditor
</div><span class="">


<h1 style="color:#575012;font-size:10pt;margin-top:1.5em">Description </h1>
 <table style="border:1px solid #b8b5a0" cellpadding="10" cellspacing="0" \
bgcolor="#ffffff" width="100%">  <tbody><tr>
  <td>
   <pre style="margin:0;padding:0;white-space:pre-wrap;white-space:-moz-pre-wrap;white-space:-pre-wrap;white-space:-o-pre-wrap;word-wrap:break-word"><p \
style="padding:0;margin:0;line-height:inherit;white-space:inherit">Praat (<a \
href="http://praat.org" target="_blank">praat.org</a>) is an acoustic analysis \
software extensively used in Phonetics research. This commit adds syntax highlighting \
for its internal scripting language.</p></pre>  </td>
 </tr>
</tbody></table>


<h1 style="color:#575012;font-size:10pt;margin-top:1.5em">Testing </h1>
<table style="border:1px solid #b8b5a0" cellpadding="10" cellspacing="0" \
bgcolor="#ffffff" width="100%">  <tbody><tr>
  <td>
   <pre style="margin:0;padding:0;white-space:pre-wrap;white-space:-moz-pre-wrap;white-space:-pre-wrap;white-space:-o-pre-wrap;word-wrap:break-word"><p \
style="padding:0;margin:0;line-height:inherit;white-space:inherit">$ ./validatehl.sh \
praat.xml  praat.xml validates</p></pre>
  </td>
 </tr>
</tbody></table>


<h1 style="color:#575012;font-size:10pt;margin-top:1.5em">Diffs </h1>
<ul style="margin-left:3em;padding-left:0">

 <li>src/syntax/data/praat.xml <span style="color:grey">(PRE-CREATION)</span></li>

</ul>

<p><a href="https://git.reviewboard.kde.org/r/128905/diff/" style="margin-left:3em" \
target="_blank">View Diff</a></p>






  </span></td>
 </tr>
</tbody></table>







  </div>
 </div>


</blockquote></div><br></div>



[prev in list] [next in list] [prev in thread] [next in thread] 

Configure | About | News | Add a list | Sponsored by KoreLogic