So, the majority of the differences in the formula between the default and plantdefault databases comes down to a few protons. The protonation of a formula, at least for plantdefault, is in turn dependent on how the original InChI strings were charged by Marvin Beans. To double-check that the protonation was right, I wrote this script to integrate my InChI package and check to see if the formulas in plantdefault do indeed match the formulas that you get after adjusting for the protonation.

Essentially, the InChI string has a 'p' layer, which looks like this:  /p-4/ and this instructs us to remove four protons from the formula, so NADPH, which has a complete formula of 30 protons, would have a 'charged' formula of 26 protons in the biochemistry. In default, it has 27, hence the difference in protons is flagged.

It appears to me that the InChI strings were charged correctly, so I believe that where the formula difference is only a proton, and an InChI string is associated, we should adopt the formulas in plantdefault.


Wow, I can't even imagine such place. Thanks!


Honestly; not a clue o.o I think it's from Github's desktop program lol


No, you'll get a sign-promo warning from gcc. There are not (unsigned)short overloads for to_string


There's now a trailing comma in here.


Enligt kodstandarden:
Use single quotes '' for strings.
// bad
const name = "Capt. Janeway";
// good
const name = 'Capt. Janeway';

Granskad.


Because of Mummy, like I mentioned in the first fix of Sheer Force and Rock Head.


> Don’t put people in a box.

From "How to make friends and influence people":

> Criticism is futile because it puts a person on the defensive and
usually makes him strive to justify himself. Criticism is dangerous,
because it wounds a person's precious pride, hurts his sense of
importance, and arouses resentment.
> B. F. Skinner, the world-famous psychologist, proved through his
experiments that an animal rewarded for good behavior will learn
much more rapidly and retain what it learns far more effectively than
an animal punished for bad behavior. Later studies have shown that
the same applies to humans. By criticizing, we do not make lasting
changes and often incur resentment.


Yes, it was basically already there. The change is that the part of l.1606 ~1634 is moved into same scope as c.push_back (registering compositeMaterials). Actually I don't see any problem at this moment, but I think the composite, shape, logic, and pos info should be a set and created together.  This is the motivation for the change. What do you think? (I have already checked that there is no crash in CMSSW and Fireworks with this change.)


What's a pull request?

http://stackoverflow.com/questions/16071168/whats-the-differences-btw-const-get-and-qualified-const-get


I'll do that


ooo exciting! :-)


This looks like an actual code change so shouldn't have been in a commit to fix jshint issues. Nobody looks at those commits when searching for relevant things.


ditto, merging those operations would just allow to cancel any reload that's before. Definitely involves extra bokkeeping of the configuration though.


Thanks for all the hard work! It looks like the Feral template throws off your skill points in the latest edition. I'm not sure why! (I did some snooping about to try and fix it, but no dice.)


@wsnipex @Montellese could it be that this magic here is gone now?

https://github.com/xbmc/xbmc/commit/ceeed0505ce9efebb6c1ecf46c17cc8df3525f99#diff-c33c38c51d8f6d40ff2f3c05a059541aL50

Even with this PR reverted no addons are installed into sourcedir/addons anymore.


and please add a comment:

- raw footage has alpha set to 0 and premultiplication set to Opaque. everything should work correctly with this
- if alpha is set to 1, then Roto will be difficult to use, because it will basically be drawing white on white
- plugins should handle correctly the image/clip premultiplication state, including the writer


this line as well.


Specify the correct path to the Docker options file.


Ah I see. If we check for a post-event change in ability frequently we should consider a flag or something.


Would it be beneficial to make mSeq atomic? Should we avoid locking and touching if we are setting the same sequence?


Since GENOME_* env var access now goes through Genome::Config, should ```require Genome::Env``` be removed also?  Otherwise, the commend still seems relevant.


Kanskje greit å kommentere det at man må sette opp Twython på korrekt måte før det vil fungere som ønskelig. Her er guide: https://twython.readthedocs.org/en/latest/usage/starting_out.html


ভাইয়া, এগুলোর নির্দিষ্ট তারিখ বলতে হবে। অর্থাৎ May এর ১ তারিখের জন্য অ্যাড করতে চাইলে। BD/May/1.txt ফাইল বানাতে হবে এবং সেই ফাইলে নির্দিষ্ট ফরম্যাটে লিখতে হবে। তারপর পুল রিকোয়েস্ট দিতে হবে। নোট নয়


Thank you for your comment. The distribution package of libqrencode does not require to run autogen.sh as far as I know. Because of that reason, I added a short note about autogen.sh in README as following:

> When you downloaded a development tree from github, it is required to run "autogen.sh" at first to generate configure script.


I don´t quite understand the use of writingSample, you set it to false on seek and when you finish (start) a NALU, but in this last case you immediately and unconditionally set it back to true.


@ace2nutzer is safe to set hz to 1000? Noticeable improvements?


nice! Ok. Adding that in.


> TODO: remove this and the corresponding test on the next WSK release

2 releases happened since back then. This kind of comments deserve in proper tickets anyways. As you see, you forget about these things.


TeamCity MonoGame :: Develop (Win) [Build 3.5.0.63](http://teamcity.monogame.net/viewLog.html?buildId=6871&buildTypeId=MonoGame_DevelopWin&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 485, ignored: 6 Build time: 00:16:05


And if that were the rule (which we've certainly never discussed/specified anywhere), we would have to add a check to the typechecker to enforce it, which doesn't currently exist. So I don't see what the difference is.


I am freaking out. I just did another testbuild for something i am working on and now all addons are installed properly in sourcedir/addons ... wtf is going on


I'm guessing that there's a race condition where `update` can be called twice with the same sequence number, since the process of building ledgers is asynchronous through the job queue. In the normal case the sequence number will be changed and we need to call `touch` while holding the lock anyway so I think this implementation is okay as-is but it would be nice to have more comments that make this clear.


Why "config.h.in*"?


读取 json 格式的 body 时，其实不需要先把 body 读出来，json.NewDecoder 接收一个 io.Reader 对象，http.Request.Body 实现了 io.Reader 接口。参考一下 gin 的做法：https://github.com/gin-gonic/gin/blob/develop/binding/json.go#L19


I… I think I missed a comma somewhere. Because of this error nothing loaded. Can you please try again? I pushed some more changes on `develop`.

	Alexandru Rosianu
Web & Android Developer
aluxian.com | startersquad.com
On 4 May 2015 at 18:05:57, Teng-hao Chang (notifications@github.com) wrote:

Unfortunately it does not work on my Mac. The app was launched but the
window never shows up. Nor any menus available.

2015-05-04 22:07 GMT+08:00 Aluxian <notifications@github.com>:

> Can you please test whether 067f7b5 on develop works?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/zhusee2/Facebook-Messenger-Desktop/commit/61758eb7384bea2fa6f3a9f6ef77abf612aec0ea#commitcomment-11025839>
> .
>
—
Reply to this email directly or view it on GitHub.


I haven't forgotten. There are bigger issues with the corrent generator's approach. Me or someone else will revamp it sooner or later.


I don't agree with the merge in case of return:

    Lock lock = new ReentrantLock();
    switch(c) {
      case 0:
        lock.lock(); // should raise an issue
        return;
    }
    lock.unlock();


The 32bit port number.


I was thinking about in-code comments.
Looks like the code itself is sound enough, but the code is really confusing and needs to be commented.


.idea


mhhh forget that - that build was done on the other slave and there are still stale addons in there which got packaged ... so i think its definite that installing binary addons into sourcedir/addons on osx/ios is broken for a while now and we didn't realise it because the dirs were not cleaned (because project/cmake didn't change i guess).


Consider changing type int to std::uint32_t


Merging.  Ok, whatever.


@wjps, that's awesome! How did you track it down?


I think you case you could just go with .forEach. Anyway if you are curious
Example: Breaking a loop  from https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Global_Objects/Array/forEach


> not canceling a reload isn't really the end of the world though. it's just wasteful of CPU.
Sure, still it wouldn't be too hard not wasting it here.

> I still don't want to merge the two as it means that the "update configuration" path can't take a fast-path
Yep, we could use a flag on the transition (arguably the same than with the two operations but with less duplication). That would still allow us to be a ta smarter in those cases.

Overall I find it's a shame not to be able to do this simple optimization and I'm still afraid of duplication in reload and reloadWithConfiguration, those state changes operations are usually the trickiest one and forgetting to update one or the other is something that I could definitely see happening.

Happy merge and keep the discussion going though, this shouldn't be blocking.


What trigger you to remove this? I know it is a very odd solution but unfortunately it is needed. We need a way to throttle the compiles of those heavy files on travis but still compile in parallel other files.


Derp, yeah this is very wrong anyway because it doesn't clear out the Vec as drain does.

—
Sent from Mailbox

On Mon, May 4, 2015 at 7:35 AM, bluss <notifications@github.com> wrote:

> A non-cloning replacement for .drain(..) is to swap the vector with Vec::new and return the .into_iter() of the old vector value.
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/tomjakubowski/ecs-rs/commit/52cfc692224e634d882e3feb5347109db4e2e84e#commitcomment-11026374


> There are bigger issues with the corrent generator's approach. Me or someone else will revamp it sooner or later.

That "somebody else" is definitely not, and in fact can't, help if you don't create issues about that. The only open issues I see are docs and feature additions :)


json body 是有格式的数据，我习惯把 body 解析到一个 struct 里，而这个 struct 的定义基本上就是接口协议了。解析的时候会有最基本的类型检查（更高级的可以参考 gin 的 binding 功能 https://github.com/gin-gonic/gin#model-binding-and-validation），而且后面使用起来不需要做类型断言，非常方便。


It's "plausibly often used" to "plausibly rarely used", based on my prejudice...


I believe this was fixed now... can you confirm?


Да нет, если уж так получилось, то ловить исключение -- правильно, но обычно считается, что лучше знать заранее, что умеют переданные в функцию объекты, а что нет.


Good point.  The `get_query()` should be omitted.

So I recently moved to having a `query_posts()` because of modeling an archive page where I needed it to be part `WPLib_Page` and part `WPLib_List`.

In the case of a method `get_posts()` there was a need to get the posts from the queried object if no previous query had been set. That meant I needed some way to say "Yes, I want to actually run a new query", hence `query_posts()`  _(but it could have been `get_query()`; depends on which one we decide on.)_

I then decided to change `get_posts()` in the post base class to `query_posts()` for consistency.  Get could go back to `get_posts()` and pass in `$args['requery']=>true` or stay with `query_posts()` or rename to `get_query()`, or other?

Discuss?


> I'm still afraid of duplication in reload and reloadWithConfiguration

But it's not reloadWithConfiguration. The semantic operation is *update* the configuration. The fact that this results in a reload (if the component provider or context changes) or a relayout (if only the size range changes) is an implementation detail you shouldn't think about.

Reload, on the other hand, is merely a concession to components that actually rely on global singleton state. That one is a hack. I don't want to compromise the purity of the former for the latter.

I do see your point, I just don't want to paint us in a corner where a sensible API (`updateConfiguration`) is compromised by a guarantee that "we will definitely regenerate all components if you call it, even if only the size changes".


> I think in practice, people will rarely mix sync and async changesets

Definitely not convinced of that but I fine with "We'll see" ;-)


Hmm or maybe another solution.

Leave brand name on plugins page, but translate in settings pages?
This may be satisfactory for redactors or part-administrators, but not sure about administrators who installs plugins. It may be useful for them only when link to "Settings" from plugins page.


type string, too.


Thanks for the bug fix Andrew!


Poor me that kept a link to the versioned file names ! ;)
Joke aside, it's much clearer to browse now


I actually added in-file comments but GitHub doesn't show the changed version because the hash changes when the file is modified... the new version should show up when checking out or viewing the whole branch. I found two other issues that I also fixed in the new version.


If a provider named as the tag is found, then that one is selected. Otherwise, it gets the first one (in alphabetical order).


lol, yes, it's safe and it runs like rocket now :D


Hey, just seen this. Is there a reason you switched to .tiff? Does it look better? The current one (.png) looks pixelated on my rMBP.


Adding issue target BroadleafCommerce/QA#537


I don't think MSVC supports constexpr, use "static const" instead


Adding issue target BroadleafCommerce/QA#538


Also add `0xC0`, `0xC1` and `0xFE`.


Hello!
This is my very first time in this Journey! I have background in Physics and Math. I am very interested in becoming a data scientist, even I have been involved with computational work in the past, I am very intimidated with this data world. Wish me good luck because now I am in-like my picture shows!!!


30411/Sporis A. Ioan Catalin/Assignment 6/graphRepresentation.c

careful, no need for a `;` here


@chrissprague Why did you put "RIP rit_object" in your commit message? I don't see any changes relating to rit_object.  Am I missing something?


Copy for upper


My bad. non_flags vector was initializet with size argc, while right value was (argc - 1), because of skip [0]. Fixed. I think it is good idea to check non_flags size to get rid of "args = " on output, when there aren't any non flag args.


30411/Sporis A. Ioan Catalin/Assignment 6/graphAlgorithms.c

as the algorithm needs not modify the adjacency matrix, you don't need a copy here


2b5fe19b50e1cfe7999ff7a89808911dc7b23484 shows the changed version...
6f49c205b3f60fb6a09b61066b283d9c91a76f7b shows more issues I fixed...


This should have been wrapped by `#if defined(__linux__) ... #endif`


> But it's not reloadWithConfiguration

My bad for the wrong wording, I definitely see the semantic difference but what we care about is that it is conveyed through the two public facing APIs which are `updateConfiguration:...` and `reloadWithMode:...`, I'd care more preventing duplication for the change operation which is in the end an implementation detail for those two APIs.

And we definitely don't have to compromise on doing only a relayout when possible, it's just one more parameter for the initialization of the state transition.


@dbo the ignorance is mine - I don't have much context on that part of the codebase. Is around here the place you had in mind? https://github.com/tmpvar/jsdom/blob/master/lib/jsdom.js#L214


Problem here, size can be one.

Can be solved by optimizing before the sort:
if (times.size () == 1)
  return times[0];


at the end, the Bellman Ford algorithm should print out whether or not we have a negative weight cycle. See third step from the code [here] (http://en.wikipedia.org/wiki/Bellman%E2%80%93Ford_algorithm#Algorithm)


Add a helper:
```java
assertCaseChangeWithInvalidBytes(concat(new byte[] {'F', 'O', 'O'}, invalidSequence));

private static void assertCaseChangeWithInvalidBytes(byte[] bytes)
{
    Slice slice = wrappedBuffer(bytes);
    assertEquals(toLowerCase(slice), slice);
    assertEquals(toUpperCase(slice), slice);
}


Backticks around `true`? :)


I did log in as the user "pi" and I don't think I used sudo for the checkout or chowned the files, I think I will leave this instruction as is with the sudo for now as I don't think I will have time to re-test before Tuesday. I will revisit it after the meeting.


@jeffcarp Yes, it's the options being set up by jsdom.env to be passed to the Window ctor. I'd suggest to add virtualConsole there.


Hi again! I was wondering what kind of changes you have done or have in mind. Lately many things in Suplemon have progressed and improved. One big new thing is custom key bindings. I'd love to incorporate your changes in Suplemon too. Let me know what kind of features you we're thinking about :+1:


Thanks very much for the PR, just what I was hoping for! One question, is there any implication of doing it in one transaction when there are 25,000+ creates happening at once? These are pretty big files in some cases..


Nice, then it should be deterministic...


Fixed in commit 2baa536a9754e1005f563c6a81abf0ba8e4660ad


/cc @rwaldron


Sorry, I meant to say: for batch update, shouldn't
`POST /projects/:project_id/response_fields/:response_field_id/batch` be `POST /projects/:project_id/response_fields/batch`? Why do we need a `:response_field_id` if we're updating more than one?


Хммм... Ну можно сказать, что основная цель этой функции - сравнивать объекты. Любые объекты. Только полиномы и функции надо сравнивать по особому. Вот.

Может быть и стоило бы ограничивать через что-то вроде if exp is [SomeClass]:
Только в SymPy много классов и мне что-то очень в лом искать у них у всех старшего предка, у которого впервые появляется _sorted_args. И это еще если мне повезет и они правильно наследование оформили.

Хотя...


Add a helper like `utf8ForCodePoint`


Use something other than "Secure". SessionCookieProtocolSetting perhaps?


We should probably check to see if there is at least one slash between
http://192.168.117.47:5000/v2.0 and tokens ?


Change to Http?


Change to Https?


@Aluxian yeah it seem to work properly! nice work


Ah thanks, fixed. I updated it in the example request, but not in the "HTTP Request" section of the docs.

:ok_hand:


Using string concat because was not able to figure how to use wild cards...


Don't commit stuff from `compilation_cache`. Add that folder to your `.gitignore`.


Most places seem to agree that very large transactions and super huge transactions (1 million inserts vs 20 million inserts) have the same performance. Since this app is meant to be run locally with a process step afterwards anyway, I doubt it'll be a problem. :)

http://stackoverflow.com/a/717134


Possible nitpickery, but: Wouldn't make `undefined` make more sense here?


What exactly do you need?

If you can annotate the classes yourself, then you can use @Instrument
@DoNotInstrument

The configuration has been changed to configuring by a builder, we could
also expose a Instrumentation.Builder.addInstrumentedPackage() method too
if that helps?

On 4 May 2015 at 01:48, Nico Küchler <notifications@github.com> wrote:

> Until now I could inherit this class and provide a custom shouldInstrument
> implementation. On this way I could register custom classes which should be
> shadowed.
>
> Is there another way to use shadows for own classes?
>
> For me it is necessary because of AndroidAnnotation. This framework spawns
> background threads a bit different than AsyncTasks. I get only stable tests
> when I provide a shadow which make the background threads synchron.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/robolectric/robolectric/commit/faecbc603dcfcbbcd724d38aa8b28ba9417969cf#commitcomment-11022088>
> .
>


Fixed in efae49d0e14ada0224e1e9eacb4d36cc6f1c7790


Hi,

I try changing these values below but the break point of my top-bar to tab-bar still the same or do i need to modify another file? Thanks...

+// Media query breakpoints
+$small-breakpoint:  em-calc(640);
+$medium-breakpoint: em-calc(1024);
+$topbar-breakpoint: em-calc(820);
+$large-breakpoint:  em-calc(1440);
+$xlarge-breakpoint: em-calc(1920);


```java
byte[] extraWhitespace = concat(whitespace, new byte[] {'\r', '\n', '\t', ' '}, whitespace);
...
assertEquals(
        trim(wrappedBuffer(concat(extraWhitespace, sequence, extraWhitespace))),
        wrappedBuffer(sequence));


At the moment `Genome::Env` still validates environment variables so I haven't removed it.  It will be removed in the second stage.  As far as the comment, it's my opinion that `Genome::Env` should document itself not `Genome` document `Genome::Env`.


30414/Stoica S. Razvan Cosmin/Assignment 6.1/graphAlgorithms.c

At the end of the algorithm, Bellman Ford should print out whether or not we have any negative weight cycles in the graph. Check out the third step from [here] (http://en.wikipedia.org/wiki/Bellman%E2%80%93Ford_algorithm#Algorithm)


Actually you're right, just looked up the Map.prototype.get() method and it returns `undefined`. Since the `Dictionary` class is designed to mimics most parts of the `Map` interface (*), it should behave the same.

(*) Compatibility allows one to completely switch over to `Map` once it is supported in all browsers targeted by us, which as of now is not the case (looking at you IE!).


30411/Danciu P. Diana Lacramioara/Assignment 6/Graphs/graphAlgorithms.c

you don't need this here. allocation is already done in the `getCopyOfAdjecencyMatrix()` function.
all you need is
```
matrix = getCopyOfAdjecencyMatrix();
```


typo.


`std::sort` does nothing if there is only one element...


Hi Kyle, is it possible to use a SCSS class here to accomplish the same result?


do we enforce lowercase when adding stuff to that index?


C++


i'd rename `sender` to `user` - that argument is always passed as a positional argument anyway


Sweet! One more step towards: https://github.com/clojure/clojurescript/wiki/Bootstrapping-the-Compiler


Right, but we are trying to index into a second element that doesn't exist.


misses the `npm install` step. I honestly don't think we should go this deep into contribution instructions anyways. This steps are true for every generator and I feel like we documented them already.


ha, a bit stupid explanation:
I had the matching png files named without capital. At my machine it didn't matter, but when I pushed, the UI couldnt find the png's. Then I renamed the png's (with capital), but the github app, didn't pick that up for some reason (it didnt see changes).


Not sure that it's clearer but imo looks cooler ).
Still cant't crack outer loop - can't find a right way to borrow self.neighbours twice...


@rafaelfranca: Thanks for the explanation - it sounds as if it's load time which is the problem, not execution time. What you describe sounds particularly relevant to the `railtie` tests which fork for every (?) test, but it looks as if `mocha` is being removed from elsewhere in `rails`. Do you know why that is?


:heart:


ping @crhym3 - any reason you kept this? It doesn't make sense this way IMHO.


That isn't true.  If there's only one element, we are taking the average of `times[0]` with itself - because integer division!


Works fine.. Merging.


From device how can I check its value (if possible)?


@ rec 1+1 = 2,   2/2 = 1, indexing into the second element is a bug.


It is on mine. SVN rev 5525
RobWork/src/rw/common/Ptr.hpp:141

	template <class S>
        Ptr<S> scast() const {
            if (_owned_ptr)
                return Ptr<S>(boost::static_pointer_cast<S>(_owned_ptr));
            else
                return Ptr<S>(static_cast<S*>(_ptr));
        }


@kotarou3 `battle.p1.active[0].maxhp - Math.floor(battle.p1.active[0].maxhp/10)` would be more accurate.


I already put the comment about locating the prodDb in a specific directory. I think putting it in tmp is not necessarily a nice place for a database to be


Sorry, haven't had a chance to circle back to this. I was actually doing some custom key binding stuff and some rearranging of the project but when i saw you were working on that I backed off. Now that that's merged I'll re-fork and send you a couple of PR's!


Yuck, good catch. Probably something with the fixed navbar stuff that caused this :/


Yes.


generic != general. This sentence deserves a proper rewording, something along these lines:

The registered functions are renamed by the toolbox, allowing generic algorithms that do not depend on operator names


Why do we recommend people to install from github instead of `npm install --global generator-mobile`?


I thought about it, but `sender` looked like the convention in your older code.


Wow... yes. Should read:
`return (times[(times.size()-1) / 2] + times[times.size() / 2]) / 2`
And if the container has only one element we must avoid this line, as you indicated originally. Good catch.


indentation error ?


>Change 78 - Merged
>Fix QS touch vibration.
>Fixes variable mixup with commit 26951cb9e90d8f1131cccc1edcb6ea61f77faaca.
>Change-Id: I36801bd868f9e6b6ead9116666bcbf25a52ec8d2

Are you shure that this is correct link?


fancy


I usually kept it when the sender was either None or unused


Why not keeping the option
if (options.closeByEscape && !keydownIsBound) {


Great catch @miguelportilla!


Installing
Note: This will not work on 64-bit systems without first properly signing the driver

Gather together the vmulti.sys/vmulti.inf/hidkmdf.sys files into single directory
Also copy the file WdfCoInstaller01009.dll (from the WDK) to your installation directory
Also copy the file devcon.exe (from the WDK) to your installation directory
Run the command: devcon install vmulti.inf djpnewton\vmulti

----------------------------------------------------------------------------------------

I don't konw when I follow here up steps, there is something wrong like:
http://blog.vucica.net/2010/10/adventures-in-driverland-pt-1.html#comments

liks this :
“echo” example:devcon install vmulti.inf "root\vmulti"
This is where I’m stuck:
Device node created. Install is complete when drivers are installed...
Updating drivers for root\vmulti from C:\projects\vmulti\sys\objfre_wxp_x86\i386
\vmulti.inf.
devcon failed.

----------------
but when I run this \buildme.bat, and then run \bin\install_driver.bat.  there is no errors  to install the vmulti drive.

-------------------------------------------------------------------------------------------


I forgot to add that I can't actually run this script at this time being as KBase is down for maintenance today.


Nitpick but that empty line shouldn't be there.


Its the Mac filesystem ;) https://www.google.ca/search?q=mac%20filesystem%20case%20sensitive

To change case easily on OSX, you have to rename to something else (ie, the first letter only) then rename again to the case changed version.


Typo - certaint = certain


Added DateTime picker for calendar controls.


@marvinsss2005 : Yes. In order for this to work as expected, you will have to modify both all the files in this pull request. Just look at the changes in the commit log on this page (scroll up). Please make sure you are running ```grunt watch``` when adding the new styling attributes. Or you could do a manual ```grunt build``` after you're done.


:heart: @Lukasa

/cc @rbcarson


usefull = useful


Yes i know. I wrote the test to fail and then we have to implement the logic.
Read this - https://en.wikipedia.org/wiki/Pawn_%28chess%29#Movement


Note that the algoflags are what th hardware on the card can do. OpenSC will attempt to use the hardwared these operations, and OpenSC if compiled with OpenSSL will expose PKCS#11 mechanisum that can do hashing and padding in software. So
algoflags = SC_ALGORITHM_RSA_RAW | SC_ALGORITHM_NEED_USAGE | SC_ALGORITHM_ONBOARD_KEY_GEN


sure, node is better


Changed name to CycleSoft


Sounds good.  I'm not picky.


In this case `secure` is actually appropriate. This option controls weather or not the text `secure` is emitted in the set-cookie header.


But actually this is not needed if you do the other tooltip stuff I described in the issue


Made changes and added the chart into a method, for convenience


fair point, saves a binding, i'll change this. thanks!


Added all series XValueMember to Date/Time for check boxes (So when checked axis stay same)


Added zoomable functions (Selectable data)


This line can be `if (_keyBuffer.Length > 0)`


Do we have API documentation of the `ctx` parameter?


certaint = certain


No . . I'm not sure what a good suggestion might be.   "temp"?   Maybe the failure is good?

Presumably (at least if they use this file), they'll change it anyway.

We could also comment out this line with a "PLEASE CHANGE" remove "url" from production.


I don't because he might extend it in the following yaml file... Lets wait for the feature to settle down and then we can work on the UI improvements :wink:


Why is there a missing test ? Did you forget to update?


This isn't the case in oldisim: https://github.com/cmccoy/PerfKitBenchmarker/blob/master/perfkitbenchmarker/benchmarks/oldisim_benchmark.py#L76


30411/Fericean G. George Vlad/Assignment6/graphAlgorithms.c

Bellman Ford does not return a MST!


Bellman Ford does not return a MST!


Thanks

Good idea

We don’t need to keep updating spring

Dan Williams, PMP, PMI-ACP, CSP

Cell: 425-8776333

שָׁלוֹם עֲלֵיכֶם

peace to you

From: Jean-Denis Vauguet [mailto:notifications@github.com]
Sent: Monday, May 04, 2015 6:46 AM
To: danielwilliams12/new_options
Subject: Re: [new_options] Updated associations between posts and users (03a9590)

You could add /spring to your .gitignore file.

—
Reply to this email directly or view it on GitHub <https://github.com/danielwilliams12/new_options/commit/03a95901405532aace03c7c183423ad5fb21fdda#commitcomment-11025471> .  <https://github.com/notifications/beacon/AGEhgzlefUruBCSoQMtm-HfkUQ0mtLhxks5oF2-hgaJpZM4EPVvu.gif>


A wee bit 'unclean' but "it checks out"


>follwing

Typo.


llllllllll


It makes no sense to create issues for functionality that will not exist.

On 4 May 2015 at 18:07, Arthur Verschaeve <notifications@github.com> wrote:

> There are bigger issues with the corrent generator's approach. Me or
> someone else will revamp it sooner or later.
>
> That "somebody else" is definitely not, and in fact can't, help if you
> don't create issues about that. The only open issues I see are docs and
> feature additions :)
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/yeoman/generator-mobile/commit/84fc34a299fb5793553ecda92c5f1e16948da5ff#commitcomment-11028242>
> .
>


@acroos with the merge to dev the MainInstitution is still 'AAA Institution' let me know if you think we should change that back or anything.


da ti ohuel


I like newlines at EOF.


Ah never mind... works since GetInfo returns the right thing - in fact I'm not sure if GetInfo ever returns None for num_machines


ya I think having a "<CHANGEME>" would be good, along with the comment above to indicate the file path

i did get the same errors testing on ubuntu a lot (permission denied /var/lib/tomcat7/prodDb...)


Thanks!


super!


lol


Looks good.

Would you be willing to make a new branch and make a new changeset with these changes? The current back-and-forth changes makes it hard to audit and it is a pretty big change. I am ready to pull when you do.

If you split fixes into separate commits that would be great too (I would have cherry-picked those already if you had). Such as the deflate.c unsigned int fix.


Right, I'm just thinking about ease of future maintenance. Such guidelines are difficult when there are many contributors. Anyway, I don't mind either way.


You can alread get this via `config.nested`.  example: https://github.com/blakechambers/matterhorn/blob/a9bd2fc485e63d9b03a37f28bd61892d3db597dd/lib/matterhorn/links/relation/base.rb#L45


Lucas, you are on the vacation, turn off your computer, __now__ :wink:


In that case is just to avoid one dependence. But there is also a mindset problem. If the library is there people will try to use. If it is not people will try to find other way without mocking.


Esse bloco poderia ser um método, ficaria mais fácil para tratar a exception e ficaria mais limpo na minha opinião. Vou mexer nele depois.


Well, one is enough, don't need two.


I like this change! Much cleaner looking


hmm thx ! I'll try if i can fix that !


Sure, should be pretty straightforward. I'll do it tomorrow.


I think you should add "/* falls through */" to this one. At first glance, I thought it was a mistake


hope it is a correct version


here's your problem, nasty little one.
You wrote: `minE.weight < adjMatrix[i][j]`. But you assigned `minE.weight=MAX`. Therefore, it will never get through this condition.

Simple fix:

`minE.weight > adjMatrix[i][j]`

Do tell if this fixes your problem


But it doen't accept the crappy utf8 characters in domain name. I think
it's ok.
Le 4 mai 2015 18:36, "Jeremy Grossmann" <notifications@github.com> a écrit :

> [image: :+1:]
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/GNS3/gns3-gui/commit/a6653a3419d9f56825b132306ee5888481ca11d1#commitcomment-11028657>
> .
>


Sounds good to me :)


This might've been added before this generator was published to npm. Should definitely change it now though.


I tested this. At least I tried as there was a lot of graphical glitches (polys taking half the screen, flashing square around  that quickly affected up to the chili menus. I didn't go through you code but I saw your post on the forums about being confused with some oGL functions and this one need some love. That didn't helped me to seriously check the promising features it have.
About features there is something that really bother me since xponen introduced it in his widget: constructors splitting. I resume that feature as 3 letters: W T F. I prefer to have 3 missiles towers built and working than 6 half built that will die (and potentially with the builders) ! I feel that is totally against the way it should ! At least if it's useful to someone, but I doubt that, it should be offered to disable that bad killjoy feature.
Beside that I love that you comment everything in your code. Good habit. Very good one!


I already have the deflate.c fix on a separate branch already so I can make pull request from it... I need to cleanup this branch anyways as the recent changes introduced a merge conflict...


I have renamed the option from nested to inclusion to avoid confusion.


I don't know how rigorous you want o be with RAML. At this point you might want to describe the body associated with a 201 response. I am fine either way. I am thinking, though, in future readers, who might not find this so obvious


@jlucerond great changes. well done.


```
if (cfg.core && cfg.core.account) {
  callback(PASSED, {what: 'gcloud', data: cfg});
  return;
}

var errNoConf = new Error('gcloud: not configured');
callback(FAILED, {what: 'gcloud', data: cfg, error: errNoConf});
```


Note this fix makes arrow installable with Python3 if the environment variable LC_ALL is not set, since README.rst contains UTF-8 chars.


@kevva please create a PR if you want to make it better.


30414/Bologa M. Marius - Vasile/Assignment6/graphAlgorithms.c

You wrote the `uni` and `getParent` functions. Why are they not used?
Integrate them in your code. Currently, there is a lot of overhead in your approach. It could be written more eimply


@rlouapre I think you are correct. Thanks for catching this. All we want to avoid is creating a duplicate database, but the code above  is overly aggressive. We still need to assign values to the *-mapping vars.


there's only one empty line.


`` js
if (!data['user.name'] || !data['user.email']) {
  var errNoConf = new Error('git: not configured');
  callback(FAILED, {what: 'git', data: data, error: errNoConf});
  return;
}

callback(PASSED, {what: 'git', data: data});
```


Absolutely...


a newline isn't a blank line. This has two newlines.


Moved onclick handling to separate javascript file, and only changing button state on success. See a575a63 and f506bf4


hauria de ser data


Remove these if they aren't used.


30411/Ghemes M. Horatiu - Mihai/Assig 6.1/graphAlgorithms.c

cute for debugging


@dbo oops my bad - just realized I linked to the same place as your link. Maybe I seriously need some coffee. Will work on a follow-up PR. Thanks!


CheckBox enable disable for Heart Rate


I don't think this is right. SDL_SetVideoMode() has a depth parameter which should determine the bpp. A depth 8 mode without SDL_HWPALETTE is still an 8 bpp mode with a palette. In desktop SDL it just might have to share the system palette with other apps. The SDL_HWPALETTE flag is only telling SDL to install its own palette so it gets all the requested colours and has the ability to do palette effects like fades or colour cycling.


yes, good but you still have to print out whether or not you have a negative weight cycle in your graph. Check out the third step from [here] (http://en.wikipedia.org/wiki/Bellman%E2%80%93Ford_algorithm#Algorithm)


Check Box enable disable Speed


Change to the name


Created datatable columns for Graph use.


ANode*


send_yasp_ack does the same things that are done in this test helper.  If this used send_yasp_ack instead, we would have more test coverage and less test code.  Create a stub for "serial_tx" which saves the TX data to a variable which can be read from the test cases.  It will also eliminate the need for hard coding SYNCH bytes and length in test cases.  In this way, YASP will be used to generate data and verify itself.


and it seems like you're doing the same thing in many other files in this generator :/


Added rows of Data from GridView for use of Chart.


30411/Jucan I. Diana/Assignment 6/graphAlgorithms.c

same observation as @ghemesm


Created Series for Graph below.


There is a confusion about the meaning of 8. This is a number of neighbours of cell on 2d grid. 3*3-1=8. So we reserve initial capacity of neighbours equal to viewport size * number of max possible neighbours. I think this should be reverted. I think my conclusions about the initial neighbours size is overvalued but I was lazy to calculate it more precisely.


Create chart from Data.


it works for me , thanks !


Hi,

Lol, i'm using css and style using style.css, i cant get scss to work really confusing. I tried integrating it with netbeans but when i compile there are errors.

Thanks...


Added Telerik controls for opening HRM


yes it's "safe" running like hell now !
I don't think that u can check this.. check your defconfig ... if u see: HZ=100 .. the it's 100 not 1000 ;-)
btw.. theoretically 10 x times faster response ! xD


Is the 2nd line in this a typo? Is it missing shouldBeCalled?


Good point, actually that may be a problem with remote server that have Chinese characters. Try 义勇军进行曲 for instance.

I think regexp does support Unicode, you have to use re.UNICODE if I remember correctly.


доделать связи между таблицами


80 is not so much nowadays with full HD (just a general remark)


Same as above.


What's up with this indentation? It's 3 spaces but it isn't really alignment as well.


30411/Drimbarean I. Maria Cristina/assignment6/graphAlgorithms.c

Your Bellman Ford should print out whether or not there is a negative weight cycle in your graph. Check out the third step from [here](http://en.wikipedia.org/wiki/Bellman%E2%80%93Ford_algorithm#Algorithm)


Lets not leave it as-is. This will crash with an unhandled exception when findSet returns a nullptr on a hash not found.

Fix:
auto j = findset (hash);
if (j != nullptr) {...}


@gdachs is this still needed?

if so. linux patch must be rebased. and after libcec 3.0 bump (should happen next few days) - libcec patch should be fixed.too.

if not, I will remove this branch tomorrow,


Good point.  That is an old comment pulled over and I don't remember why I called it Internal.  I will remove the "Internal" designation because I don't currently think it should be internal.


This was to fix #10


I tried to merge these two files into my existing 1.2.7 install and I'm getting an error

Class Controllers\Admin\AssetsController does not exist

Am I missing anything obvious?

Apologies - I'm not a dev and I don't pretend to be - I'd just really like to get this feature working before 1.3 is out.

Thanks!


Update to Version 4.2


It is actually not possible to move from a deleted section or to an inserted section, don't know why apple put those restrictions in place. It will work in this layer and crash later though.

This is a very specific case that actually breaks the general rule that a move could be modeled as an insert and a delete.

I otherwise double checked the logic and it looks good, this could use a little more test for the base cases though.


I find this confusing. Why do we need a body here?


Yup, I'm on it.


Well done! :)


I understand the concern -- keep in mind that unless we can figure out a 'perfect' solution then we need to choose a lesser of evils.  I am happy to keep working on finding a perfect solution but in the meantime I think it is a lesser evil to disclose some stealth info as opposed to totally hiding the attacker from their target.  I'll note below another imperfect-but-very-good solution, another imperfect-but-better-than-nothing solution, as well as my one idea for pursuing a 'perfect' solution.

As I noted in my intro comment to the PR, we do need some kind of additional change because without this or some other solution, the server will (at least in many cases) designate this info as stale-- the info is deemed to be from the previous turn, and certainly if the target empire still has the system in view the server decides that since they can no longer detect the stealthed attacker the info about it must be stale (and it might even be that the info was always getting designated as stale if the target empire even simply retained Basic visibility of the system).  Being stale means that the human client MapWnd suppresses display of the fleet and ship and makes the entire handling essentially worthless (for human players at least).  As I noted, it might be possible to change the stale info determination, this would be a potentially 'perfect' solution, but I looked at it for a bit and a modification to the algorithm did not come to mind.  Suggestions are welcome.

One very easy alternate solution that would have limitations from both player perspective and yours, but might be a better compromise, is in this case of basic visibility, rather than giving the recipient empire the actual stealth, tell them the stealth is just one point higher than their current empire detection.  They would know that this just really means it is at least that high, and that is not new info to them-- it is a simple deduction from the fact that the object was stealthed.  But having this recorded in the stealth meter saves the info from being designated as stale by the stale-info algorithm.   I think this still has a minor weakness in that once the empire's detection strength advances then the object would still get caught in the stale info determination and then its display would get surpressed.  But that would normally take a while and is still much much better than the current situation of never getting any display of the attacker at all.

An alternate solution might possibly be to change the timing of at least this information update relative to the increase in the turn counter; that way the display on the MapWnd would not be suppressed for at least one turn.   That seems rather on the lame side, but again is far far better than the current situation of never getting the display at all.


30414/Ghiurau A. Alexandra Cristina/Assignment 6.2/graphAlgorithms.c

so which edge are you choosing here?


Ah, I see. If I were you, I'd make myself a pot of coffee and set aside 1 hour on reading the basics around sass and grunt. It will seem confusing at first, but it will save you lots of time ahead. Everything you need to get started should be described in the readme file of this repository.

If you plan to only deal with css and not to reap the benefits of grunt and sass, there are probably better themes for you than this, to be honest.

Anyway, good luck! :)


Very similar functionality still [exists from before in SDL_UnlockSurface()](https://github.com/kripken/emscripten/blob/aad005f54a21a91b64bc07fe7ede67bd79f4f2d2/src/library_sdl.js#L698). The main functional difference is this new code has a pitch equal to width (rows follow one another in memory) and that old code has pitch equal to 4 * width (meaning consecutive bytes used for pixels, and then 3 * width unused bytes before the next row). This new code is also better optimized.

In effect, after this there are two different 8 bpp modes.  The only purpose I see for the old code is for supporting 8 bpp modes without SDL_HWPALETTE, but the proper solution for that seems to be making use of the depth parameter from SDL_SetVideoMode() and using this new code for all depth 8 modes. This is related to #3400.


Comment updated.


@ryao do you happen to know if illumos has an equivalent?  If so then adding a wrapper might be better.  As always we can revisit this if/when it becomes and issue.


Ya rock @kevva! I was planning to do it but feel free (:


I really know the words:

Building
Start a WDK build environment
Navigate to the location of your vmulti source root directory (eg. C:\projects\vmulti)
Build the drivers and test program using build -wgc

eg. C:\projects\vmulti  ----->>  must be "Project root directory"
"Project root directory"
"Project root directory"


@alexpekurovsky You're commenting on a commit (a3997774a999cbad049400fee6ccbc977ceac386) from over four and a half years ago. This was version 0.11, the current cookbook is v3.4.18 - you can review the changelog [here](https://github.com/hw-cookbooks/postgresql/blob/develop/CHANGELOG.md).

The postgresql.conf is [rendered from attributes](https://github.com/hw-cookbooks/postgresql/blob/develop/templates/default/postgresql.conf.erb) now, as is [pg_hba.conf](https://github.com/hw-cookbooks/postgresql/blob/develop/templates/default/pg_hba.conf.erb). Much of the hardcoding is gone, and attributes are used to drive configuration. The README should have more information about this.


why not strict comparison?


>  `/*jshint camelcase:false */`

Just use [camelcase-keys](https://github.com/sindresorhus/camelcase-keys)


Gather together the vmulti.sys/vmulti.inf/hidkmdf.sys files into single directory

the directory must be in project , and must be the sub directory(first layer) of project.


Sure.  I'd like to keep `is_published()` but we can add an `is_status()` too.

Might also want these explicit methods?

- `is_draft()`
- `is_future()`
- `is_pending()`
- `is_private()`
- `is_trashed()`
- `is_auto_draft()`
- `is_inherited()`


Unbelievable, it works. I could not do it (((


I think this should be `is_a? Array`


I assume you are referring to `if (err == null) {`?  I dunno, it's an easy fix though.


Better to just remove `camelcase` from the `.jshintrc`.


TeamCity MonoGame :: Develop (Win) [Build 3.5.0.64](http://teamcity.monogame.net/viewLog.html?buildId=6875&buildTypeId=MonoGame_DevelopWin&guest=1) is now running


@hastalafiesta- , build your kernel with
`CONFIG_IKCONFIG=m
CONFIG_IKCONFIG_PROC=y

Then you can check it via
`insmod /system/lib/modules/configs.ko
zcat /proc/config.gz | grep HZ`


My mistake, I was using WidgetInput for testing ellipsis functionality. I've fixed it now.


> Better to just remove camelcase from the .jshintrc.

Well, that's an option as well though I'd prefer just not use non-camelcased things. Consistency ya know. Fine with eighter way, though.


`name` and `singular_name` need to be set for each post type, or the latter needs to be derived from the former.  If I've missed `menu_name`, `name_admin_bar` and `add_new` can you add?


기본 테마를 Theme.Holo.Light.NoActionBar로 정함.


이건 나중에 지워도될듯 ㅈㅅ


500에러 등을 잡아냄


Love it, MUCH MUCH needed indeed. It looks and runs great!!! Settings menu was a nice addition as well as the fantastic new look


Extensive enough!


This commit seems to have been made in error. It reverts the pull request accepted in #4. Also, the commit message indicates a merge, but the commit only has one parent.


space after ;


/rubocop nitpick mode:
* you don't need to do string interpolation, because `join` will return a `String` already
* prefer single quotes for string literals, so it should be `join(' ')`

Recommended:
```ruby
sql_sys_admin_list = node['sql_server']['sysadmins'].join(' ')
```


Fair enough... You have no comments regarding the database version?


yea, but its my habit to use strict comparison always by default, lol


Yes on mine as well, but the result is really wrong in the installer!


@ZeroChaos-  impacket fix for the suave template I'll give him some credit


I found the way to calculate it more precisely. Working on new PR.


You will probably need this line from this commit.


> btw: that means actually nobody can build in dualcore @ thumb 2 ??

I guess, only our devices (u8500) unable to :( I have feeling that exactly this commit sucks, causing CPU1 never awake :(


I'd love to help with testing too. Are there any binaries available?


Må også takke høyst meget for rosen =)


Everyone needs to agree if we are going to use UTF-8 or ISO 8859-1 for the file encoding.
Is the locale for the editor not being set correctly?


Hvis du aksepterer pull-requesten så kan du legge til det du måtte ønske ang. Twython. Akkurat nå ser det ikke ut til å være i bruk? Den oppretter tw, men bruker den ikke til noe.


> It makes no sense to create issues for functionality that will not exist.

Well you said that there were "bigger issues". I don't see any open issues about "bigger issues", so it makes sense to create those.


@rec I assume you're joking to reinforce Vinnie's point. Failing to call `update` would break the logic of this code.


I'm really not a big debian person (I use ubuntu in some cases but generally I prefer arch), to that end this may seem like a silly question.

I notice the bitcoin debian configs have a lot of things that are not present in these: https://github.com/bitcoin/bitcoin/tree/master/contrib/debian

Just wondering if we should be looking to implement these other things (I'd think yes, but again I'm uncertain).


This alpha setting loop was previously inside `if (!surfData.image) {`, so it should have only run once when the image is created. Now it runs every time when copying image data from the canvas. This is a significant slowdown.

The only reason for this loop seems to be setting alpha for 8 bpp code. Copying from canvas to 8 bpp isn't implemented anyways, and the loop should only need to run once to initially set alpha. In 32 bpp, [SDL_UnlockSurface() sets alpha](https://github.com/kripken/emscripten/blob/05a26d92ef6da845cfcbff79c5fc7a2de56263bf/src/library_sdl.js#L803).

Related to #3400


After more contemplation I think we should drop `query_posts()` and go with `get_query()`; I'll do that but we can revert if you think otherwise.


I forgot to mention in the commit message that I removed references to the static Console class in the MOS6502 unit-tests.

We should not have to rely on a separate class in the unit-tests. Whether we pass a Console reference or use shared static Console class, the unit-tests should only test the CPU and its methods.

That's why I still have the following code on MOS6502.powerOn()
```
 if (Console.ram is null)
 {
	Console.ram = new RAM;
 }
````

If a Console was not intiialized prior, we can still unit-test the CPU using a freshly-initialized RAM.


Commit 691390d5fe23ff4a05228ca394056e17f6584c28 contains the remaining fixes... There is too many overlapping edits to separate it safely to more than one commit.


Oi @lucassch !

Seria legal que você tivesse incluído o SQL para desfazer a alteração também.


lol i'm not an expert xD .. but maybe we can "fix" this in a much easier way .. just hope for this ... if i found any way to fix this, i'll tell u immediately ! ..


> lol i'm not an expert xD

yep, that was just an irony. :)


background-color: rgba($primary, .05);


@dgraham Cool! For those of us that haven't kept up as well, can you explain the rationale behind using `const` instead of `var`?  When I think of using constants, I don't think of replacing instances of `var` in a function scope like `filter`.


this is because of a bug that I fixed in the later commit. currently we create new zones for all callbacks. even if no custom zone was installed.


Remove this property and add `@extend .round;`


En la función myReversa estás diciendo que recibís un parámetro *array_a_trabajar*. Necesitás entonces la declaración de la [línea 1](https://github.com/lucasverra/JS_TP_reversefunction/commit/7870b68a4831327bf918ffa884c4b45d15ffbf91#diff-ac7c8ae54e64585f834ae1fa6521897cR1)?
Por otro lado, recordemos que tu función solo puede tener control sobre el contexto que contiene. Es decir que todas las variables que utilices deberían estar dentro de su contexto.


By the way, if you need faster compilation, talk with monstro78 about build server.


Hi,

In netbeans if i compile style.scss then it will be minified to style.css. So in FoundationPress it is minified in foundation.css right? For instance if i change the background color of my top-bar then what file i need to modify in scss mode?

Thanks...


Why so complicated ?! He just need to open defconfig and search for CONFIG_HZ=


Rename to `.list-text-primary`


Rename to `.list-text-secondary`


#5709

Ping @vieux @crosbymichael


No, and am fixing _(and also changing name of method to `get_query()`)_


why?

we don't concatinate this file into anything else.

I'd prefer to keep it as is.


remove it. Currently I have not enough time for the necessary kernel patch. I will redo it, if it will change.


TeamCity MonoGame :: Develop (Win) [Build 3.5.0.64](http://teamcity.monogame.net/viewLog.html?buildId=6875&buildTypeId=MonoGame_DevelopWin&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 485, ignored: 6 Build time: 00:16:37


`drivers` variable is not used.


This does not work on a Windows machine.

I tried to change it to:

"build": "SET TARGET=build && webpack --config config/index.js",

However, this does not work either.

Can you please at least let us know how this will work on Windows?

Thanks!

P.S. Same applies to "dev" below


I considered it but then kept it as is for readability purposes. we are not trying to save bytes here.


Thanks a lot!


@JoelKatz There's nothing funny about this or Tom's comment. Is it the intention of `update` to be called with a zero sequence to mean "call `update` without setting `mSeq`?"


color: rgba($secondary, .8);


*Now, this `each` function is very generic.* Not *this is `each`*


Was close to change one of these into:
```sql
(200000049x,'Grz3s! Xfurry! X-savior and UDB Team!',0,0,8,15,NULL);
```
... and maybe i'll change it :D


color: rgba($secondary, .4);


font-size: 1.2rem


these travis errors point to sbx queue problems


The link points to VSO instead of GitHub. I assume you meant https://github.com/Microsoft/PartsUnlimitedMRP..?


Declaring variables in a `for loop` gives false information that there is scope in a `for loop`. I would declare `i` outside of the `for loop`


done.

> Am 04.05.2015 um 16:40 schrieb Matteo Beccati <notifications@github.com>:
>
> Thanks @hwde <https://github.com/hwde>. Could you please also fix all the references to it?
>
> —
> Reply to this email directly or view it on GitHub <https://github.com/hwde/revive-adserver/commit/1db084b2fd67a66bd036d168ab96719d41d69f57#commitcomment-11026463>.
>


This commit is outdated. Look at the latest version.
https://github.com/demonh3x/hof/commit/ad1f6f02576fdc8f90ac1dd4ffce19139c4f18e7


This add a couple pretty non-standard practices and ideas, but I think it will work.

1. Dynamically hiding options that are irrelevant. (I guess this is probably done with options like Autio Input->Transmit). But, at least if they're hidden that room could be occupied by other future attenuation settings for other drivers, perhaps.
2. Since this is a PulseAudio-only patch (for now, at least for the same-sink attenuation) the options test explicitly for PulseAudio before they'll be visible.


missing EOL


hehe ok :-)  hmm thx it's good to know.. if need this, i'll ask him .. actually building time from scratch is acceptable for me.. i can build codina kernel in 22 Minutes .. and Lubuntu Kernel 25 - 30 minutes.. (don't remember exactly since i don't build it very often)


I edited my message, so not every parts were mailed to you. I'll just paste the full message again:

---

You could add `/spring` to your .gitignore file.

Edit: oh, it's already in the .gitignore. Because the spring directory was already tracked though, you would have to tell git to stop tracking it:

```
git rm -r --cached spring
```


WINEGROWER, DROP, BASKET, NORTH_WEST = 11, 163, 3, -4
WINEGROWER, TAKE, WINE, NORTH_WEST = 11, 167, 0, 4

super, That's how I imagined inversion, reverse numbering, and the name of the action for the convenience of code does not affect the inversion, as such, am I right?


I somehow missed this and the ones there are for posts, not terms.  I've fixed.


Just added a case for `val.Cap() > length > val.Len()` in e1eb23e18d2f6e34def8a21cc3f0288410032acb


Why not upgrade to 1.3.*?


`s/those/these`


https://github.com/Dead2/zlib-ng/compare/develop...mtl1979:test1 shows the clean version of what I've changed... You can also see the clean diff from the original pull request page...


ok - np, just maybe give this number a name then?


Goddammit


Dunno, but I uncommented.


By the way, using ccache reduces compilation time several times. I'm not sure, is that build server so fast, or it is because of ccache, but I've build latest build in about 5 minutes ;)


What about:
```
sed -E -e 's!^(\s*-rabbit\s+error_logger)\s+\S*!\1 tty!' -e 's!^(\s*-rabbit\s+sasl_error_logger)\s+\S*!\1 tty!'
```


Yeah, I’ll rework it ground up.
> On 4 05, 2015, at 20:34 , Piljavsky Sergey <notifications@github.com> wrote:
>
> ok - np, just maybe give this number a name then?
>
> —
> Reply to this email directly or view it on GitHub <https://github.com/pillserg/golr/commit/db4d80a625b9700f7cd358744fcbd1cd7163fef4#commitcomment-11029591>.
>


Hum, interesting observation. I think is just a matter of convention. Is this exactly what you mean?:
```
var i = 0;
for (; i < people.length; i++) {
  ...
}
```


Of-course it works - i wrote it, it may work wrong, it may be ugly - but it'll work - that's my credo :) you should remember )


looks OK, merging


Limitation of the neutron stack.


I'd remove the word "off-color" and use something more verifiable like "sexual".


rev1 of readme


lanjutkan


Согласен, просто когда писал, не было под рукой примера. Может у тебя есть что-то свежее?


Good idea.


No rather:

```
var i;
for(i = 0; i < people.length; i++) {
  ...
}
```


Oh wow, that's very generous, thank you!!


Perhaps it should've been put above the preg_quote functions.. I made it in a hurry :)


@stoeffel shipped to npm


:rocket:


I like the idea. Thanks! :+1:


Not sure if the explicit arguments in the return function as well as the `bind` function (e.g., `argument`, `argument2`, `argumentN`) make sense. They're not being used and a good JavaScript programmer should understand your use of `arguments`


Did you intend to commit this file? Normally the files in the .vs directory aren't supposed to be in source control.


The body would look exactly like the `GET /ip_addresses/{ipAddressId}` body. I had figured it would be redundant, but that sounds like a wrong assumption on my part. Will fix in the new PR.


rax retry tests


> this is because of a bug that I fixed in the later commit.

I'm only saying that without your later fix, this commit is buggy. That's why I think your should squash the commits (or change their order)


And may be add a test from my first comment above.


Actually today, we can just rely on the mtime of `/ostree` changing.


For portability, we should specify an encoding here (```UTF-8```)


How do they conform to the SOLID principles?


Could you please share the run_extras.sh shell script?


Is there any reason to keep these outside the class here?


He asked, how to do so from the phone, so answer is above.


Can you tell me what site that was that you made the Connect 4 screenshot? The natural improvement for that one is to quickly play out the wrong answers. In every wrong answer, blue wins in either 1 or 2 moves.


What are your thoughts of turning this into a plugin? I'd like to keep using the official plugin so I can stay up to date with any other changes.


此function裡頭的刻意縮排是為了方便除錯使用


Using a unsigned constant and then converting the & expression back to signed doesn't seem useful.


@anatol we now add have the release tags for the versions. https://github.com/facebook/proxygen/tags


If you depend on the reader of the code being a good JavaScript programmer, this change is redundant. The idea with this change was to make the behaviour more obvious for everyone, not only good JavaScript programmers. But I think the audience for this is more closer to good JavaScript programmers than anyone. I'll consider reverting it.


I think we should have an "img/" directory in the dist folder. If we get many images, we'll want them in their own folder


It's a Windows thing then.


https://github.com/audahadi/proprietary_vendor_xiaomi/commit/3d5c8b8d2a2a2bfaf77c01f2f9926c86a5915c31

missing /bin file


Yes, I already added it


Maybe the option can expect an Int used as the max-len?


this should be `['img/**/*']` so we can have sub-folders in "img/" and it will copy those as well


Curtis, I couldn't pull this one in whole, but I did make the change.  Thanks.


RPM conditionals are crazy, but I think if I'm reading this correctly you're only including device-mapper-libs if the fedora macro is set to >= 21.  I think you want this to also work with centos/rhel 7 too though.

Also, do you need to pull in device-mapper-event-libs as well?  There was a problem with that library not getting pulled in.


Make `dired` use `gls` on OS X, if found (maybe installed with `brew`).

If not available, make sure `ls` is called with legal options only, so to avoid this error:

```emacs-lisp
ls does not support --dired; see `dired-use-ls-dired' for more details.
```


I'm curious - is this needed?


TeamCity xBIM :: XbimExchange :: Build [Build 281](http://ncl-xbim.hq.4projects.com:8080/viewLog.html?buildId=1586&buildTypeId=XBIM_XbimExchange_Build) is now running


If you have any questions or problems do not hesitate to contact me! :)


should switch to 0123 to make sure it's at the top of the list


Did you want to include libsepol and libselinux as well?  Can we run without those on Fedora/RHEL?


@avo-odoo and @tde-banana-odoo the commit message says:
"inbox: removed 'compose a new messages or write to my followers', because those 2 options are
already available."
I don't think the features already available, this is not happens until 90cb409be933bef5b5a7788980818927598ba8e1 yet, this is the latest commit for branch 8.0 today.
The Issue #2042, shows many users not understand what happens with "send a message" and me too.
Commit 5209fbc7ed9f shows user don't know how to send a message so far without modifications or git revert.
I have seen this feature available on branches saas-6 and master.

Please, can you possible show any direction here how to be aligned with odoo developers?
Thanks


The check for the empty regex array is now done before the compilation of the expression.
To avoid a double join, I stored the joined string into a variable.

Is this what you intended?


TeamCity xBIM :: XbimExchange :: Build [Build 281](http://ncl-xbim.hq.4projects.com:8080/viewLog.html?buildId=1586&buildTypeId=XBIM_XbimExchange_Build) outcome was **FAILURE**
Summary: Exit code 1 Build time: 00:02:07


unless is a one off.


Could you please move this back so the diff is clean and this patch can be reviewed?

If we are going to move code around I'd rather have it happen in a separate PR (or better yet not at all unless there's a very good reason).


Deleted


I hate Result<Cursor> (i.e. Cursor is already a Result all by itself), but we should also rip out Deferred and Result and find something better IMO (i.e. even the author in his talk about them winds up saying "This code doesn't really look any better."). That's another big change, with no real win for users (except we could have some sane deferred error handling...)


how? did you read the rest after the line break? It reads:
Used in over 40 countries, CommCare is the most widely adopted, technically advanced, and evidence-based mobile platform for low-resource settings.


Added top comment regarding software.


Comments


Let me retract my previous statement and say that any good programmer should understand your use of `arguments`. I think it's better to provide code that you would write rather than code you wouldn't normally write.


Providing an example of the publisher Id might be useful


`'leechseed'`

You have an extra 'h'


I guarantee you the games do the same thing, just take a look at Pokécheck's implementation.


Yeah, this is required for VS experience. By default shared proj doesn't have a Configuration. Unless we add config values it can't pass Conditional required at CordovaApp.projitems.


Yes u're right.. and right now i saw that .. i has already commit that fix ... but anyway i don't know for what it should that good to check on the Phone if it's running at hz=1000 .. the difference i quite big.. just feel it ;-)


develop (totalgee) - x86_64-Linux-Ubuntu-14.04-clang-3.5: OK (346 of 346 tests passed)

![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)


"y" and "lim" need to be declared Uint64 as well


thanks for the doc!


You don't really gain much with this function pulled out here (and it makes this harder to read).


Good catch.  Fixed.


@audahadi I just seen the commit that the file was there but missing in the copy. Is this even needed for the device? If not I will just drop this commit altogether.


Extracting the char cast  from the ?: (`(char)((value < 10) ? ('0' + value) : ('A' + (value - 10)))`) generates shorter code.


Último despliegue, funciona bien :)


OMG !! Amazing speed :-D

I've also tried to add ccache into makefile .. but i got errors .... what's wrong ?


Perhaps move "Keep in mind..." to the end of the first paragraph because that sentence applies generally.


develop (totalgee) - x86_64-Linux-Ubuntu-14.04-clang-3.5-LibC++: OK (346 of 346 tests passed)

![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)


the installing to src_tree/addons should be handled by the special darwin CMAKE_INSTALL_DIR. If the double definition is the problem, its rather easy to fix. Just defined another CMAKE var without the "hardcoded" toolchain and CMAKE_INSTALL_DIR in Makefile.include.in and use that in the xbmc-addons-include instead.


Each function does only one thing and it does it well (SRP).
Each function is valid in a very wide context, so you don't need to modify them, you can combine them with others to extend their functionality (OCP).
You can swap their implementations and the clients will not notice. For example: each (sequential, parallel), map and filter (eager, lazy). (LSP).
They are the most segregated interface possible; only one function. (ISP).
They receive another function that will deal with low-level details. The receiving function only depends on the high-level policy; the signature of the received function. (DIP).


Yep, also a good change.


Isn't this required anymore?


심플에리어 현재 문제
- [ ] 화이트월드 프로텍트 풀어도 안풀리는 문제
- [ ] /영역생성 기능을 각월드별로 허용하는 기능(sa setarea)
- [ ] 영역구매했는데 풀리는 문제
- [ ] 임대주를 위한 임대보호 명령어 제작
- [ ] 영역출입금지 명령어 추가(영역정보조회에서 확인시키기)
- [ ] 자기영역바깥으로 물이 안나가게 처리


Since you like really long method names, you might name this clearAllHistory to make it clearer what it does.


nice one :+1:


develop (totalgee) - i386-Windows-8.1-VisualStudio-14: OK (346 of 346 tests passed)

![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)


Moved it back, sorry about that.

I moved it there because it felt very weird to have this method before the `updateConstraints` one (which is where the `_adjustableConstraints` dict is populated).


TeamCity DPCFLib :: BuildAndTest [Build 8](http://46.101.150.79:8111/viewLog.html?buildId=18&buildTypeId=DPCFLib_BuildAndTest&guest=1) is now running


Same strange signed - unsigned dance as earlier.


I agree that it feels forced and that it is not the way I would normally write it. :+1:


Was passiert, wenn du n=20 wählst? Wäre  interessant zu wissen


TeamCity DPCFLib :: BuildAndTest [Build 8](http://46.101.150.79:8111/viewLog.html?buildId=18&buildTypeId=DPCFLib_BuildAndTest&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 28, ignored: 1 Build time: 00:00:53


Is the idea to apply outstanding offers to all the SchedulerNodes at every SchedulerEvent? There are several scheduler event types.


@nathanfranklin, @tuopuu I just created the branch for the issue #124.

I think that regarding the lattice uml we should focus more on the actual structure of the hdf5 rather than the h4lattice class itself, as this is already covered by the ABCLattice diagram.

I tried to go through the h5lattice class myself but im not able to identify any descriptor, @tuopuu can you give me some help here?


Hat das mit dem übergeben von u als Parameter nicht funktioniert?


Changed main institution in dev @acroos per item-4


I don't know why but connections keep alive before closing a server.
I found two ways to avoid this situation.
agent: false or Connection: close


Test had been published. It computes exactly the safethreats and is simple and cheap as it can be.
http://tests.stockfishchess.org/tests/view/5547b2240ebc5940ca5d6877

On local tests, it fares better than master and your version @ 9 +0.06
Hopefully it will be strong enough for LTC.

And eventually, introduce those pinned pawns in a few other places in the code, which I had not been successful so far. General changes, such as changing ei.attackedBy[PAWN] to contain exact, unpinned  attacks, does not work for some unknown reason.


The same code exists in the html encoder, maybe there should be a ForbidHtmlCharacters method somewhere?


Thanks! You'd also need to update the doc strings of functions containing the string "current window" to prevent any confusion to a user not expecting `avi-all-windows` to be `t` by default.


inside namespace


Is this null check required? I see that containersBeforeSchedMap is updated in line 173 (beforeSchedulerEventHandled()->applyOffer())


It also turns out that aws-sdk in Hackage is somewhat outdated and there's a newer version in github. Not sure what that's about.


I would summarise that in the post instead of the sweeping statement you have made.


Have you installed it?
`sudo apt-get install ccache`


We might want to add another property for the Date to this object being pushed, in order to have a record of when the data was scraped. Something like:

```date_scraped: new Date()```


hmm, are you using an ati card? I can probably fix that by getting rid of the gl.Lists, which I'll look into today I guess. They don't seem to make DrawWorld any faster anyway. :|

As for the constructor splitting, I made CostOfJob very very easy to replace, so if you write your own version I could make an option for it. :P Personally I prefer to have 6 missile towers built at once, which is why I asked xponen to do that in the first place, but job priorities can be practically anything. Just note that lower cost = higher priority.


local $dir_list[3] = ["Loots", "logs", "Debug"]
This one should be taken from global variables i think
loot_log_cleanup(100) - this number should be adjustable and whole function should have checkbox in gui to run or not.
Also put this in main au3 file is not the best idea.


Function to split the timestmap is working.. but the hi end is overflowing the int (thats why i have the bigint here)
Anyway its not working because of this .(when i combine it in the big int it overflows)
Need to do more research :... fyi there are datecodes that do not overflow and the fimestamp seems to be in the ballpark. There is light ... but its far away


I didn't know how to describe it before. Just had a feeling about it. :+1:


Despite what they said, you don't need this special case (though it was a bug, good catch Mickey!)

When `size()` is 1, you take the average of the first element with itself - not a problem!

Another possibility

    if (times.size() % 2):
        return times[times.size() / 2];
    return (times[times.size() / 2] + times[(times.size() - 1) / 2]) / 2

Note:  you don't have the right spacing around the `+` - and you missed an error in that line.  That probably isn't a coincidence...

Please also remember the "last line effect" - if there is a bug in your routine it is far more likely in the last line than any other.


nu toate culorile sunt vazute in lista in cazul in care incearca sa extraga culoarea secundara


This is a working data exapmel ... first was a test second is real data;


Yes, agreed.  But I'm thinking right now we should wait at least until we need it in multiple places.  Since we are now publicly publishing WPLib I think better to move very cautiously about adding new things?

For example, what should that utility module look like?  Maybe we should wait and discuss that before we push forward?


Should be

$(LOCAL_PATH)/recovery/root/etc/twrp.fstab:recovery/root/etc/twrp.fstab


That said, I should have made it private.


Is this bad? It seems to work fine in practice but couldn't find any definitive statements on whether or not it's blessed by the standard.


Another way would be to use the universal prefix arg to allow the user to do the search in either only the current window or among all windows; by let binding `avi-all-windows` to t or nil based on the optional arg val.


Thanks for moving it back, much easier to review. I think you and I have different expectations about method ordering!


Fixed.


develop (totalgee) - Win64-Windows-8.1-VisualStudio-14: OK (346 of 346 tests passed)

![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)


tested and working


nice function, have you tested it?


Can you clarify the question?


Using `ToCharArray` here seems overkill, `new[] { '\\', 'b' }` is not significantly less readable.


Sharingan pls


changing printf(_x( to printf(__( is apparently not sufficient :-)


 - __style__: [src/chaiscript_stdlib.cpp:18](../blob/cf4efacbe8be78c01ba42b77218d8f91ee2a6253/src/chaiscript_stdlib.cpp#L18) The function 'create_chaiscript_module_chaiscript_stdlib' is never used. (unusedFunction)
 - __style__: [src/main.cpp:169](../blob/cf4efacbe8be78c01ba42b77218d8f91ee2a6253/src/main.cpp#L169) The function 'get_eval_error' is never used. (unusedFunction)
 - __style__: [src/main.cpp:140](../blob/cf4efacbe8be78c01ba42b77218d8f91ee2a6253/src/main.cpp#L140) The function 'help' is never used. (unusedFunction)
 - __style__: [src/main.cpp:216](../blob/cf4efacbe8be78c01ba42b77218d8f91ee2a6253/src/main.cpp#L216) The function 'myexit' is never used. (unusedFunction)
 - __style__: [src/main.cpp:158](../blob/cf4efacbe8be78c01ba42b77218d8f91ee2a6253/src/main.cpp#L158) The function 'throws_exception' is never used. (unusedFunction)


develop (totalgee) - x86_64-Linux-Ubuntu-14.04-clang-3.5-AddressSanitizer: OK (346 of 346 tests passed)

![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)


whoops.. lol Gonna fix


haha xD nope ! and then just add ccache .. where ?? i see u'Ve removed it again from Makefile ... and now ??
btw: i've pushed up in my repo Stock GB and JB Kernel Sources ... so if we need this sometimes.. so that saved for me now 1GB on my HDD .. because of deleting now ... xD


@hollywoodheinz look at this for me


Yes, it is needed because the offer might not have been available at the time beforeSchedulerEventHandled was called. Only if the offer was available, we would have added it to the map.


Good question.  We should discuss.  Right now `{ThemeClass}::get_root_url()` will work.  But you don't know what that class is generically, of course.


If !mux is recursive, why it is defined here for windows an linux oses? I saw the output with both or some removed, so I in doubt here. I expected that removing from windows and linux would give the same result.


Si Senor.


Sorry about this, if you really mind I can rebase and apply patch just the changes, I did not notice that everything had been reindented.


Seems to work well


 examples/ECMAScript5/exmascript5_tokens.ts

filename wrong + directory in upper case


Sorry about that  - https://github.com/galaxyproject/planemo/commit/80c753b51d56ef3acdc5ac6d6785665891f4ec6f


Fixed.


which also... what? :)


Updated.
Let me know if you have any question or comment. Thanks.


Yes, fixed.


You are right. Ignore this. At the time of making this change, I had a wrong understanding of this method is called. That is why the fix that I made for supporting continuous scheduling is not valid. Take a look at the refactoring commit which  I did in my private repo. Continuous scheduling is disabled in that. We need to find a way to intercept the call before scheduling happens. It cannot be based on events.


lol. I remember, how painfully was to download it from ***** samsungs own site, because of slow speed :D I had to restart download several times :/ thanks, now I also can delete it :D


It's working here.
You have to rebuild the .po/.mo to exclude the context, or else it won't load.


$groups can be empty. This will cause a warning:
Warning: Invalid argument supplied for foreach() in c4m_og_get_user_group_nodes_by_bundle() (line 593 of /Volumes/webdev/www/amplexor/ec/capacity4more/capacity4more/modules/c4m/og/c4m_og/c4m_og.module).


develop (totalgee) - x86_64-Linux-Ubuntu-14.04-clang-3.5-ThreadSanitizer: OK (346 of 346 tests passed)

![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)


develop (lefticus) - x86_64-MacOS-10.9-clang: OK (347 of 347 tests passed)

![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)


When I asked around in the JavaScript room they recommended to declare `"use strict";` at the functional level, rather than at the module level. Just letting you know, not sure what difference it would make in our case.


The long method names are because there are multiple protocols that expose `clear`.


I'm working on the testing for v1.0 but since I'm the only one testing it and I had a few personal things going on. The release is going more slow than I expected


Ich hatte es umgeschrieben, allerdings das STackoverflow beispiel ist mit einer anonymen funktion. Wir haben keine anonyme und dann habe ich nicht ganz verstanden wie ich ode45 klar mache dass die u parameter sind und wollte auch erstmal testen ob es insgesamgt überhaupt funktioniert. Das steht noch auf meiner Todo liste.


good to know, does the .babelrc is read when require('babel/register') ?


develop (totalgee) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (346 of 346 tests passed)

![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)


You could use ´if (url.trim() !== '')` here, since someone could also for example introduce 3 spaces in the fields and it might pass!


I agree; I seem to recall that I did this for some kind of compiler reason, but that might have evaporated by now.


xD ^^

how to use ccache ?? i've installed right now ...


I am not so sure about that. The JVM generally optimizes for what javac outputs. I guess we could test it.


it makes the modules much smaller to remove all the bubbles after they've been parsed. Eventually we probably want to make the parser be a sentence transformer, in which case this won't be as necessary, but it won't hurt to do this here; we have already parsed every single bubble that can exist.


This was something I threw it for you to review and comment on.  Please consider how it should be handled and recommend an alternative.  As long as it does not conflict with the overall planned library conventions that I might have in mind but not yet discussed I'll defer to you here.


Careful, you're setting the global variable `message` here :smile: without `var`, a global will be set.


You need to update the .po with this .pot change I've pulled, it should work then.
What language are you using?


Try to modify Makefile or so:
`export USE_CCACHE=1`

Not sure if last command works (I use it when build ROM).


develop (totalgee) - x86_64-Linux-Ubuntu-14.04-gcc-4.6: OK (346 of 346 tests passed)

![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)


Maybe.  I'd be interested in seeing your pull request on that.

We would want to make sure it does not paint us into a corner, but if it is internal only that may not matter.


applyOffer bumps up the capacity of the node prior to scheduling. Nothing is done during heart beat. NmFreed should be taken care by RM itself right? Why do we need to handle that?


That's a reasonable point; it might be worth (temporarily?) bridging `Result` and `Cursor`.

I've been keeping my eye on LlamaKit, PromiseKit, etc. in the hopes of something better coming along.


hmmm or maybe .... wait ..


hah! No worries :) That'll be a nice addition.


como estas profesor, pudo ver loq eu le mande anteriormente por correo ?


develop (totalgee) - x86_64-Linux-Ubuntu-14.04-cppcheck-1.67: OK (0 of 0 tests passed)

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


It seems to be the prevailing style for constants, and it avoids the need to qualify their names. That's all, I think.


thanks, good catch. Since this will throw a NullPointerException if either is null in this case, I will remove the assertion.


Along those lines we need to discuss a strategy for including candidates for inclusion in WPLib.   Maybe a WPLib Beta repo?


Yes perfect, it's not needed to test version against 217


93441412-HVACTemplateMissingSetpoint-4328 (mjwitte) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (1646 of 1647 tests passed)

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


Checkpoint [31]:  Incomplete.  Missing *before_action :flash_attack* and associate method to show the flash message.  Please update, resubmit for review, and send me a message when it’s complete.


or like this:

export USE_CCACHE=1

and then make .... i mean not in makefile .. but directly in build script .. or ??


Just this command  "export USE_CCACHE=1" or only modifiyng Makefile (don't use both variants at the same time).


I agree we should unbundle it and add it as a dependency.


develop (totalgee) - i386-Windows-8.1-VisualStudio-12: OK (345 of 346 tests passed)

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


This will be squashed with mpalmi/clip@72f2560f3f85a97fe7e87c9c6c554852a1997a04
Addresses @tomhurd's feedback:

tomhurd commented on 72f2560 3 days ago
> init_manage_var_lib_files instead of init_var_lib_manage_files?
> I didn't get any initrc_t denials when testing.

https://github.com/mpalmi@72f2560f3f85a97fe7e87c9c6c554852a1997a04#commitcomment-11010768


I am flattening the List<Map<Sort, K>> into a Map<Sort, List<K>>. If you know  a better way to do this, I am all ears, because I agree this is quite ugly. In the meantime, I will add a comment explaining what it is doing.


Using ccache in build script causes errors in the end of compilation(at least for me).
I mean this:
CROSS_COMPILE="ccache /home/chrono/kernel_build/novatp_arm-eabi-4.9-master/bin/arm-eabi-"


The AI is understandably happy with its outpost_pod part test design, but even if it needs nothing more in the design we should probably have it copy it to a new design, with a new name, for actual deployment :smiley:
![Image of AI outpost ship design] (http://i.imgur.com/QsHEHFr.png)


I guess you meant `next = lit_Undef`? If so, please just let me know, I can fix it here.


I'm not sure I follow why that would be simpler; it would make the method twice as long.


93441412-HVACTemplateMissingSetpoint-4328 (mjwitte) - x86_64-Linux-Ubuntu-14.04-cppcheck-1.61: OK (0 of 0 tests passed)

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


Yes, only Windows, I propose to keep it as it is


`destination` and `destinationLength` instead of `buffer` and `length`? As is now it's inconsistent with TryCopyCharacters and "buffer" doesn't make it clear that it is destination.


The `<br>` breaks the description on the npm site lol


`.es6` extension would be nice too
There was a fix in [`5.1.12`](https://github.com/babel/babel/blob/0c570cb5998b4435673369371279f1da46aa7fbb/CHANGELOG.md#5112) so yes, I think it's beeing loaded.


This is a terrible change.  How dare you!


And if they continue to be issues, make them TBU methods, don't write the same method in two places.  That's like programming 101, day one shit.


okay.. i believe that'S wrong .. so no wonder..

i mean this:

	export USE_CCACHE=1

	export ARCH=arm

	export SUBARCH=arm

	export CROSS_COMPILE=/root/codinap/android_tools/arm-eabi-4.9/bin/arm-eabi-

	make codinap_f2fs_defconfig

	make zImage

	make modules

found also here:

take a llok a the bottom .. "optimizing a build env.. ...."

https://source.android.com/source/initializing.html


The original code checked the return value from ```Genome::Sys->rename``` while the new code does not.  From my reading of rename(), it can return false without throwing an exception.


Do you have an example where `display/type` is empty and therefore the other cases are needed?


yay awesome! soyuka what do you think about this PR? Anything to enhance?


Why catch this error and re-throw? Also, it's important to log `e` as to why the failure happened.


Now that the patch has been included in vim ([see here](https://github.com/vim/vim/commit/1b741763301d12600e15effe784d534edd19b54c)), that patch can be marked as `NA` in `version.c`.


Sorry, not `NA`, but the patch should be marked as included.


TeamCity xBIM :: XbimExchange :: Build [Build 282](http://ncl-xbim.hq.4projects.com:8080/viewLog.html?buildId=1587&buildTypeId=XBIM_XbimExchange_Build) is now running


Looking at it right now great job so far <3.


so....


Catching and re-throwing is generally not a good idea - also, you are missing details of the failure - `e`


```C#
for (...)
{
    destination[i] = source[i];
}
```

generates better code.


TeamCity xBIM :: XbimExchange :: Build [Build 282](http://ncl-xbim.hq.4projects.com:8080/viewLog.html?buildId=1587&buildTypeId=XBIM_XbimExchange_Build) outcome was **FAILURE**
Summary: Exit code 1 Build time: 00:01:15


develop (lefticus) - x86_64-Linux-Ubuntu-14.04-clang-3.5: OK (347 of 347 tests passed)

![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)


develop (totalgee) - Win64-Windows-8.1-VisualStudio-12: OK (345 of 346 tests passed)

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


Might want to consider storing an array of `Tile`s instead of holding `JSON`. It seems a bit cleaner to store it in the native format rather than the serialized format, and then the subscript accessor wouldn't need to create a new `Tile` instance every time.


ES6 is sexy!


This clarifies things - in an if expression without an 'else' clause, the result will always be unit no matter what happens in the 'then' clause


This looks good – maybe change it to `warnings.warn('…', DeprecationWarning)` to follow Django's lead?


private (assuming you don't use this externally in later commits)


private


can the method be renamed to logFailureMessage ? as it puts failed always.


Ok.. but if you'll build kernel as part of ROM, you'll see that android build system uses command
`make -C /path/to/kernel -O=/path/to/output/dir CROSS_COMPILE="ccache /cross/compiler" `


Another ForbitHtmlCharacters candidate.


It looks good! I think the only edit I will make is to have the time and reply line on the same line. For example: "As of <time> the gem conversion rates for <amount> gems are:"
I feel like this way it would be more compact.


I think this typedef may be used in the struct containing the function pointers.


Why not change the default language in the shared file as well.


develop (lefticus) - x86_64-Linux-Ubuntu-14.04-clang-3.5-LibC++: OK (347 of 347 tests passed)

![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)


`.es6` missing here too

From the docs:

>All subsequent files required by node with the extensions .es6, .es, .jsx and .js will be transformed by Babel. The polyfill is also automatically required.

http://babeljs.io/docs/usage/require/

But I think we should keep `.es6` and `.es` only.


Heya Jeanfrancois

I am not sure about those dropping semantics of all-white-space-body parts in the else clauses (introduced in previous commits resulting in this refactoring commit) after the protocol handshake being done. We happen to run into a case where the last body part (or parts) might legitimately contain whitespace-only data. It getting dropped is plain wrong and leads to problems. Do we really have to take provisions for "padded data" after the handshake ?

Thanks for your advice and cheers, Christian.


`addIntrumentedClass` would be what I need. I can't annotate all classes which I like to shadow because they belongs to external frameworks like AndroidAnnotations.

I need to shadow https://github.com/excilys/androidannotations/blob/develop/AndroidAnnotations/androidannotations-api/src/main/java/org/androidannotations/api/BackgroundExecutor.java for synchron test execution.


aha very interesting .. thx ! i'll build now.. but i know.. the difference i to see in next building .. because for the first time the ccache Folder is just empty xD


IP substitution addresses issue 4


Aloci de două ori aceleiași variabile. Valoarea adăugată prima dată (`new T[index]`) se „pierde”, e disponibilă imediat pentru garbage collection.


Thsi is a nitpick mostly, the caller has the burden of attempting maxAttempts before calling this method. It would be simpler if we pass in attempts as one of the parameters to make it more explicit. But there is nothing wrong in this.


Please see [PR](https://github.com/josdejong/mathjs/pull/350#issuecomment-98792252) for composition function.


La fel ca mai sus. Practic faci două alocări în loc de una.


NOTE: If you need this private key to deploy, you can copy it from ot10 (or current **devapi**) to your local stash like so:
```sh
scp ot10:/home/opentree/.ssh/opentree ~/.ssh/opentree/opentreeapi-gh.pem
```


A lot of stuff is not related to our hardware, or needs drivers we don't
have, so it won't make any difference if merging it.

On 2 May 2015 at 23:24, Oğuzhan Yiğit <notifications@github.com> wrote:

> yeah thats true but as far as ı can see we are missing whole of patches
> from upstream (e.g
> https://www.codeaurora.org/cgit/quic/la/kernel/msm/commit/?h=LA.AF.1.1.1_rb1.3&id=ee2422a4b081b2b8bad143af5fcefdb852ec4b2e)
> and several mdp driver, power management updates
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/arco/android_device_samsung_ancora/commit/0f48d523d12b7e452bdd0494a0cced48f00f4f79#commitcomment-11013744>
> .
>


@mickeyren I'm curious here, what's the rationale for deviating from master?


```C#
for (...)
{
    if (!_allowedCharsBitmap.IsCharacterAllowed(text[i])) { return i; }
}
```

generates better code.


I hate russ3.txt so much. :-1:


Not sure we want to remove the padding here...the thumbnail sections looks pretty crowded without the top/bottom margins.


For-ul ăsta ar putea să se oprească mai devreme.


Yeah, it doesn't work on our device. We need something like the last two
commits here:
https://github.com/arco/android_packages_apps_Torch/commits/cm-11.0

On 3 May 2015 at 23:50, Oğuzhan Yiğit <notifications@github.com> wrote:

> also @arco <https://github.com/arco> Did you test the following commit (
> ArchiDroid/android_frameworks_base@3987eea
> <https://github.com/ArchiDroid/android_frameworks_base/commit/3987eeacbb8568e4b195de2d795a54cde3baf8cc>)
> for torch tile issue ? we need something like this for torch issue
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/arco/android_device_samsung_ancora/commit/0f48d523d12b7e452bdd0494a0cced48f00f4f79#commitcomment-11018516>
> .
>


Bun, am obținut funcționalitatea. Dar nu e optim, la fiecare operația aloc memorie și mut toate elementele. Poate aș putea să aloc memorie mai rar? De exemplu, în `RemoveAt` n-am de ce să aloc alt array, pot să-l folosesc pe cel existent.


joi is used in [lib/definition](https://github.com/fritzy/VeryModel/blob/master/lib/definition.js#L2). Probably should be in `dependencies`


You just check for if poll exists here, and inside the function if poll_end ACP setting is activated.
But what if end time just isn't specified? Does this function have to be executed? And will it pass?


Sorry for all the commits, I'm at school using Github online. I just thought - perhaps you can revert all of these commits and I can properly remerge the changes from my git client at home?


@dgnorton If you're escaping the .(dot) here, Shouldn't the literal string be `^payments.`?. Why are you returning the backslash as it's only the escape character?


All in one commit


> _I think we need to utilize a PHPDoc notation like WP does to indicate at what version a property, method, function or class was introduced..._

Agreed.  Need your help with figuring out out what to use.

> _I also think we need to utilize a PHPDoc notation like WP does for indicating whether something is public or private..._

Sure. I guess we are going to need to QA steps in the process for things like this.

> _Additionally, as an open source project, we need to be adding the GPL notice to the top of each file._

WordPress does not do that and [StackOverflow thinks we should not](http://stackoverflow.com/questions/845895/putting-license-in-each-code-file).


Did the style guide change to require space on all parenthesized expressions? Used to be that nested parenthesized expressions didn't require additional padding.


Thanks! By the way, unless I'm missing something this can cause some security problems since the job submitter is also can add a comment.


At present, let's keep it here.  This is safer and I'm already touching too much code.


@steveshimizu is mean


done


Is there a better way to name this class? If u rethink the name...u will be able to understand the responsibility of this class too...do u think it should include everything what it does now?


Sounds good :D


develop (lefticus) - x86_64-Linux-Ubuntu-14.04-clang-3.5-AddressSanitizer: OK (347 of 347 tests passed)

![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)


Click the three dots dude :) Here's the full message:

Shell script I use to initialize my build environment which also calls CM envsetup.


I don't think we want to change this, either. This removes the left/right margins, so now the padding in between thumbnails is bigger than the padding outside, which looks uneven.

Just for reference, here's how the padding layout worked before:
[8][--8--thumbnail--8][--8--thumbnail--8][8]

The two middle 8 paddings combine to make 16, so the outer left/right/top/bottom padding balances it out.


Yup, good idea.  That's out of scope, though.


Imagine you run this app for more than 50 times...each time u are creating the same hotels rite...but diff objects for them..Is this needed? Is there a way/Pattern to avoid it?


It's gone.


When we build from CLI CordovaApp[Debug|Release].projitems are generated on every build. In VS, we can't do this due to build params missing, so we're not touching CordovaApp[Debug|Release].projitems.
defaultLanguage comes from contents.xml and all the values that are changed in contents get applied both on VS and CLI on every build.


What's so broken about the unban buttons on the ban list?


But there is `css` in the URL.  Worst case we can do an `wp_remote_get()` on it and inspect the HTTP Content-Type of the response and then cache it.

Or we could move to a signature of `enqueue_external( $handle, $src, $args )` and that would allow us to pass in a `'content_type'` arg?


What I'd expect is that if you reuse a stream id (say, two requests to the same endpoint in parallel) the two distinct response streams would get interleaved. I didn't see any special handling for this in `multiplex` so I can't see how it would be able to differentiate data on the two streams, but I may be misunderstanding.


What are those three lines above for?
I don't really understand them, and they look pretty complicated. Three assignments for what?

I thought the check would be easy, just parse values to a date and see if it is valid xD


:open_mouth: Looks like you deleted the lines by accident. To put them on the same line, you would just need to remove the \n at the end of line 32. Then, add a space after "(UTC)", and lower-case the T in "The current gem...".


93441412-HVACTemplateMissingSetpoint-4328 (mjwitte) - i386-Windows-7-VisualStudio-12: OK (1646 of 1647 tests passed)

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