Lighthouse has a new layout. Prefer the old one? Return to the old layout, and switch back any time from the link at the top of each page.

Activity (RSS)

Sunday, February 07 2010
NPPCR-HUDSON-BUILD-TESTS.bat: add lea... was committed by Thell Fowler at 10:59 PM
25 [trivial] introduce Hudson job and build script was updated
  • Jocelyn Legault
    Jocelyn Legault commented at 6:17 AM

    I updated to the new GTest and GMock (1.4), and I've build both libraries in Debug and Release with both VC8 and VC9.
    I've also compiled manually N++CR with VC8...

31 Integrate the most recent Google Test | Mock was updated
  • Jocelyn Legault
    • State changed from needs_ack to acked
    by Jocelyn Legault at 5:54 AM

    GTest and GMock compile fine in all configuration, and so does N++CR when linking against the new libs.

    Good work! :-)

Friday, February 05 2010
Merge branch 'tf/LH-31/google-test-an... was committed by Thell Fowler at 10:58 PM
Merge branch 'tf/LH-25v5/introduce-hu... was committed by Thell Fowler at 10:58 PM
31 Integrate the most recent Google Test | Mock was updated
  • Thell Fowler
    Thell Fowler commented at 10:56 PM

    (from [c839c280b349f137748ae060099ddf8be896225c]) google projects: update to test/mock version 1.4

    • Remove dependency of Boost TR1 from VC 8 project since Googl...
  • Thell Fowler
    • State changed from open to needs_ack
    • Assigned user changed from Thell Fowler to npp-community
    by Thell Fowler at 8:06 PM

    This essentially invalidates LH-19 since the 1.4 update includes the side-by-side setup in the Test and Mock, so N++ needed to use it too.

  • Thell Fowler
    Thell Fowler commented at 8:04 PM

    (from [8c111e47b5b9a1f73aba8b037cf4bbf872e43f86]) google projects: update to test/mock version 1.4

    • Remove dependency of Boost TR1 from VC 8 project since Googl...
  • Thell Fowler
    • State changed from new to open
    by Thell Fowler at 3:21 AM
google projects: update to test/mock ... was committed by Thell Fowler at 10:56 PM
25 [trivial] introduce Hudson job and build script was updated
  • Thell Fowler
    Thell Fowler commented at 9:05 PM

    Should read

    select the 'Setenv Plugin' for installation.

    Corrected

    The New job creation step doesn't mention what type of job we should create. - Mentioning ...

  • Thell Fowler
    Thell Fowler commented at 8:57 PM

    (from [363433bf3f22e1d78373dd805777b8d6fd7f0d8e]) NPPCR-HUDSON-BUILD-TESTS.bat: add leakdetect switch

    • This patch makes use of the NPPCR leakdetect switch to e...

  • Jocelyn Legault
    Jocelyn Legault commented at 6:30 AM

    From the README file

    • The following line:

    select the 'Hudson Setenv Plugin' for installation.

    Should read

    select the 'Setenv Plugin' for installation.

    As i...

33 Add -leakdetection switch to output mem-leaks to the console was updated
  • Thell Fowler
    Thell Fowler commented at 8:57 PM

    (from [363433bf3f22e1d78373dd805777b8d6fd7f0d8e]) NPPCR-HUDSON-BUILD-TESTS.bat: add leakdetect switch

    • This patch makes use of the NPPCR leakdetect switch to e...

  • Jocelyn Legault
    • State changed from needs_ack to acked
    by Jocelyn Legault at 4:48 AM

    This one looks good. However, didn't we discuss we should deal with a few messages in the message pump before exiting? Did you encounter problems in that regard...

NPPCR-HUDSON-BUILD-TESTS.bat: add lea... was committed by Thell Fowler at 8:57 PM
google projects: update to test/mock ... was committed by Thell Fowler at 8:04 PM
37 [FIX] Memory leak in ScintillaEditView was updated
  • Jocelyn Legault
    • State changed from needs_ack to open
    by Jocelyn Legault at 4:29 AM

    While the fix in itself works, I think we could skip the whole dynamic assignation altogether.

    I'd go with either:

        TCHAR *generic_fontname[128];
    #ifdef UNI...
36 Notepad_plus.cpp: rewrite of menuID2LangType() was updated
  • Sune Marcher
    Sune Marcher commented at 1:43 AM

    Quick note: tested loading those 41 language files, only see a ~300ms time diff on the calls to menuID2LangType() calls, with debug version - no wonder I didn't...

  • Sune Marcher
    Sune Marcher commented at 12:37 AM

    std::map implementation uses a tree, and is therefore a O(log(n)) search. Yours is O(n), which is worst. Funnily enough, the current implementation with the sw...

Thursday, February 04 2010
36 Notepad_plus.cpp: rewrite of menuID2LangType() was updated
  • Thell Fowler
    Thell Fowler commented at 11:20 PM

    Sune,

    However, on my 2GHz laptop, running 5.000.000 iterations of 17 lookups, the original takes ~450ms and my new version ~4200ms. Considering I need ~5mil it...

  • Jocelyn Legault
    Jocelyn Legault commented at 5:26 PM

    OTOH there's several examples in the NP++ codebase with large amounts of code duplication - generalizing code results in less codelines and easier maintenance.

    ...
  • Sune Marcher
    • State changed from needs_ack to open
    • Assigned user changed from npp-community to Sune Marcher
    by Sune Marcher at 10:02 AM

    First: thanks for the feedback, guys.

    Jocelyn

    Grouping your values by 3's doesn't improve readability, but rather deteriorates it IMO.

    Well, the entries don't...

31 Integrate the most recent Google Test | Mock was updated
  • Jocelyn Legault
    Jocelyn Legault commented at 5:30 PM

    The previous version did also include GTest, but I yanked it out for the version I brought to our depot.
    My heart goes to keep them as separate projects, but I ...

  • Thell Fowler
    • Assigned user set to Thell Fowler
    by Thell Fowler at 5:20 PM

    I'll be glad to do this. Should we keep two distinct projects (Test and Mock) as submodules, or change to a single Google Mock install since it bundles Google T...

Merge branch 'tf/LH-37/ScintillaEditV... was committed by Thell Fowler at 4:12 PM
Merge branch 'tf/LH-35/scintilla-setu... was committed by Thell Fowler at 4:10 PM
37 [FIX] Memory leak in ScintillaEditView was updated
  • Thell Fowler
    • State changed from open to needs_ack
    • Assigned user changed from Thell Fowler to npp-community
    by Thell Fowler at 4:08 PM
  • Thell Fowler
    Thell Fowler commented at 4:07 PM

    (from [b020bea0056a697a9bc131227d3321838551613e]) ScintillaEditView.cpp: fix createHotSpotFromStyle mem leak

    • When a document view is activated that has a hotsp...
  • Thell Fowler
    • State changed from new to open
    by Thell Fowler at 3:41 PM
  • Thell Fowler
    Thell Fowler created the ticket at 3:39 PM

    There is a memory leak in ScintillaEditView when a hotspot style is present within an activated tab. It needs fixing.

ScintillaEditView.cpp: fix createHotS... was committed by Thell Fowler at 4:07 PM
23 [Feature] RoxyScroll was updated
34 [FIX] Memory leak in EncodingMapper was updated
  • Jocelyn Legault
    • State changed from needs_ack to acked
    by Jocelyn Legault at 12:34 AM

    Looks Kosher!

Wednesday, February 03 2010
36 Notepad_plus.cpp: rewrite of menuID2LangType() was updated
  • Jocelyn Legault
    Jocelyn Legault commented at 8:36 PM
    • Code size and line numbers isn't a metric we really do care about. That does not mean we encourage code bloat; However readability, stability, reusability and ...
  • Thell Fowler
    Thell Fowler commented at 6:23 PM
    • Load time using this patch takes 40% longer. Originally tested opening with 10 randomly chosen example files the average jumped from 5.4 to 8.4 seconds average...
  • Sune Marcher
    Sune Marcher commented at 12:16 PM

    (from [4317709c9499ee0e6dca6d81f23da9f430b6c917]) Notepad_plus.cpp: rewrite of menuID2LangType()

    • Focusing on readability, ease of extension for new language t...

  • Sune Marcher
    Sune Marcher commented at 7:19 AM

    (from [cd8e23474d39fee53b79c02e34e58de551dedf8d]) Notepad_plus.cpp: rewrite of menuID2LangType()

    • Focusing on readability, ease of extension for new language t...

  • Sune Marcher
    • Assigned user changed from Sune Marcher to npp-community
    by Sune Marcher at 7:16 AM

    (from [d0195ab4287c594215dc4fb011621817895b94c0]) Notepad_plus.cpp: rewrite of menuID2LangType()

    • rewrite for easier maintaining, as well as code&binary size.
    • S...
1 earlier change…
  • Sune Marcher
    • State changed from new to needs_ack
    • Assigned user set to Sune Marcher
    by Sune Marcher at 6:39 AM
Notepad_plus.cpp: rewrite of menuID2L... was committed by Sune Marcher at 12:16 PM
Notepad_plus.cpp: rewrite of menuID2L... was committed by Sune Marcher at 7:19 AM
12 Automate .rc file build version was updated
Notepad_plus.cpp: rewrite of menuID2L... was committed by Sune Marcher at 7:16 AM
version generator: handle spaces in p... was committed by Thell Fowler at 7:16 AM
Sune Marcher Sune Marcher was invited to npp-community at 5:33 AM
25 [trivial] introduce Hudson job and build script was updated
33 Add -leakdetection switch to output mem-leaks to the console was updated