TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 18827](https://teamcity.mesosphere.io/viewLog.html?buildId=160265&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 1, ignored: 254 Build time: 00:04:37


Typo in commit message... Actually #77 was finalized (the rest info is correct).


Ansible is procedural and idempotent, Heat is declarative. Therefore I don't think it is enough just to ensure that the stack exists. Instead a stack-update should be triggered with the supplied template and environment arguments. If nothing has change then the stack-update is idempotent, otherwise heat will make the necessary changes to bring the resources in line with the declared template.

The rub here is that shade doesn't support stack_update yet ;) I intend to propose it soon


The name and email will be replaced within the file that's offered to "interested researchers"? The first item in the list says that names/emails won't be available to the public anyway, so it sounds a little confusing.


Initial test that zero is equal to zero.


OK, I'll rebase then


Thanks!


Noticed, thank you.

/Commit f605a59


This fixed the A2DP issues introduced with 342c27c353b4221c3da3f0804b7e973caa05d2c1


pylint stores the results from the previous time you ran it in ~/.pylint.d (yeah they clutter up your home directory as well). So if you want to just see if you are making improvements, you can just run this script every time you make changes to the code. If you want results consistent with what you see in jenkins, you have to first checkout master, then run pylint, then checkout the branch you are working on, and run this script on the working branch.

Just and FYI on how this would work.


Blob is final, it cannot be mocked. Will revisit later.


Yes. I both re-use the default display widgets, and when they are not enough, add fragments:

```
    <!-- Cleanup -->
    <drop css:content="fieldset#folder-listing" />
    <drop css:content="label" />
    <drop css:content=".fieldErrorBox" />

    <!-- Disable the default theme containers -->
    <drop css:theme="#content-container" />
    <drop css:theme="#column1-container" />
    <drop css:theme="#column2-container" />

    <!-- Main abstract content -->
    <append css:theme="#main-container">
      <div class="row">
        <div class="col-xs-12 col-sm-12" style="text-align: right">
          <xsl:copy-of select="document('@@theme-fragment/abstract.volumeline',
                               $diazo-base-document)/html/body/*" />
        </div>
      </div>
      <div class="row">
        <div class="col-xs-12 col-sm-12">
          <xsl:apply-templates css:select=".documentFirstHeading" />
        </div>
      </div>
      <div class="row">
        <div class="col-xs-12 col-sm-3">
          <xsl:copy-of select="document('@@theme-fragment/abstract.authors',
                          $diazo-base-document)/html/body/*" />
        </div>
        <div class="col-xs-12 col-sm-9">
          <xsl:apply-templates css:select=".documentDescription" />
          <xsl:apply-templates css:select="#formfield-form-widgets-notes" />
          <xsl:copy-of select="document('@@theme-fragment/abstract.readlink',
                               $diazo-base-document)/html/body/*" />
          <xsl:copy-of select="document('@@theme-fragment/share-links',
                               $diazo-base-document)/html/body/*" />
        </div>
      </div>
    </append>
```


and here we have one closing bracket too less.^^


Here the second test to see that 2 equals 2 pennies. The source code is changed so that "pennies" is equal to the amount given.


I try with lmast commit i have this
>{rsx::thread} RSX: Program destructor invoked!
{rsx::thread} Exception: Assertion failed! Result is C4650338h
(in file Emu\RSX\VK\VKTexture.cpp:162, in function vk::texture::create)
RSX: Program destructor invoked!


why?


"Going to the email" would make more sense if replaced with "**Getting** the email", since the email is an address, rather than a place.


it's possible that a board could have both enabled at the same time. Each could be individually addressed through the bus. However the mcode would need to be changed to allow selection of either device.


wow pindel programista


Fix for #3013


seems fine. a local variable might make it feel less redundant. :)


If you're using Babel for ES6 you can use arrow functions instead of `_.bind`.


Oops! Missed this commit
Cherry-picking...


LGTM


`_.first` exists as an alias.


This works.


I don't recommend using this version as it contains a lot of extra code that would never be used. The simplified version in my later commit cleans it up and makes it more readable.


I'm a core heat developer and ansible user, so I have opinions on os_stack ;)

Let me know if there is any way I can help with this, including coding if necessary.


You forgot blue -> blauw


PURPOSE OF THIS SCRIPT: build and test models to predict the winner of NFL games using information from the GAME.csv file


Here the test is to check that a nickle is given if five is the amount. The initial hash is changed to the way it was initially. Two if statements are given one to make nickles true and one to make pennies true.


Ah, here's the problem.  The #1 is not getting substituted because it is inside \verb.  Not sure how to fix.


Haha, fixed. Thank you again.


update_travis_toolchain (lefticus) - Win64-Windows-10-VisualStudio-14-Debug: OK (390 of 390 tests passed, 0 test warnings)

![Build Badge](http://img.shields.io/badge/build%20status-warnings-yellow.svg) ![Test Badge](http://img.shields.io/badge/tests%20passed-100.0%25-green.svg)


2007-hvac-system-list-order (kbenne) - Win64-Windows-7-VisualStudio-12: Tests Failed (1902 of 2397 tests passed, 0 test warnings)

![Build Badge](http://img.shields.io/badge/build%20status-warnings-yellow.svg) ![Test Badge](http://img.shields.io/badge/tests%20passed-79.35%25-red.svg)


I'm leaking the first string here, I should check if I already passed above and, in case, clean the string before making a new one


I can't get this to work. If you move to ssh for your git access you'll need to change this.


@eddywashere dis?


update_travis_toolchain (lefticus) - x86_64-Linux-Ubuntu-14.04-clang-3.7-LibC++: Build Failed

![Build Badge](http://img.shields.io/badge/build%20status-failing-red.svg) ![Test Badge](http://img.shields.io/badge/tests%20passed-100.0%25-green.svg)


Strange -- the name `memoise` rather than `rex` is printed?


The consuming code (ov_string_print) is IIRC not happy with a NULL. Perhaps the right code would be
*strOut = '\0';
@LEvertz @Stasik0


should probably add something to determine the vref, it is useful in current calculations


update_travis_toolchain (lefticus) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: Build Failed

![Build Badge](http://img.shields.io/badge/build%20status-failing-red.svg) ![Test Badge](http://img.shields.io/badge/tests%20passed-0.0%25-red.svg)


This seems to have landed without a PR-URL 😭


Could this be just a local variable instead?


Could these be to-upstream web-platform-tests instead?


This has been replaced with a better version in the newer commit. It prints actual current value based on the vref.


and these.


- [ ] rename `max_num_bytes` to `buf_len` & check for equality not greater than at deserialization.
- [ ] __attribute__((warn_unused_result)) would need to be hidden via a macro and defined only on supported compilers (gcc & clang), we also compile on ICC.
- [ ] In the array container type (and run) you don't need to serialize all of capacity, only cardinality. They're dynamic vectors where `cardinality <= capacity`.
- [ ] Again in array container type (and run) we make strong assumption in the code that data is sorted and all distinct values, this should be validated at deserialization. A simple for loop with a check of strictly monotonic values should suffice. I do not recommend sorting if the serialization code does not sort.
- [ ] `bitset_container_deserialize` should force (or not) the computation of cardinality. If it doesn't compute it, set it to the sentinel value `-1` (it might be useful to define a macro for this). Still, the field needs to be validated with the actual storage.


ACK


Think you closed the wrong issue FYI


As noted there (REANNZ@f25cc1a4b5567569d56f91fa02b3b9342661f946), the swappable property does have an effect (understood), but it does not extend to quietly ignoring its' migrations like in this case.

Anyway, it's OK.


Thanks, has created pull request #9320, was the typical copy and paste from another place ;)


Has anyone noticed that the messages are repeated???


This test is to see if appropriate change is given for six. The source code had an if statement added and if the amount is greater than five to give one nickle and then take the amount - 5 for pennies.


You need to be in call

It appears as the right most small icon near the top.


What's your rule of prefixing variables with `$` sign?


we have OV_VL_MAXINT for that :-)


This unused/broken deprecation functionality was removed as it duplicates the functionality of `maintain.deprecated`


Does this mean we're getting rid of \stdout completely?


Users shouldn't need to change the extension name, they just need to provide the name without the file extension.


looking good! :+1:


the two if statements were flipped because if the lesser amount could full fill the amount it would end before getting to the rest of the code.


update_travis_toolchain (lefticus) - x86_64-Linux-Ubuntu-14.04-gcc-4.9: OK (398 of 398 tests passed, 0 test warnings)

![Build Badge](http://img.shields.io/badge/build%20status-warnings-yellow.svg) ![Test Badge](http://img.shields.io/badge/tests%20passed-100.0%25-green.svg)


Fixed! Thanks @TravisEz13  - this is really good info to know and excellent to have in there :+1:


I'm not sure this tests is covered in CalendarHelpers, right? @tarekgh


JQuery objects in JS.

var $canavas = $('canvas');


All references in Kodi's Makefile.ins are wrong (as you may have figured in my commit), but my cleanup was rejected by other Team Kodi members. For example, I got this reply just today:
https://github.com/xbmc/xbmc/commit/5138926fd8009f27613f2b08a8f0423be4a7779f#commitcomment-16612366
Out-of-tree works well in my fork - I use it all the time. I have yet to see an advantage of switching to cmake.


Not a problem! Trying to help you. This looks good!
You can now do a Pull Request to push this upstream.


Could you please use the README template at https://github.com/Zewo/READMEGenerator.
You don't need to use the tool just use the `README` from the repo as a guide.
Thanks!


https://mail1a.de/ this one too


If the code doesn't make if an argument is null, then throw. If you can properly deal with it, don't. So initialize it with an empty array.
I can't conclusively say what the compiler will optimize (probably nothing), but the JIT might do it. In any case, this is such a tiny operation, it's unlikely that it's even worth thinking about it. You could even use a foreach as the index isn't needed.
My point is: after the constructor, guarantee that something is there and forget about any checks, be it null or empty. This isn't particularly about this class anyway. Just avoid null like it's the pest (especially when returning something from a method: return an empty list or array, but never null).


You can found this exporting a site in Legend mode and checking it with the contents of "site_color_key.txt"


No, it will handle GH cases where you clone with either HTTPS or SSH. That's why there is the replacement of "git@github.com" with "https://github.com".


@garagatyi does this doc https://wiki.codenvycorp.com/display/PSR/REST+services+links impact reference's name (using dots as word separator) as well?


OK, how about: `The model equations are represented by all physics equations and material relations`
alternatively, we define an EQUATION as just an object that has in addition to CUDS_COMPONENT also `physics_variables` and also `models `defined. The fact that it has `models `, will hint that it is in fact a model equation.


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 18829](https://teamcity.mesosphere.io/viewLog.html?buildId=160306&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 1, ignored: 254 Build time: 00:06:12


No problem, also a good thing is to be aware that these kind of things are still in the git history, and [could actually be sought for](https://github.com/kootenpv/gittyleaks). Just saying, it's probably not dramatic in this case.


thanks!


nope, right one


Thanks for this @jay-zhu!


Here the lines were removed and turned into arrays.


@TimGraf  Sorry, somehow I forgot about your prototype! Thanks for that.
The theading issue here is due to the execution context. Use 'play.api.libs.iteratee.Execution.Implicits.trampoline' which is basically just executing everything on the same thread. This one is also commonly used in filters - as long as its non blocking and fast operations.


lol


@misternu


yes, i haven't created the meetup entry, yet


есть мнение, что pop не должна возвращать значение, которое убирает.
см. http://www.amazon.com/Exceptional-Engineering-Programming-Problems-Solutions/dp/0201615622 или http://embedded.cs.ccu.edu.tw/OldVersionWebPages/vertaf/internal/wares/Quantum%20Framework/Resources/Cargill94.pdf


Выражения, где вычисляются непрострелянные банки, можно сократить, т.к. из числа вычитается это же число, что дает 0;


did someone just add advertising to the project,  i will have to research how to delete

    On Thursday, March 10, 2016 12:58 PM, Gene Leynes <notifications@github.com> wrote:

 Delete the previous comments I mean.—
Reply to this email directly or view it on GitHub.


@danmconrad @jimbol

Trying again. Short version this time.

Moved padding/margin into the Icon to make a button with background, and then used padding and negative margins on the Touchable* wrapper to create an arbitrarily large hitbox that doesn't affect layout or visual rendering. Might be a useful technique.


Is this still needed?


typo it should canola and not canols


I would prefer if we get it to work with a fair ReentrantReadWriteLock, I 'm just not sure in what order the StampedLock grants access...


latex-documentation (michael-okeefe) - x86_64-MacOS-10.9-clang: OK (2129 of 2129 tests passed, 0 test warnings)

![Build Badge](http://img.shields.io/badge/build%20status-passing-green.svg) ![Test Badge](http://img.shields.io/badge/tests%20passed-100.0%25-green.svg)


Should update this to use new cmap support (now merged), and to choose a better default colormap.


Why does `img/` and `fonts/` have trailing slash but `js` and `css` do not?


closes #680


Que nivel, puto amo de Java ....


Why are we still using require here?


Yeah, the duplication is a bit unfortunate, but the deduplicated version is worse. Dropping.


I think this dash can be removed


@paulofaria  done


contrib/codeanalysis/runcppcheck.sh: line 9: ../cppcheck-1.66/cppcheck: No such file or directory


Sure. Do you recall seeing this change before you put line 370 above in? It's kind of frustrating to me because I can't work out exactly how `NODE_ENV` gets set to `"production"` in my case. I should probably just let it go


Oeps...


87382760-VAVTerminalUnitSchedule-Issue4285 (mjwitte) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: OK (1581 of 1581 tests passed, 0 test warnings)

![Build Badge](http://img.shields.io/badge/build%20status-warnings-yellow.svg) ![Test Badge](http://img.shields.io/badge/tests%20passed-100.0%25-green.svg) ![Coverage Badge](http://img.shields.io/badge/coverage%20status-70.85%25-green.svg)


What do you mean?


You should not push your own credentials over the app.php default template


@klq I believe I've finished Pre-Work part 1.


An issue is here!


Here in line 4, there's a spelling mistake!


Oh, this'll end up below!!


Oh, this'll too end up below!!


Français,
   日本語,


Can the `url` be `http://the.earth.li/~sgtatham/putty/0.67/x86/putty.zip` instead? Otherwise the hash will break when the latest version changes.


awesome!! :rocket:


@SeanFeldman - this is the guidance that's been added for Sql Scripting as well. The related change about the EnclosedMessageTypes is on headers.md


Did you get it working on a Pi ? Kind Regards.


Please remove query from view before we merge


Could you explain this change?  The checks this function does are different than the ones of the old code.


Hey @scakemyer,
Would you be so kind as to explain what the ratio settings you added here do?

So, "Share ratio limit (percent)" and "Seed time limit (hours)" I get.
"Seed time ratio limit (percent)" is using what time as base for the %?

If we use all, which one has priority? And finally, these only work if we have "Background handling of torrents" enabled, right?

Thanks :)


Try moving the nominal base height up. I think you're running into problems because the motors are trying to go below 0 degrees, so the bisection is "timing out."

On that note, we should implement a check that kicks out of the iteration loop if angle_max=0 or if angle_min=pi/2

Sent from my iPhone

> On Mar 10, 2016, at 5:03 PM, megantuttle <notifications@github.com> wrote:
>
> @Alexanderallenbrown I got rid of fminsearch, but angle bisection method takes so long I can't actually tell what is happening
>
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/Alexanderallenbrown/MotionBase/commit/77002aadb6b971e959c321e67a13b4909c5198c1


@m-mcgowan While testing this I also noticed that most of the debugging statements were commented out.  Due to the fact that they kind of overrun the logs with too much info.  There is an extraneous statement in there still that fills the buffer with null terminators
```
#ifdef MDM_DEBUG
     memset(buf, '\0', len);
 #endif
```
Perhaps we can comment that out as well as part of this soon to be PR as it's unnecessary.


It's a good candidate for optimization. There are some calls that use a denominator or `1` so there are savings to be had. I'd like not to do it in this PR though. The code that scales to keep as much precision as possible would need the same check. It's not _that_ bad, but I'd like keep version 1's code as straight-forward as possible and add the optimizations later.


Thanks.  Misunderstood and thought it was being deprecated in favor of
```js
_.head
```


Actually, thinking about it again, I am probably wrong...


I found out why, pillow stopped support for not saying from PIL for some reason.


@indualagarsamy I had a comment in [ASB native sample](http://docs.particular.net/samples/azure/native-integration-asb/#necessaryheaders-v6_x) that was a bit more general.


Haha nice gif I like it!


@kpeckett Right. Although I think we could remove this anyway.


87382760-VAVTerminalUnitSchedule-Issue4285 (mjwitte) - Win64-Windows-7-VisualStudio-12: OK (2129 of 2137 tests passed, 3 test warnings)

 * 10 tests had: EIO diffs.
 * 8 tests had: ESO big diffs.
 * 7 tests had: Table big diffs.
 * 2 tests had: AUD diffs.
 * 6 tests had: ERR diffs.
 * 2 tests had: MDD diffs.
 * 2 tests had: MTD diffs.
 * 4 tests had: MTR small diffs.
 * 2 tests had: RDD diffs.
 * 3 tests had: ESO small diffs.
 * 1 test had: Table small diffs.
 * 4 tests had: MTR big diffs.

![Build Badge](http://img.shields.io/badge/build%20status-passing-green.svg) ![Test Badge](http://img.shields.io/badge/tests%20passed-99.63%25-yellow.svg)


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 18832](https://teamcity.mesosphere.io/viewLog.html?buildId=160333&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 1, ignored: 243 Build time: 00:04:36


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 18833](https://teamcity.mesosphere.io/viewLog.html?buildId=160337&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 1, ignored: 254 Build time: 00:04:47


Nisi ubacio dj_static u requirements.txt @0xspx  molim te pazi na to


Garner calls this RAS syndrome (Redundant Acronym Syndrome syndrome). Can you move this, and all the other RAS syndrome entries, into their own check in this file, `after_the_deadline.redundancy.ras_syndrome`?


Definitely :) One thing to note in this case is that these creds are already assumed to be public because they will be visible when viewing the source of an angular app. Auth0 accounts also come with a secret key for the server which should always be kept out of commit history :)


If you want to support current code for some backwards compatibility, this has to be the other way around. Ie, RDF4JException must inherit from OpenRDFException.


Istanbul coverage :
100% Statements 48/48
100% Branches 20/20
100% Functions 11/11
100% Lines 48/48


(Move to RAS syndrome. I'll comment on the first few of these, and then stop.)


This is totally wrong. You are just calling `saveMetaModelLibraryTreeItem` which is same as clicking on `save` button.

Btw, this is also crashing the editor. Steps to reproduce crash,

- Open `pendulum.xml` from TLMPlugin/MetaModels/OmcOmcPendulum.
- Double click on `pendulum` in the Libraries Browser.
- Click `Save As` button.


Takav nacin URL specifikacije django kaze da je deprecated @0xspx  obrati paznju na to


Of course. I didn't delete it because it's little note from me to me ;-)
When I will write new version I will use only english language for comment. Thank for you note!


However, to support backwards compatibility the package name must also not change.


Validates can be done in one lost to save space


update_travis_toolchain (lefticus) - i386-Windows-10-VisualStudio-14: OK (390 of 390 tests passed, 0 test warnings)

![Build Badge](http://img.shields.io/badge/build%20status-warnings-yellow.svg) ![Test Badge](http://img.shields.io/badge/tests%20passed-100.0%25-green.svg)


Then at the same time, our main concern is not the request itself but more the background operations (such as S3 uploads in the background or sending emails). and we won#t protect these with a lock in the filters.
The easiest solution might be to use an AtomicBoolean (serverRunning) and return serviceUnavailable as soon as it#s flipping to false.
And the graceful period is then as simple as sleeping for whatever time?


Please make the correct form "Bō".


Molim te ubaci lepo verzije @0xspx u requirements.txt kako su bili ranie


This is just a simple introduction.


*list


(RAS syndrome, last mention.)


update_travis_toolchain (lefticus) - x86_64-Linux-Ubuntu-14.04-gcc-5: OK (398 of 398 tests passed, 0 test warnings)

![Build Badge](http://img.shields.io/badge/build%20status-passing-green.svg) ![Test Badge](http://img.shields.io/badge/tests%20passed-100.0%25-green.svg)


Please keep the tests that are not covered by CalendarHelpers. or add the test to the CalendarHelpers.


Should be indented with additional 2x indentation width, i.e. 8 spaces.
If you import [settings.jar](https://github.com/k9mail/k-9/blob/master/tools/android-studio/settings.jar?raw=true), Android Studio should do the right thing when wrapping lines. See also [CodeStyle](https://github.com/k9mail/k-9/wiki/CodeStyle).


:)
On Fri, Mar 11, 2016 at 12:05 AM Paulo Faria <notifications@github.com>
wrote:

> awesome!! [image: :rocket:]
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/Zewo/INIParser/commit/b74a6f92f5ab30cc90b2577dc1fe884093ccde94#commitcomment-16630602>
> .
>


First CM, then Omni?
What's next? Bugfree AOSP? Slim Rom?^^


To bova morali še popravit. Zaenkrat je narejeno tako, da preriše kvadratek (z belim kvadratkom), ki ga želi izbrisati. Rabim le toliko, da vidim ali razred Igra deluje pravilno in ali z razredom Gui pravilno "komunicirata".


I wonder if they fixed this triangle to be a bit more generic now.  There are lots of hacks around making the stupid triangle work depending on the hover card orientation.


develop (axelstudios) - x86_64-Linux-Ubuntu-14.04-clang-3.5: OK (2336 of 2397 tests passed, 0 test warnings)

![Build Badge](http://img.shields.io/badge/build%20status-warnings-yellow.svg) ![Test Badge](http://img.shields.io/badge/tests%20passed-97.46%25-yellow.svg)


Actually, Jason, this is just playing around to demonstrate...


On second thoughts, if we eliminate 2. as an option then a string value is interpreted as a file path, and a dict value is an inline template.


/*
 * To change this license header, choose License Headers in Project Properties.
 * To change this template file, choose Tools | Templates
 * and open the template in the editor.
 */
package time1packagetest;

import com.deitel.javafp.ch08.Time1;

public class Time1PackageTest {

    public static void main(String[] args) {

        // create and initialize a Time object
        Time1 time = new Time1();

        // output string representation of the time
        System.out.print("The initial universal time is : ");
        System.out.println(time.toUniversalString());
        System.out.print( "The standard time is : ");
        System.out.println( time.toString());
        System.out.println();

        // change time and output updated time
        time.setTime (13, 27, 6);
        System.out.print("Universal time after setTime is : ");
        System.out.println( time.toUniversalString());
        System.out.print( "Standard time after setTime is: ");
        System.out.println(time.toString);
        System.out.println();

        // attempt to set time with invalid values
        try
        {
            time.setTime( 99, 99, 99 );

        } // end try
        catch (IllegalArgumentException e)
        {
          System.out.printf( "Exception: %s\n\n", e.getMessage());

        }

        // display time after attempt to set invalid values
        System.out.println( "After attempting invalid settings:");
        System.out.print( "Universal time: ");
        System.out.println(time.toUniversalString());
        System.out.print( "Standard time");
        System.out.println( time.toString);

    }

}


typo


Last but not least, it would be great if you could test this with Matt/Ben.
The most important part is to verify how the Loadbalancer takes a node offline.
- 503 on the version route?
- 503 on any response?


Please another entry with the alternate form "Challah".


#5507-Polygonal-and-TriangularDoor-have-the-same-value (mjwitte) - x86_64-MacOS-10.9-clang: OK (2129 of 2129 tests passed, 0 test warnings)

![Build Badge](http://img.shields.io/badge/build%20status-passing-green.svg) ![Test Badge](http://img.shields.io/badge/tests%20passed-100.0%25-green.svg)


Yes, PORT and STOP_PORT isn't constants.
It will be fixed.


Is there any good reason for this double slash?
-> `www.geocaching.com//track`


is there a unit test for this?


Probably should have 'or die...'


This is what a comment on the main scene looks like.


there we go. that looks way more logical to me now :D
thumbs up


@csirving, there are **bits** in the prolog/epilog needed on the production deployment of drive-data-roll to enable testing.


[HZR] 2.2 Developer loads Hot Zone Regions as GeoJSON overlay


Here are my thoughts, starting with the most pedantic
1) ranks = ["Ace", "2", "3", "4", "5", "6", "7", "8", "9", "10", "Joker", "Queen", "King"] is probably not how card ranks work. Jacks seem important.
2) Jokers might also be important, but we only want 2 of them per deck, and they don't have suits, so we should probably think about that use case.(edited)
3) I would question the wisdom of defining card ranks (and anything that relies on card ranks, like addition, subtraction and the lt and gt) within the card class. What if there is a game where we want aces to be a higher value than Kings? That leaves us monkey patching values in all of those games, and that feels wrong to me.
4) I don't know what the line length limit is set to, but I'm almost never in favor of enforcing short line lengths. I find 4 spaces to be more legible than 2 and I think descriptive variable names are almost always worth it, and that having to shorten them in favor of derpy line length requirements is almost always a bad idea for code tractability. I mean, there are obviously exceptions. If someone's sshing in somewhere and reading the code in their terminal, then they'll probably disagree, but I don't think there will be many people that have to do that with this project. (Or most projects now, considering that it's 2016, and in general, this is not the way things are done anymore.)


Scroll up and down, the WTF, THEY ... BLOCKED IT repeated twice


With the current settings Android Studio formats it like this:

```java
    public void delete(String trashFolderName) throws MessagingException {
    }
```

So we should stick to that. Otherwise I prefer the version without space between opening and closing brace.


[BANDARKIU](http://www.bandarkiu.asia) MERUPAKAN AGEN POKER ONLINE INDONESIA, DOMINOQQ ONLINE, DAFTAR BANDARKIU ONLINE SEKARANG JUGA,POKER ONLINE DAN DOMINOQ TERPERCAYA, MINIMAL DEPOSIT RP 25.000, 1 WEB 5 GAME: POKER ONLINE, DOMINOQQ, ADUQ, BANDAR Q, CAPSA ONLINE


Kill all these "State of Xs". For some of these, e.g., New York, it's useful to be able to distinguish the City of New York from the State of New York. For other, such as North Carolina, their government seems to use both forms (see the title o f http://www.nc.gov/).


Omni will be the last 5.1 project for the P8000 I think. ;)


...also it's still not compiling, so this is just a test as of now


I don't understand the commit message, specifically cleaning of a card leading to dirtying the previous or next one.  I assume that's because of global remsets, i.e. pointers into the nursery?  Shouldn't those card be dirty anyway?


Your guess is as good as anyone else's.. Probably.


On the Tic-Tac-Toe, you have some useless lines of code.


develop (axelstudios) - x86_64-Linux-Ubuntu-14.04-cppcheck-1.61: OK (0 of 0 tests passed, 0 test warnings)

![Build Badge](http://img.shields.io/badge/build%20status-warnings-yellow.svg)


Here are my thoughts, starting with the most pedantic
1) ranks = ["Ace", "2", "3", "4", "5", "6", "7", "8", "9", "10", "Joker", "Queen", "King"] is probably not how card ranks work. Jacks seem important.
2) Jokers might also be important, but we only want 2 of them per deck, and they don't have suits, so we should probably think about that use case.(edited)
3) I would question the wisdom of defining card ranks (and anything that relies on card ranks, like addition, subtraction and the lt and gt) within the card class. What if there is a game where we want aces to be a higher value than Kings? That leaves us monkey patching values in all of those games, and that feels wrong to me.
4) I don't know what the line length limit is set to, but I'm almost never in favor of enforcing short line lengths. I find 4 spaces to be more legible than 2 and I think descriptive variable names are almost always worth it, and that having to shorten them in favor of derpy line length requirements is almost always a bad idea for code tractability.
I mean, there are obviously exceptions. If someone's sshing in somewhere and reading the code in their terminal, then they'll probably disagree, but I don't think there will be many people that have to do that with this project. (Or most projects now, considering that it's 2016, and in general, this is not the way things are done anymore.)


One can't have enough error checking and commenting of error conditions. In this case we've actually seen this problem in Torin. While it doesn't currently occur (due to kBitmap having been implemented), it may do so again in the future, and in that case a warning or error might be helpful.


Also, to keep with the formatting in the project as a whole, you shouldn't have a space before a parameter. For example:
`method( param1,param2);` - bad
`method(param1,param2 );` - bad
`method(param1, param2);` - good


Yes, You are right. By the way the crash is not due to my changes. Whenever we follow the steps  your provided the editor crashes. The crash doesn't happen when we create a new metamodel and do Save or save as button.


Are you sure this dependency? :)


SoftMine is a fork of PocketMine-MP


That's weird.  I'm currently uploading the Tic-tac-toe project.  And I'm making the tic-tac-toe complete and modular before I improve it's efficiency.


Which lines are you talking about?


Looks good to me.


Should be `} else {`


Istanbul coverage:
- 100% Statements 48/48
- 100% Branches 20/20
- 100% Functions 11/11
- 100% Lines 48/48


Awesome!


We have this one, and p.m., in `nordquist.redundancy` already. It's fine to leave it here, but we'll probably merge these eventually.


Unable to forward Observatory port 8181: Error code 1 returned when attempting to run command: /usr/local/google/home/eseidel/Android/Sdk/platform-tools/adb forward 8181 8181

This commit looks suspicious.

Those should be tcp:8181 iirc.


i agree now we have a "memleak" on the memstack, nice find @HolgerJeromin


Можно просто указать всю папку. Часто папки, созданные редакторами/ide указывают в глобальных .gitignore, т.к. на проекте разные люди могут пользоваться разными редакторами (чтобы не захламлять локальный .gitignore)


Kill this. There are other forms of accounting (see https://en.wikipedia.org/wiki/Financial_accounting).


@jackiehluo Hey Jackie, thanks for making these changes! I almost filed an issue a while ago requesting that each of these toolbars (for the preferences view and the single panel thread list view) get a unique class added to them. From what I can tell, there isn't any good way of just targeting these.


How do we generate code that allocates a new ref? Should this be `Lift(RefNew(c.v))`? Do we need both, maybe?


@kpeckett Fixed in b3a03cbcf4e0725296e482d8c36cf630d637ec69


I created a fork


TeamCity ReOpenLDAP :: devel [Build 240](https://binbug.bigbrother-matrix.ru/teamcity/viewLog.html?buildId=1871&buildTypeId=ReOpenLDAP_Devel&guest=1) is now running


By the way, I added a README.md


update_travis_toolchain (lefticus) - x86_64-Linux-Ubuntu-14.04-cppcheck-1.61: OK (0 of 0 tests passed, 0 test warnings)

![Build Badge](http://img.shields.io/badge/build%20status-warnings-yellow.svg)


Done.


looks good


Why you decided to use arrays instead of `gnu.trove.list.TIntList` and `gnu.trove.list.TByteList`?
These collections are quite small and fast. Also, you can remove a bunch of `System.arraycopy` calls.


Let's kill only "add up", which can be used in the idiom "don't add up" and can't be replaced by "add".


Yup.  Looks like this broke flutter start. :(  Reverting.


Test comment please ignore.


This can be further simplified to:

```java
String mimeType = part.getMimeType();
return isSameMimeType(mimeType, "text/html") || isSameMimeType(mimeType, "text/plain");
```


@kpeckett As pointed out on Slack:
While working on CB, we are not using an unofficial version of CB, so we should have to change that to the "developer" version.


Kill this. According to Wallace, figuring out which form is preferred is a hairy problem.


Thanks. Will update with next commit to this pull request


ok


Currently, it's too complex according to best practices (PEP 8).  But I'll improve it once I finish the whole project.


hello,

I have tried this but nothing is happening. Is this working?

Thanks!


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 18834](https://teamcity.mesosphere.io/viewLog.html?buildId=160345&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **FAILURE**
Summary: Exit code 1 Build time: 00:05:09


@RedstoneAlmeida but wait not all people are using SoftMine :/


OK


Should this be `rc3.d`?


Is there some magic going on here or am I just being dense? How is an npm module being referenced by a different name?


Agreed on the seperate functions.  Refactor it down to something more manageable.  Same could be said for the auto initialization pieces (Everything before waitforstart).


update_travis_toolchain (lefticus) - Win64-Windows-10-VisualStudio-14: OK (390 of 390 tests passed, 0 test warnings)

![Build Badge](http://img.shields.io/badge/build%20status-warnings-yellow.svg) ![Test Badge](http://img.shields.io/badge/tests%20passed-100.0%25-green.svg)


One thing you could do to not include it every test is to move 30-49 here (the rest of the inside of the loop) out into a separate function which accepts TestCase, AtPath, and Ucommand, then copy this assignment of AtPath and Ucommand into your test (so you can make a file in the temporary test directory), make the file, and then pass the Case, at, and ucmd to the new function.

I'm probably not saying this very clearly, let me know and I can clarify on anything.


But which lines are you talking about, so I can look at them?


new changes


@jamiewilson Yeah, you're right! Where exactly are these two elements? I couldn't find them, and we wanted to make sure that the builds were passing, so I just commented out these two lines. It probably wouldn't hurt to file an issue to make this work in the future.


Suggested image belongs to a different example in the tutorial (check 2.1 Static Sites and 2.4 Dockerfile)


Lines with if and return.


The test suite finished running the test files. Below is a summary of the test results:

| Report name | Tests passed |
| ----------- | ------------ |
| _Broken | **11/13** |
| _DVB | 5/5 |
| _DVD | **0/3** |
| _DVR-MS | **0/2** |
| _General | **11/27** |
| _Hauppage | **0/3** |
| _MP4 | 2/2 |
| _NoCC | 10/10 |
| _Teletext | 21/21 |
| _WTV | 13/13 |
| _XDS | **30/34** |
| _Options | **82/87** |
| _DTVCC | **11/14** |
It seems that not all tests were passed completely. This is an indication that the output of some files is not as expected (but might be according to you). Please check the [results](http://ccextractor.canihavesome.coffee/reports/174) page, and verify those files. If you have a question about this report, please contact @canihavesomecoffee.


All of the lines with if's and return's?


At least it's not alex's if(f==0){ ... if(f==1) { } }


Done


Please use proper [Godoc](http://blog.golang.org/godoc-documenting-go-code) style documentation so that the doc parser can properly format and auto doc the function.


No, I just noticed a few


TeamCity Xbim :: Xbim Essentials :: XbimEssentials [Build 281](http://xbimbuilds.cloudapp.net/viewLog.html?buildId=832&buildTypeId=XbimEssentials_XbimEssentials) is now running


Alex did the if thing with no modifications to f.


Knowledge is the way


@ccx-testerbot runtests master


Thank you for the request. It has been added to the queue (id: 175). To see the progress you can go to the [status](http://ccextractor.canihavesome.coffee/test/175) page. Please note that depending on the current queue, it could take a while before any results will be visible. In each case I will report back here once the tests are done.


Which functions/lines are you talking about?


looks good


See the other comment about migration vs. user action. In your opinion, would it make sense to also handle this through a migration?


I will comment on them


If you're talking about `check_if_is_divisible_by_a_certain_number()`, it won't work without the return statement.


TeamCity Xbim :: Xbim Essentials :: XbimEssentials [Build 281](http://xbimbuilds.cloudapp.net/viewLog.html?buildId=832&buildTypeId=XbimEssentials_XbimEssentials) outcome was **FAILURE**
Summary: Compilation error: Xbim.Common\Xbim.Common.csproj (new) Build time: 00:01:41