bugLaTeX semantic checker - Bugs: bug #54751, Consider raising Warning 12 for...

 
 

You are not allowed to post comments on this tracker with your current authentication level.

bug #54751: Consider raising Warning 12 for cases like "something, i.e. \(x\)"

Submitter:  Ivan Kokan <ivankokan>
Submitted:  Fri 28 Sep 2018 10:23:48 AM UTC
   
 
Category:  None Severity:  3 - Normal
Item Group:  None Status:  Fixed
Privacy:  Public Assigned to:  gvol
Open/Closed:  Closed

Fri 30 Sep 2022 04:19:39 AM UTC, comment #4: 

I have what I believe is a reasonable fix for this (and for https://savannah.nongnu.org/bugs/?53347).  I have updated the regex to "([^A-Z@.])\.[.!?:]*\s[ \`([]*[a-z]" which I think accounts for most cases that we care about.  Let me know if you think there are any other character we should skip past.

I'm hoping to make a release soon, after which it will have to propagate out to the distributions.

Ivan Andrus <gvol>
Group administrator
Tue 02 Oct 2018 02:44:17 AM UTC, comment #3: 

I realized that I have some local changes that I was working on to fix https://savannah.nongnu.org/bugs/?53347
so you're right that it doesn't work correctly now on trunk.  But even after my changes, I'm going to have to make a few other changes to get it to work.

It would be helpful if you had an idea about what characters should do what.  My thought is that things like quotes and parentheses should be skipped over like spaces and then the next letter would determine whether we warn or not. So for example "something, i.e. (x)" would warn, but  "something, i.e. (X)" would not because it would think that the capital X is starting a new sentence.  That seems like it would be fairly robust.  Thoughts?

Ivan Andrus <gvol>
Group administrator
Mon 01 Oct 2018 08:39:18 AM UTC, comment #2: 

Here is the example for attached test_12.tex:

C:\Users\Ivan\Desktop\test_12>chktex test_12.tex
ChkTeX v1.7.6 - Copyright 1995-96 Jens T. Berger Thielemann.
Compiled with POSIX extended regex support.
Warning 12 in test_12.tex line 1: Interword spacing (`\ ') should perhaps be use
d.
something, i.e. something
               ^
No errors printed; One warning printed; No user suppressed warnings; No line sup
pressed warnings.
See the manual for how to suppress some or all of these warnings/errors.

C:\Users\Ivan\Desktop\test_12>chktex --version
ChkTeX v1.7.6 - Copyright 1995-96 Jens T. Berger Thielemann.
Compiled with POSIX extended regex support.

(Installed in January through MiKTeX, no updates since then. MiKTeX is up-to-date as of yesterday.)

(file #45122)

Ivan Kokan <ivankokan>
Mon 01 Oct 2018 04:17:38 AM UTC, comment #1: 

That actually does raise a warning for me.  What doesn't is "something, i.e. (x)".  

But yes, it does seem that making a change like that would be useful.  I'll try to look into in the not too distant future.  Feel free to ping me about it if you wish.

Ivan Andrus <gvol>
Group administrator
Fri 28 Sep 2018 10:23:48 AM UTC, original submission:  

Warning 12 is being raised on this: "something, i.e. something", but not for this one: "something, i.e. \(x\)".

Of course, not considering the semantics of "i.e.", \(x\) could start the new sentence but I think it would be more useful to raise Warning 12 here as well. What do you think?

Kind regards, Ivan

Ivan Kokan <ivankokan>

 

(Note: upload size limit is set to 16384 kB, after insertion of the required escape characters.)

Attached Files
file #45122:  test_12.tex added by ivankokan (50B - application/octet-stream)

 

Depends on the following items: None found

Items that depend on this one: None found

 

Carbon-Copy List
  • -email is unavailable- added by gvol (Posted a comment)
  • -email is unavailable- added by ivankokan (Submitted the item)
  •  

    There are 0 votes so far. Votes easily highlight which items people would like to see resolved in priority, independently of the priority of the item set by tracker managers.

     

    Follow 5 latest changes.

    Date Changed by Updated Field Previous Value => Replaced by
    2022-09-30 gvol StatusConfirmed Fixed
        Assigned toNone gvol
        Open/ClosedOpen Closed
    2018-11-27 gvol StatusNone Confirmed
    2018-10-01 ivankokan Attached File- Added test_12.tex, #45122

    Back to the top

    Powered by Savane 3.13-4448.
    Corresponding source code