3
Vote

Display last selected line instead of current one

description

Selecting one line (mouse click) and the moving to the previous one shows the last line selected not the current.

file attachments

comments

gonzalocontento wrote Oct 11, 2013 at 3:58 PM

in the attached image line 133 contents is shown mistakenly in 132

HagenRaab wrote Oct 14, 2013 at 1:37 PM

I cannot reproduce this. Or I don't understand the problem.

wrote Oct 14, 2013 at 3:42 PM

gonzalocontento wrote Oct 14, 2013 at 3:42 PM

In the attached image, I was in line 52 then I selected (moved the mouse to) line 51.
Contents of line 52 replaced contents in line 51.

BTW: it seems to be working when I use the keyboard to move around instead of the mouse.

Please let me know if you need any file from my configuration.

Windows 8.1 64 bits.

HagenRaab wrote Nov 21, 2013 at 9:17 PM

Actually I cannot reproduce this. Is it for all your files or just a specific one?

gonzalocontento wrote Dec 9, 2013 at 2:15 PM

Please dismiss this issue: I have reasons to believe that is some combination of hardware/driver in Windows 8.1.

My ATI FirePro 2260 is showing some additional-annoying rendering issues, even with the latest beta driver. I started to see these issues with Terminals as well.

I played with the source code and I got it to the point where the cells started to having issues. I believe it's just a combination that causes the rendering issue to show up.

Thanks for your help.

vemundha wrote Mar 13, 2014 at 1:25 PM

Hi,
It's for all log files inside logexpert.

I have tracked the issue to (on my system) Nvidia driver. My DELL has both a Intel (HD Graphics 3000) and a Nvidia graphics card (Quadro 3000M). When running only the Intel card and 1 screen (the laptop screen) it's ok. But adding the Nvidia card (with more screens or enabled it inside the Nvidia control panel.. 3d settings/manage 3d settings/preferred graphics processor) then I get this issue.

I have tested with all the latest Nvidia drivers for Win8.1 and x64 (332.50 and 334.95) and get the same issue on both.

I have seen the issue on 3 installations of Win8.1 x64.

tcbhat wrote Sep 3, 2014 at 9:06 PM

I had the same issue. I fixed this by commenting out the uncommented portion and uncommenting the commented portion of this function in LogWindow.cs
    internal string[] GetColumnsForLine(int lineNumber)
    {
        return this.columnCache.GetColumnsForLine(this.logFileReader, lineNumber, this.CurrentColumnizer, this.columnizerCallback);

        //string line = this.logFileReader.GetLogLine(lineNumber);
        //if (line != null)
        //{
        //  string[] cols;
        //  this.columnizerCallback.LineNum = lineNumber;
        //  cols = this.CurrentColumnizer.SplitLine(this.columnizerCallback, line);
        //  return cols;
        //}
        //else
        //{
        //  return null;
        //}
    }

gonzalocontento wrote Sep 15, 2014 at 7:03 PM

Awesome I applied your fix and it works

Not sure if it is safe change though!

I think is up to HagenRaab
       internal string[] GetColumnsForLine(int lineNumber)
        {
            var line = logFileReader.GetLogLine(lineNumber);
            if (line == null)
                return null;

            columnizerCallback.LineNum = lineNumber;
            var cols = CurrentColumnizer.SplitLine(columnizerCallback, line);
            return cols;
        }

vemundha wrote Feb 6, 2015 at 4:55 AM

Same issue on 1.5.5493. Tested today. I noticed that the issue is only when you click lines using the mouse and not when you move up/down with the keyboard (if that is helpful information for anyone).

HagenRaab wrote Feb 8, 2015 at 11:58 AM

Hi guys,

I still cannot reproduce this on any of my systems (Win7 32 and Win7 64). The fix of user 'tcbhat' disables the column cache. This is perfectly safe. The cache was introduced due to performance issues of the RegEx columnizer.

Anyway I'd like to solve the problem. So it would be nice if some of you could send me the configuration (Settings Export) plus an example file. Please also state if you use any columnizer and highlight settings.

Thanks

wrote Feb 12, 2015 at 2:45 PM

gonzalocontento wrote Feb 12, 2015 at 2:45 PM

Please see attachment.

For your convenience, I got rid of columnizers and highlighters. Same issue.

Regards

wrote Mar 6, 2015 at 7:36 AM

wrote Mar 12, 2015 at 3:43 PM

wrote May 13, 2015 at 11:08 AM

vemundha wrote May 13, 2015 at 11:08 AM

@hagenraab any progress on this?
I have attached the testdata from my email to you 9.feb.

Since February I have upgrade to Win10 - same issue! (And again: like I said it's not there if I run using only the laptop screen on my Dell 6800, with Nvidia K4100M. But running docked and with 2 screens (or 3 screens) I always get this issue. And I have seen this issue on multiple laptop's running Win8 and dual screen.

Let me know if you need more info.

STEPS TO REPRODUCE:
I tested with 1.5.5493.
Is there any log files. I checked %temp% did not see any straight away. If there is give me the path and I'll fw it.

In the image (attachment: debugData - line text error.zip) showing the issue. I did this.
  1. Open the file
  2. Clicked on line 6
  3. Clicked on line 5
  4. MADE A screen shoot ;)

HagenRaab wrote May 23, 2015 at 3:35 PM

Hi,

I just tested with your attached file. I also imported your settings. Still cannot reproduce on Windows 7 (running in Parallels on Mac). At work I also have 2 screens attached to a Laptop. I never spotted this issue. But it's also Win7 at work.

HagenRaab wrote May 23, 2015 at 4:01 PM

Now also tested on Win 8.0 (running in Parallels on iMac). Still cannot reproduce. But as said: It's not a Laptop.

LeoRomano wrote Jun 17, 2015 at 2:40 AM

The issue is on WIN 8.1 64 bit for sure as I've tried it on real hardware and a Hyper-V VM and can reproduce the issue.
Since it's bugging me so much, and it makes the app look bad to others (I think the tool is great but still learning it), I'll try out the fixes suggested and report back.

HagenRaab, thanks for the great product and work you've put into this!

nclemeur wrote Oct 1, 2015 at 11:18 PM

I can confirm that issue seems to be fixed in 1.6 alpha /5734. I had this issue with 1.5 which was quite annoying.

Thank you for this great app.

vemundha wrote Oct 13, 2015 at 11:54 AM

Same here. The alpha (1.6.5734) works fine! No issues with last selected text :) Hope you have a beta version soon ;)

MDouglas wrote Feb 17, 2016 at 8:23 AM

I also had the same issue on a DELL OptiPlex 9020 running Windows 10 x64. Installing the latest 1.6 ALPHA build 5883 fixes the issue (as long as the cache option is kept disabled).

mcmoutletyy wrote Jun 27, 2016 at 1:29 AM

http://www.oakleysunglassesb.us.com/
http://www.oakleysunglass.us.com/
http://www.oakleysunglasses.qc.com/
http://www.rayban-sunglasses.net.co/
http://www.raybansunglasses.qc.com/
http://www.ray-banoutlet.name/
http://www.polo--ralphlauren.net/
http://www.ralphlauren-outlet.in.net/
http://www.burberryoutletonline.org/
http://www.burberry-outlet-sale.us.com/
http://www.toms-shoes.us.org/
http://www.tomsoutlet.com.co/
http://www.michaelkorsoutlet-mk.eu.com/
http://www.michaelkorsoutlet.in.net/
http://www.christian--louboutin.net/
http://www.tory--burch.org/
http://www.toryburch-outlet.net/
http://www.gucci--outlet.org/
http://www.guccihandbag.in.net/
http://www.cheap-jordan-shoes.net/
http://www.air-jordan.in.net/
http://www.cheap-jordans.in.net/
http://www.jordansretro.in.net/
http://www.nikeshoesinc.net/
http://www.nike-shoes.in.net/
http://www.air-max-2015.net/
http://www.airmax-90.in.net/
http://www.airmax-95.in.net/
http://www.nike-free-run.net/
http://www.nikefree-5.com/
http://www.chanel--handbags.net/
http://www.chanelbags-outlet.us/
http://www.true-religion.cc/
http://www.north-faceoutlet.in.net/
http://www.coach-outlet-store.in.net/
http://www.abercrombie.ar.com/
http://www.hollister.cn.com/
http://www.hermes.us.org/
http://www.new-balanceoutlet.net/
http://www.reebok-shoes.net/
http://www.lululemon.com.co/
http://www.yogapants.eu.com/
http://www.rolex--replica.us/
http://www.omega--watches.us.com/
http://www.wedding--dresses.net/
http://www.montblancpens.in.net/
http://www.insanityworkout.us.org/
http://www.nike-mercurial.org/
http://www.jimmy--choo.us/
http://www.uggboots.mex.com/
http://www.uggs--outlet.net/
http://www.nike-rosherun.net/
http://www.roshe--run.com/
http://www.nikeroshe.net/
http://www.tiffany-andco.net/
http://www.vans--shoes.com/
http://www.timberland--boots.com/
http://www.beatsheadphones.in.net/
http://www.salvatoreferragamo-shoes.com/
http://www.cheap--jerseys.us.com/
http://www.cheapnfl-jerseys.us.com/
http://www.jerseysfrom-china.us.com/
http://www.mizunowave.net/
http://www.mizunorunning-shoes.com/
http://www.basketball--shoes.net/
http://www.cheapey-eglasses.in.net/
http://www.tommyhilfiger.us.org/
http://www.tommyhilfigeroutlet.net/
http://www.ed-hardy.in.net/
http://www.levisjeans.org/
http://www.bcbgmax.in.net/
http://www.balenciaga.in.net/
http://www.marcjacobs-handbags.in.net/
http://www.chaussureslouboutin-soldes.fr/
http://www.sac-michael-kors.fr/
http://www.nikeblazer1.fr/
http://www.nikeairforce-1.fr/
http://www.michaelkorsukpurse.co.uk/
http://www.michael-kors-canada.ca/
http://www.burberryuk.co.uk/
http://www.timberlandboot.org.uk/
http://www.oakley--sunglasses.co.uk/
http://www.louboutinshoes.org.uk/
http://www.air-jordans.co.uk/
http://www.chanel-handbags.uk/
http://www.katespade-uk.org.uk/
http://www.truereligion-jeans.co.uk/
http://www.newbalance-trainers.co.uk/
http://www.conversesale.org.uk/
http://www.lululemon.me.uk/
http://www.beats-by-dre.co.uk/
http://www.macmakeupuk.co.uk/
http://www.maccosmetics-uk.co.uk/
http://www.adidas-trainersuk.co.uk/
http://www.gucciuk.uk/
http://www.fitflop.me.uk/
http://www.adidasoriginals.org.uk/
http://www.hermes.org.uk/
http://www.pandorauk.co.uk/
http://www.rolex--watches.co.uk/
http://www.omegawatches.org.uk/
http://www.michael-jordan.co.uk/
http://www.michael--kors.org.uk/
http://www.weddingdresses.me.uk/
http://www.converse.org.uk/
http://www.hollister-uk.me.uk/
http://www.tiffany-and-co.org.uk/
http://www.airmax-90.co.uk/
http://www.louis--vuitton.me.uk/
http://www.mulberryoutlet.me.uk/
http://www.north-face.org.uk/
http://www.timberland.org.uk/
http://www.rayban--sunglasses.org.uk/
http://www.asicstrainers.org.uk/
http://www.prada.me.uk/
http://www.toms-shoes.org.uk/
http://www.tommy-hilfiger.org.uk/
http://www.marcjacobs.org.uk/
http://www.uggs.org.uk/
http://www.ugg--boots.org.uk/
http://www.ugg-australia.co.uk/
http://www.insanityworkout.co.uk/
http://www.nike-free-run.co.uk/
http://www.ferragamo.org.uk/
http://www.fitflops.me.uk/
http://www.nike-mercurial-superfly.co.uk/
http://www.nikestoreuk.co.uk/
http://www.guessfactorycanada.ca/
http://www.nike--store.fr/
http://www.chaussure-nike-pas-cher.fr/
http://www.adidas--superstar.fr/
http://www.adidas-zxflux.fr/
http://www.montre-femme-homme.fr/
http://www.michaeljordan.fr/
http://www.air-max-90.fr/
http://www.bracelet-pandora-bijoux.fr/
http://www.bracelet-swarovski-bijoux.fr/
http://www.sac--chanel.fr/
http://www.ralph--lauren.fr/
http://www.new-balance-femme-574.fr/
http://www.michael--kors.fr/
http://www.lunetterayban.fr/
http://www.tommy--hilfiger.fr/
http://www.lunette-oakley.fr/
http://www.ugg-australia.fr/
http://www.nike--huarache.fr/
http://www.asics-gellyte.fr/
http://www.scarpe-nike-store.it/
http://www.scarpe-adidas-superstar.it/
http://www.pandora-bracciali.it/
http://www.orologi-rolex.it/
http://www.tiffany--gioielli.it/
http://www.michael-jordan.it/
http://www.hogan-outlet-online.it/
http://www.swarovski-gioielli.it/
http://www.coco-chanel.it/
http://www.michael-kors-borse.it/
http://www.burberry-outlet-online.it/
http://www.hollister-co.it/
http://www.mac-makeup.com/
http://www.guess-factory.net/
http://www.adidasoutlet.net/
http://www.adidasshoesoutlet.com/
http://www.tods-shoes.org/
http://www.fendi.in.net/
http://www.guccibelts.us.org/
http://www.fitflopsale.net/
http://www.kobe9shoes.net/
http://www.kobebryantshoes10.com/
http://www.lebronshoes12.net/
http://www.lebron-11.net/
http://www.lebronjames-shoes.com/
http://www.kevindurant-shoes.net/
http://www.fivefingersshoes.org/
http://www.puma-shoes.org/
http://www.salomon-shoes.net/
http://www.dolce-and-gabbana.org/
http://www.dansko-shoes.org/
http://www.stuart-weitzman.org/
http://www.chicagoblackhawksjersey.org/
http://www.pittsburghsteelersjerseys.com/
http://www.redskinsjerseys.com/
http://www.denverbroncosjerseys.net/
http://www.goldenstatewarriorsjersey.org/
http://www.dallascowboysjerseys.net/
http://www.coltsjerseys.org/
http://www.giantsjersey.net/
http://www.chiefsjersey.net/
http://www.atlantafalconsjersey.net/
http://www.lakersjersey.org/
http://www.sanfrancisco49ersjerseys.org/
http://www.spursjersey.com/
http://www.chicagobullsjersey.com/
http://www.baltimoreravensjersey.com/
http://www.clevelandcavaliersjersey.net/
http://www.chicagobears-jerseys.net/
http://www.newyorkknicksjersey.org/
http://www.seahawksjerseys.us/
http://www.newenglandpatriotsjerseys.net/
http://www.arizonacardinalsjerseys.net/
http://www.manchesterunitedjerseys.us/
http://www.greenbaypackersjerseys.us/
http://www.chelseajerseys.net/
http://www.oaklandraidersjerseys.net/
http://www.neworleanssaintsjerseys.net/
http://www.clippersjerseys.net/
http://www.barcelona-jerseys.com/
http://www.bostoncelticsjersey.com/
http://www.newyorkjetsjerseys.com/
http://www.minnesotavikingsjerseys.net/
http://www.realmadridjerseys.us/
http://www.thunderjersey.com/
http://www.miamidolphinsjerseys.net/
http://www.philadelphiaeaglesjersey.com/
http://www.miamiheatjersey.net/
http://www.bengalsjerseys.us/
http://www.detroitlionsjerseys.com/
http://www.tennesseetitansjersey.net/
http://www.houstontexansjerseys.net/
http://www.carolinapanthersjerseys.us/
http://www.jaguarsjersey.net/
http://www.buccaneersjerseys.net/
http://www.buffalobillsjerseys.com/
http://www.ramsjersey.net/
http://www.chargersjerseys.com/
http://www.nba-jerseys.net/
http://www.clevelandbrownsjerseys.net/

wrote Sep 11, 2016 at 3:18 PM

Associated with changeset 35110: Merge branch 'Bug-1710-long-lines'

changed method to test if it triggers the problem

replaced all CurrentLogFileReader.GetLogLine with GetLine method

removed unused class

renamed file IBackgroundProcessCancelHandler

renamed interface IBackgroundProcessCancelHandler

fixed build erros

undo config changes

fixed behavior to act more like the old coding still under development

added logging to catch blocks

removed completly old logger class

replaced old logger with nlog

introduced nlog partial (safety commit ;) )

replaced fields with properties

Merge branch 'refactoring' into branches/refactoring

reenabled unit test build, fixed in CI nuget restore

BREAKING changed config to xmlSerializer

changed output path for plugins to ../bin/LogExpert/Plugins

reenabled unit test build, fixed in CI nuget restore

BREAKING changed config to xmlSerializer

changed output path for plugins to ../bin/LogExpert/Plugins

changed output path for plugins to ../bin/LogExpert/Plugins

updated solution binding

changed output path

Merge branch 'branches/refactoring' of http://gogs.ragal.de/thomas/Logexpert-git into branches/refactoring

changed output path and added .gitignore

excluded Unit test projects in release build configuration

Merge branch 'branches/refactoring' of http://gogs.ragal.de/thomas/Logexpert-git into branches/refactoring

Merge branch 'branches/refactoring' of http://gogs.ragal.de/thomas/Logexpert-git into branches/refactoring

excluded Unit test projects in release build configuration

fixed wrong NET version for NUnit

fixed wrong NET version for NUnit

changed line to be compatible to NET 3.5

updated columnizer projects to .NET 4.0

upgraded all projects to .NET 4.0
small bugfix in ProcessLauncher with index in substring
fixed plugin registry to check plugin folder and parent folder
added another unit test project but at the moment it does not really work

added option to disable column cache
tabified ConfigManager and SettingsDialog

moved filtergrid and datagrid CurrentCellAddress.Y to property
and another method moved to base

changed verrsion 1.6
some refactoring

some refactoring

moved delegate to field
introduced string format

minro changes

removed duplicate fields, methods,
moved files to new folders
created files for classes
moved outptu to ..\bin

removed unused class
minor formatting

refactored bookmarks
moved bookmark related events to BookmarkDataProvider
introduced new property BookmarkProvider

splitted hilight file
minor refactoring

minor refactoring
removed this, reordered methods, props,
removed vs hosting process

removed unused CmdLine.cs
Win32 removed unused method
added some braces and removed this.
HighlightEntry fixed misspeled assign in Entry

some refactoring
LogWindow removed some methods/variables that hides base members

minor refactoring in paramparser
added some braces to pluginRegestry

changed of some classes the accessor
moved external call to win32 class
some more refactoring
removed unused methods

some refactoring

removed unused delegate

refactored delegate to use type FilterPipe to avoid cast and removed unused parameter

refactored filter pipe
  • renamed variables
  • created regions
  • removed this
  • introduced using
refactored Filter.cs
  • removed this, unused variables
  • refactored to autoimplement property
  • renamed fields
  • changed to string format
  • ..
pulled ColumnizerCallback and LogExpertCallback to extra files
refactored some variables and introduced property for _logFileReader

fixed wrong assigment in constructor of Columnizer Callback

removed copy in build events

possible fix for issue #1155

merged changes from HagenRaab changeset 32577
including fix for #1432

removed empty else block
removed this.
added if to _calcThread.Join and only Abort on timeout

replaced EventHandler with Action not necessary to create so much objects and pass them around
merged both Calc methods in one method
replaced SetDisplayHeight with property

added new class BaseLogWindow only for refactoring
started to clean up TimeSpreadCalculator, moved SpreadEntry to new file

removed unused property sessionFileName inclusive persister field
moved LogWindowClosing to event region

minor refactoring
intrudoced method for progressBarInit
moved private method to region

removed this.

replaced some fields with auto implement properties

removed unused methods

removed LoadingFinishedFx, was always null

replaced where possible non generic delegates with generic delegates

replaced PatternStatisticFx with generic action

removed useless if

removed QualityInfo from this file

Moved fuzzyBlockDetection to extra class
moved QualityInfo to extra file
intrudoced new interface for fuzzyBlockDetection

Moved Damerau- and YetiLevenshtein to extra files
added surrounding block for one line ifs

replaced Util.IsNull with string.IsNullOrEmpty

Util.cs added regions

removed unused method FindSimilarLine1

renamed fields

removed unused field _bookmarkColor
removed unecessary this
added braces to if and for
changed double casts to one safe cast with null check

removed unused or only assigned to variables

replaced field call with property
replaced property with backing field with auto property

replaced field call with property
replaced property with backing field with auto property

removed duplicate code SelectLine

removed commented out code fragments

hopefully added to all ifs braces

added braces to some ifs

added some more regions to events

fixed tabs

moved filter paint cell method to PaintHelper
replaced PaintCell method with PaintHelper method

removed redudant method CellPainting and replaced it with call in PaintHelper

removed redudant PaintHighlightedCell method

removed redudant method BorderWiths from LogWindow
changed accesor to public

moved methods to regions and some refactoring

moved CmdLine to new folder

undo wrong delete of CmdLine.cs

file reordering
  • upgraded projects to .NET 3.5
  • *.Designer.cs minor changes in comments and generator version
changed ColumnizerLib reference to project reference
  • upgraded to vs2012 (the old vs 2008 sln still exists and working)
  • *.designer.cs minor changes like comments and used tools version
  • binded / added sln 2012 to source control
Logexpert.csproj changed PostBuildEvent, added /y parameter to override existing files

binded projects and solution to tfs

added dockpanelsuite from http://sourceforge.net/p/dockpanelsuite checkin 115 (MIT license see $/logexpert/branches/refactoring/WinFormsUI/license.txt)
  • moved LogExpert.sln to parent folder
  • removed slash from post build in LogExpert.csproj
Branched from $/logexpert/trunk