I was thinking about your replies in the way of verboseness.  I wanted to put a suggestion out there that brings together some of this stuff across comments:

* BallotContest (or simply Contest)
  * MeasureContest
  * CandidateContest
* BallotChoice (or simply Choice)
  * MeasureChoice
  * CandidateChoice

This has the advantage that there is the relationship "XXXChoice is a choice in a XXXContest."

Also, I feel like "choice" (in addition to being shorter than "selection") better brings to mind the idea of an unfilled choice.  For example, it feels less natural to say, "What are the selections on this contest?" as opposed to "What are the choices?"  To me, "selection" connotes more the idea of what a voter already picked.


Why? Just found them in my deps tree and was wondering what the reason for that is.


Yes, the get_projects function is duplicated from the home view and the projects are displayed in the same way as they are displayed currently in the home page. Please inform me what needs to be factorized.


I'm not sure I'm following. Yes, it is invoked for a specific failed operation.


Lines 7-9 could be simplified to one line in the controller:

```ruby
  authorize @comment
```

Then the `CommentPolicy#create?` method could check the `@topic` and `@post` authorize this user. It'd look something like this:

```ruby
class CommentPolicy < ApplicationPolicy

  def create?
    comment = record
    post = comment.post
    user.present? && PostPolicy.new(user, post).show? && TopicPolicy.new(user, post.topic).show?
  end
end
```


I mean do replace inside codeRegister, it would be better for everybody :)


Yes, providing a link in the commit message would make it a little easier to follow.


You're right. Thanks! I'll add the correct one.


@Alys here we go, I'll start running this on all the accounts that can't get resets.


Este paquete lo tengo guardado en una carpeta de neatbeans


finally!! :)


Your use of mixed tabs+spaces indentation in this file borders on "internet hate crime".


Ok mae esta bien..


@BrennanConroy did you test it in VS?


Namespaces?


SVG is an XML format the height and the width are just values in the data. This does not work with photos but more with illustrations and other vectored images (logos, graphics). I did not test for a size because the image because it is vectored can be any size even if  both height and width are zero.


This is the CM specific policies, which are included by default during build, so there's no need to add it in device.


> Somewhere I would explain how "transaction" works.

There is a dedicated section for this a bit earlier in the docs. Is that what you mean?

>  If it fails, then a regular exception will be propagated.

Ok.


Yes, but there is going to be no visible change


Right, please see 67fc031.


How about calling it `gitconfig` right away? It is clear from cloning to somewhere else than `/etc/` that this is not the authoritative `gitconfig`, but it is good enough to be copied into that location as-is...


@Bilge, the file name already comes in through the `source` parameter.  The difficulty is knowing when to prepend `x` to the short options if they're provided without a dash.

@highlyunavailable, Will you submit your last comment as a pull request against 2014.7?  If you can't get to it today, I will do it since I think we should have this in 2014.7.3.  Thanks.


I'll fix it, I wanted to make some other changes anyway...


No worries.


Doesn't work unfortunately. The torch tile disappears, and the camera app hangs during launch.


max_element работает за O(n), что недопустимо


Consider .Any() instead of .Count() == 0. The latter enumerates the whole collection. This occurs in multiple locations in this file.


I don't know, I find the usage of store-specific exceptions not very desirable. I would rather say we should provide more abstract exceptions at least for the most common things (also see [OGM-774](https://hibernate.atlassian.net/browse/OGM-774)(. Thus I didn't go for showing a specific exception here.


@Free-BSG , let's look at the make.bat file. I'm not seeing the .png --> .PNG conversion on the Linux builds.


Нужно использовать типы P и T в качестве типов для приоритета и самого добавляемого элемента


Duplicated code is usually a bad idea. Since the code is supposed to
achieve the same goal, it should be put in a single location. So that if
the code is modified, it has to be modified only once.

On Thu, Mar 12, 2015 at 7:28 PM, Nitika agarwal <notifications@github.com>
wrote:

> Yes, the get_projects function is duplicated from the home view and the
> projects are displayed in the same way as they are displayed currently in
> the home page. Please inform me what needs to be factorized.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/NitikaAgarwal/osm-tasking-manager2/commit/8a1612922dc881d168e930633fc9e768042c935b#commitcomment-10170949>
> .
>

--
-------------------------------------------------------------
  | Pierre GIRAUD
-------------------------------------------------------------


Don't add static constructors when field initialiser is enough


90019124_RadianceData (rpg777) - i386-Windows-7-VisualStudio-12: OK (2165 of 2184 tests passed)

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


@TakaGoto The answer to all those questions lie in the fact that I wrongly commited this before it was done.

a later commit removes these issues


I was thinking about the last commit with the `#ifdef MZ_LONG_DOUBLE`. Now I think that it's not a good idea to put this `#ifdef`. Moreover, I think that all these `#ifdef MZ_LONG_DOUBLE` in this file should be removed, because all these optimizations can be also enabled even when the extflonum operations cause errors. But I had no tested the idea, so perhaps it can cause some problems.


Maybe better `install -d -m755 "$pkgdir"/etc/profile.d`, as it is done here: https://github.com/git-for-windows/MSYS2-packages/blob/267a3deb79f4fffbb740544e1ecf3a75b109d435/filesystem/PKGBUILD#L82 ?


O texto desta nota não está claro. Imagino que "a informação" em questão seja o título da seção, mas aonde deve ser repetida em *front*? Seções são naturalmente identificadas abaixo de *front*.


yea.... sorry about the bag of mixed nuts.  I was in scramble mode to get something deployed using the changes to the API.

you missed (at least, if not more):
- restyling to be a bit more pythonic. imo, code is still messy.
- changing `data` to `persistent`
- changing `/data/blocks` to `persistent/data`
- exposing new options
- exposing new targets


What is the channel?


very nice work


I haven't made many researches yet. It breaks on compilation. What is your kernel configuration?


bower files are installed in app/public/bower you don't need this in this case!


Correct.


Good catch.


Fixes crash on main menu. (unset bone index - should default to 0)


Yes, I understand. The problem was that in the first time we thought the problem lies there that we're using a subfolder. So we've changed everything according to the subfolder. But then we realized that this wasn't the root of the problem. (It was because of using "\" instead of "/" for Unix systems) And after making the changes for that, we decided that it's okay not to use a subfolder... Maybe we can integrate it again, but I think it's not that important ;)


Tebrik ederim seni tanımıyom ama güzel yapmışsın


89911514-Allow-OtherSideCoefficients-in-AFN (jasondegraw) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: Tests Failed

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


If you agree with [my comment](https://github.com/git-for-windows/build-extra/pull/6#commitcomment-10171049), this would need to be changed to `"$(cat "$srcdir/$pkgname"/gitconfig)" \`, too.


ugh, this is sooooo sad @thefosk !


omni


I'd like to just self:write('') here all the time and not need to check the content encoding but this write seems to cause errors with a https server


@normanmaurer I think we can make the whole `FlowState` class `private`. WDYT @nmittler @Scottmitch ?


@MartinNowack I'm unfamiliar with STP's C API so sorry if this sounds dumb. You are probably right but why should the pointer to the STP expression be deleted? It's clearly being used by the expression it is part of (``res``). If I'm reading this change correctly as soon as ``ex`` goes out of scope ``vc_DeleteExpr()`` will be called on it even though it's part of ``res``.


> Hum I guess not

You guess what not? Maybe you can specify which piece you suggest to move where? Currently the new section is a sub-section of "6.3 On flush and transactions". DYT think it should be on the same level, i.e. 6.4?


Bash Scripts


//Code can read .txt file from the disk, it worked. I have hard time reading your code, can you see if it can be merged into your code ? Thanks, Zejie.


@ch3cooli @YueLinHo

Something like this could bring us major performance improvements as we do not need so many locks as with libgit and do not need to create a new repository objects again and again.
See https://stackoverflow.com/questions/29016897/how-tolerant-against-external-changes-to-a-working-tree-is-libgit2-with-a-loaded


Thanks


AssertionError with the String, Throwable constructor is >= 1.7 . So let's go with IllegalState? http://docs.oracle.com/javase/7/docs/api/java/lang/AssertionError.html#AssertionError(java.lang.String,%20java.lang.Throwable)


It look don't work .... i don't understand why. http://pastebin.com/58vjwRSC


I don't think it poses a problem. The same list of ops is referenced by this.


@gwensmu you will need to update the counts. I believe there are 5 goals in the fixtures for this participant. Therefore it should be ```cells: ["Goals", 5, 5, 5]```


Boom!!


Looks good.  I see that all generic references to being able to spray to roxie were removed as well between lines 540 and 544


@avital great idea!


Yes.  I'm planning for some sort of "start --mode=advanced" or "config --mode=advanced" target that would expose additional configuration options, such as nginx worker_processes.


If don't build `data` anymore, should we delete the icon file from the repo? Seems pointless to keep around now, right?


Seems fonts were not really moved, just symbolic links?


yes

On Thu, Mar 12, 2015 at 1:02 PM, Chris Schaefer <notifications@github.com>
wrote:

> oh i see what your saying now -- simply change:
>
>    - python/example-executor -> python/test-executor
>    - java/example-executor -> java/test-executor
>
> and all should be well to merge, right?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/chrisjs/mesos-go/commit/e3605c5e405235ce5783aca533e0569928442a0d#commitcomment-10169031>
> .
>

--
James DeFelice
585.241.9488 (voice)
650.649.6071 (fax)


+1 ... it may be package private for testing, not sure.


What's this hooked up to?


Yeah I'm fine with that.


Because third-party adapters use them in their test suite. :)


if admin.tableExists(tableName) returns false, we don't want to execute admin.getTableDescriptor(tableName), as that will return TableNotFoundException in that case


Um, the purpose of this file is not to display on github, but as input to JSdoc. It produces http://3dmol.csb.pitt.edu/doc/index.html
Have you checked that these changes don't break the documentation?


You must be on the release and not on git and then did not test your change from u->HasMode(whatever) -> u->IsProtected, so this patch actually doesn't work. IsProtected was only changed to have the protected privilege in 6d61a842866f3a32551f46c46609d688697f2f2e.


Because these numbers are powers of two...


Do we want to keep the browser changes in order to fix the browser eval issues?


You forgot to fill that in


ok. great.


This won't work - the timezone of the test runner shifts the resulting date. This worked for you - assuming PDT is your timezone. For me it fails (PDT is UTC-07, while CET is UTC+01).

You could split the tests into tests containing timezones (as your reference contains timezone data this should work) and test without timezone data. The dates could be check against a triplet of integer directly queried and compared with the resulting date. If you introduce tests with times in them, without timezone you will also see timeshifts depending on timezone of tester.


My bad. Reject this and I'll do another.

—
Sent from Mailbox

On Thu, Mar 12, 2015 at 6:48 PM, Eli Grey <notifications@github.com>
wrote:

> You forgot to fill that in
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/mildfuzz/classList.js/commit/d4ec50a96e7b70cdcbf35791e074f026d703027b#commitcomment-10171305


this one should fix #25
but not sure if it does with you @rtyler since I did not see the error locally


Thank you for your contribution!


its fine, we are checking ```tableName.startsWith(specEntry.getValue().getName())
``` this before calling ```isParent(tableName, specEntry.getValue())```


varagaragarin


Thank you!


Fixed by standardizing the whole file.


Grrrr.  I'm going to add an an editor config to the project to prevent this atrocity.  Sorry!


ack.


> If you want by default use Windows home directory for users you need just provide proper /etc/nsswitch.conf file.

Okay.

How do I do that, i.e. where is a description of the `nsswitch.conf` file? ;-)


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


I think you should just kill this and do ".*". The `element(2, X)` field of this tuple is going away in rebar3 anyway, so better be prepared. If you want to lock down the version, do so with the tag: `{tag, "2.1.0"}`.


@normanmaurer no, it can also be negative.


i don't think that will be the case, @abaranau , please correct me if i am wrong.


Creating test function for testing runtime speeds in EAX-ECX


right


<!-- PMABOT:SOB -->
This commit is missing Signed-Off-By line to indicate that you agree with phpMyAdmin Developer's Certificate of Origin. Please check [contributing documentation](https://github.com/phpmyadmin/phpmyadmin/blob/master/CONTRIBUTING.md) for more information.


Creating IO Controllers, could've used a boolean but; I needed only a 1-Bit integer not an 8-Bit Boolean.


By the way, if this way of doing things passes muster, I can do an IDV pull request so we don't need to keep a local override of GeoGridAdapter.


This should work !
Queen should not be a blocker in "middlegame".

Rook might still be worth a middle game penalty.
My guess is that, in middlegames, if you have no Minors,
than blocking with Rook is better than nothing, so should be OK.

so you might want to penalize only if some other minors are available

else if ((pos.pieces(ROOK) & blockSq) && (pos.pieces(Them, KNIGHT, BISHOP))
    mbonus += rr * 2;


This looks great, @r-a-y. One request - could you not roll back 7902c49caa? (the filter name change)


Im a newb at github, just want to help out.


No, that extra checking isn't needed since it'll always be included. The php handles the conditions for this before it's sent to the js.


ack


`F::GetPluginsDatFile()` уже возвращает с директорией, поэтому в пути имеем дубль пути до `plagins.dat` - вкл-выкл плагинов не работает


Can you fix this warning?

/home/travis/build/esa/pagmo/tests/test_normalized.cpp:88:8: warning: implicit
      conversion turns floating-point number into bool: 'double' to 'bool'
      [-Wimplicit-conversion-floating-point-to-bool]
                        if (fabs(norm.get_lb()[k] - (-0.1) > EPS) ||
                            ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ ~~
/home/travis/build/esa/pagmo/tests/test_normalized.cpp:89:5: warning: implicit
      conversion turns floating-point number into bool: 'double' to 'bool'
      [-Wimplicit-conversion-floating-point-to-bool]
                                fabs(norm.get_ub()[k] - ( 1.0) > EPS)) {
                                ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~


no sorry for the mess, will remove both


Is copyright a multi-value parameter? I don't believe so. If not, don't convert to an array.


Love it, but does the shell really need to be restarted after every timeout?


npnp ;)


> Love it, but does the shell really need to be restarted after every timeout?

Well, it's restarting the shell after changing the gsettings value (which only happens once during a playbook run)..  But even that might not be necessary.  Just trying to be on the safe side.


`text` isn't really meaningful


Thanks, I just found that (through the broken link in MSys2's SourceForge bug tracker). I would still have preferred an example, but I can fight with that enormous bulk of information, no problem.


r=larsbergstrom


@peernohell pictures is written wrong on line 92
nice PR btw


Thank you!


ack.


It seems on some archs it makes no difference. My GT650M gets the boost. A Maxwell (GTX980) I tried showed no difference at all. My laptop core i7 show a boost while a desktop core i7 I tried it on seems to actually show a slight regression, if anything. But for x86 it doesn't matter much because all important formats use SIMD.

I haven't checked it on Tahiti yet. The md5crypt/phpass formats is the only ones you can tell a difference with, all others are transfer bound.


89911514-Allow-OtherSideCoefficients-in-AFN (jasondegraw) - x86_64-MacOS-10.9-clang: OK (1650 of 1650 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)


saw approval from larsbergstrom
at https://github.com/pcwalton/servo/commit/586c12ccc4de666c0cfbef84689c2e8ed7d7719d


merging pcwalton/servo/border-spacing = 586c12cc into auto


pcwalton/servo/border-spacing = 586c12cc merged ok, testing candidate = 8e811229


Yeah, this looks like something that could be of use to them.


I'm not sure what you will think about these tests, but I left them in because I used them to make sure the cookies were being translated correctly.  In hindsight these should have been parameterized tests, so if you don't think they're silly to keep around I will at least convert them.


Ooh, this is a tricky one.  I'm not sure we can use the C++11 braced initializers at all here in a "can gcc 4.4 handle it sense".  And even if we can, we need to come up with a policy on when we *should* use them.  I just finished the new Meyers book, and there really isn't a rock-solid case for either adhering to curly braces or parentheses all the time, so we really need to pick one to be preferred.  Currently, I'd lean towards just using parentheses, since that's what all our C++98 code uses.


Thank you for your contribution. I like the plugin, however I'm not sure if 'Requires' is the best naming option.  I think it's to similar to Moose' required attribute. How about calling it RequiredGroup (analogous to your other plugin), DependentGroup, ... ?


:heart:, glad to see this getting some love!


I told @shashiranjan84 how to support more methods. Open an issue on unirest-java repo and I will implement it.


Написаны юнит-тесты для метода equalsPass:

testEqualsPass - проверка полного совпадения пароля
testNotEqualsPass - проверка полного несовпадения пароля
testNotEqualsPassFirstItem - проверка несовпадения первого символа пароля
testNotEqualsPassSecondItem - проверка несовпадения произвольного символа пароля
testNotEqualsPassLastItem - проверка несовпадения последнего символа пароля
testNotEqualsPassLowercase - проверка на несовпадение паролей разных регистров(нижнего регистра)
testNotEqualsPassUppercase - проверка на несовпадение паролей разных регистров(верхнего регистра)


It should be 1.22rem (22/18)


Задача выполнена полностью по постановке.


I wonder if the `W` here should be the writer itself, so the first component would be `BufWriter<W>`?


@bors: retry


TeamCity MonoGame :: Develop (Win) [Build 3.3.0.2230](http://teamcity.monogame.net/viewLog.html?buildId=5666&buildTypeId=MonoGame_DevelopWin&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 444, ignored: 6 Build time: 00:14:01


LGTM!


I think it would make more sense here to change this to `if signature.errors[:email].include? :not_present`, since that way if we ever add additional verification on these fields, it wouldn't break the validation.

It could be that I'm misunderstanding, though?


Yes - I definitely think pulling all defaults and requiring EVs to be set (or values passed on the command line) is the best option.


Hey Penkamaster,

Hopefully you don't mind, but I started working on the bruteforcer a little bit!  I translated the variable names into English, and I'm trying to figure out a way to get it to automatically recover from failures.  I think I might have gotten around the most common one, but the unknown_instruction error is proving trickier.  I haven't checked those changes in yet, because they need to be tested a bit more.  I also want to try improve the parsing so it doesn't crash if you use extracted .map files, and maybe add a GUI option in to launch it without needing the "-bruteforce 1 / 0" option.  I might get distracted by other things before I complete it all, but that's my ideas so far.


Is @Tuuleh working on integrating this slug into a FB share button?


Ah, nevermind, this is reused for LineWriter.


wohoo!


Note the comment above here is outdated, since `Steam` is not a thing any more.


additionalSlinkyGHXWork (mmatts) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (1644 of 1655 tests passed)

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


this is probably OK to stabilize right now


so, let me use this for now on my jmdns branch, and once we know what is needed we can create the jlibtorrent java-pretty version with the cool methods.


SOON :cat2:


good

Can you delete your branch?


not


No, I think it's safer to use include?. I'll change that, thanks! :-)


Whoops!  Didn't mean to do that!  I've reapplied the commit to the PR.


@parassaini - don't hard code these values in javascript, pass these values from data- attributes.


Как сделать так, чтобы автоматически удалялись все связанные с вопросом ответы?
Ответ: ```has_many :answers, dependent: :destroy```


As far as I know; `so` is a verbal way of communication and is to be avoided in written text.

I would say that you should leave it be alternatively add a full stop.


additionalSlinkyGHXWork (mmatts) - 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)


Ya, I noticed ina big project this code was getting duplicated EVERY FILE!  I wasn't sure what this did ...


i think its better to return before_create since we dont need a new reference after update.


have a typo to fix here...


@parassaini - Please remove commented code.


done deal


It's slightly informal, but less informal than "you have been warned" from the next line.


@parassaini - Look if we can create one js file for both of these files.


I think here we need to insert another HTML link testing, as this was one of the bugs I've fixed for the previous release. The links have to be fine, specially when running on virtual/remote machine.


So should I check if $field['calc_dec'] is set in the PHP then? Will $field['calc_dec'] exist if the form has not been updated since the site has been running 2.0?


i think you can use string interpolation here instead of  + "-"+


Extra space, and ellipsis should be a single character. (I know it wasn't before.)


Can you replace all 4 empty cells with only one with `colspan='2' rowspan='2'`?


Whoops.  I replied from my work account.


additionalSlinkyGHXWork (mmatts) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: Coverage Too Low

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


looks fine. I'll merge it.


Is this `clone` necessary?


This is a not binary file! And another too
![1](https://cloud.githubusercontent.com/assets/9640205/6626176/88b50dc6-c8fd-11e4-8ee1-73a678533f84.jpg)


89911514-Allow-OtherSideCoefficients-in-AFN (jasondegraw) - Win64-Windows-7-VisualStudio-12: OK (1656 of 1656 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, while we're breaking things , config-keys that use `-` can be replaced by `_` for consistency with the rest of the API parameters?  That way they are also valid variable names and can be referenced without the string lookup.  Thoughts?


00-HTML-CSS-basics/ejercicio Css (2.1, 2.21 2.3, 2.4, 2.5.2)/index.html

En general, no se usa el tag **< br >**. Si querías hacer una lista podés usar **< ul >**. Con CSS podés eliminar el bullet ```list-style:none```


Per comment below: Can `-` be replaced by `_` for consistency with the rest of the API parameters?  That way they are also valid variable names and can be referenced without the string lookup.


La estructura sería más semántica si el h5 estuviese dentro del header.


Weird commit message.


na konec, takže imho tam tyhle tři tečky nepatří.


test


Maybe, while we're breaking things , config-keys that use - can be replaced by _ for consistency with the rest of the API parameters? That way they are also valid variable names and can be referenced without the string lookup. Thoughts?


No blank line after `if` statements always breaks my heart a little.


Nunca uses nada que provenga de W3Schools.com.
Ver: http://www.w3fools.com/


00-HTML-CSS-basics/ejercicio Css (2.1, 2.21 2.3, 2.4, 2.5.2)/superstyle.css

te faltaría indentar un poco mejor acá


@ntdb I cannot get this to pass. For me, it returns 1919 instead of 433.  Does it pass for you when regenerating wiki/course_data?


ok sorry. I had those files lying around somehow. I'll remove the commit =)


Request informative docstring.


Rewrote test1 prototype to be C 99 (ANSI C) compatible.
Pointer to char in place of std::string (plus this is faster).


Since we want to support Safari as well, and it [currently requires](http://caniuse.com/#feat=transforms2d) the `-webkit-` prefix, add that as well.


This still isn't right. The P should also be capitalized: `a_PosX`


What's the `-mdn-` prefix?


So this will be used for local register allocations. This will be faster and more accurate than doing it from an external header.


yeah sorry, accidentally recommended this for a sec


All the targeted browsers work without a prefix, so leave only the non-prefixed version.


i should've caught it too but "width-calc" made sense as "general-calc"


Two spaces for indentation.


Might be high time to switch from `waitress` to something more standard like Promises ([`Bluebird`](https://github.com/petkaantonov/bluebird).  I can drop a commit in to fix this.


You can't just delete GitHub comments as I can still see them by going in your history.

"GameCrafter: Wow, getting banned from PocketMine for two weeks was the best thing that happened to me. It's quarter-finals and that was good to get my mind off PocketMine for a while. So no magical red ribbon on a Steve that crafts games!"

You have a red ribbon here: http://prntscr.com/6fychf


Remove the extra indentation.


No. Why does this need to be reverted? o.O


The code for d3 won't fit when using MySQL, since the length of the `code` property is limited to 60,000 characters.


tady je navíc presenters /app/templates je správná cesta pro šablony na 2.2


Does this behave correctly if both `horizontal` and `vertical` are none? AFAIK, only the first arm of the `match` will be taken.


This certainly builds on gcc 4.4 (I Buildbot-ed before asking for review), and I think it's also allowed by [policy](https://confluence.lsstcorp.org/pages/viewpage.action?pageId=20283399) ("Initializer lists" are the first thing listed).

I've not read the new Meyers, but he did make a draft of the article to which I assume you're referring [available on his website](http://aristeia.com/EC++11-14/parens%20or%20braces%202014-03-18.pdf) last year, which is an interesting read. When I looked at it, I remember coming to the end convinced that `{}` was the way to go, only to be surprised when Meyers (and now Jim) disagreed. Maybe worth noting that Stroustrup "strongly recommends" `{}` "in every context".

I'm a bit nervous of making policy for everything; often, I'd prefer to trust developers & reviewers to converge on a solution which is readable and consistent with the rest of the codebase on a case-by-case basis. I suspect I'm outvoted on that, though, so have no objection if somebody wants to suggest a specific policy.


What I meant was that it was a good thing that I got banned until the end of the month because I have quarter-finals this month, and I can't go a day without checking the site. So don't make assumptions if you don't know what I mean. Are you still in school? Cause you said you would quit PM on your profile.


@rouge8 Are you happy now?


OK all set in 2feef4


We should figure out which _MSC_VER this actually was introduced in and test for that.


Pep8: No blank line in the end of file.


OK makes sense if those are deprecated. I myself thought they weren't hence
my suggestion.
On Mar 12, 2015 11:04 AM, "Stéphane Bachelier" <notifications@github.com>
wrote:

> @XhmikosR <https://github.com/XhmikosR> will revert this change, see
> @sindresorhus <https://github.com/sindresorhus> yeoman/grunt-filerev#76
> (comment)
> <https://github.com/yeoman/grunt-filerev/pull/76#issuecomment-78427423>
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/stephanebachelier/grunt-usemin/commit/f8640f00f942e173b0bcdde0d84db960fee6ea12#commitcomment-10159359>
> .
>


Женька! В твой город Сидней вчера (11.03.15) полетел наш товарищ https://www.facebook.com/sergey.glotov
Он пытался связаться тобой. Не получилось. Напиши ему в фейсбук, или камментом ниже :)) Как с тобой связаться :)))


#254 still occur on my client using 9dc5a1a


I have absolutely no idea. In general I'd say that if some code doesn't make sense, and there's no comment or commit message explaining it, then feel free to get rid of it. :)


rm tab


Okay but in this case I would not put the core together with the other bundles, instead I would put them in its own repo.
So in this case I would show a `simple` way.

BTW: the `build.scripts.require` should not be appended into every bundle only in the core, this can cause some errors, because require.modules will be overwritten each time. Maybe this is the problem for https://github.com/componentjs/bundler-example/issues/5#issuecomment-78547512


Good flame! Will be addressed with some pending change


Add the sections that you can actually test in that case, and I'll sort out the rest. Mainly I think that is avoiding `inet_ntop()` and friends. Those were added in Windows Vista, whilst `getaddrinfo()` and `getnameinfo()` were available from XP.


Hi it seems good work! I'll try it! Just a style question, why did you use capital package names? Is this an Android convention? Coming from "pure" Java world I never saw this...


here is where the mapping for the thumbnail should be. @anarchivist can you help?


We should get the bitcoin configuration parameters here too, in particular rpcpassphrase hopefully has a fair amount of entropy.


@chrisrogers could you check to see if it looks ok removing the span3 and using owlCarousel?


That means the constants are added every time the `addContentElementTyposcript()` method is called. Do you think I'd be a good idea to have a static array indicating if the constants for an `$_EXTKEY` were already added to prevent multiple adding of the constants?


this is pretty cool.
for selecting from lists what about defining the existential and universal quantifiers?:
```
def thereExists(List[T] l) : T = ???
def forAll(List[T] l) : T = ???
```
(we won't call them but will look for them in the macro)
The predicate now becomes:
```
Predicate[User](forAll(_.getAccounts).getAmount > 10)
Predicate[User](thereExists(_.getAccounts).getAmount > 10)
```
which seems to clarify the meaning of filtering on the content of a list.


We should use proper links on the ms syntax. Then it works on other formats as well


https://github.com/Araq/Nim/blob/devel/tools/niminst/niminst.nim#L466

The main file is assumed to be the same as the ini, but with a `.nim` ending instead. If this behaviour should be changed instead, we would have to specify that `compiler/nim.nim` is the main file somewhere.


 > the reason for my using bundles was to have standalone builds

I don't think the bundlerr is build to provide standalone bundles, it's for the case when you want to share and split components via different files.


test


Никогда не делай приписки AZ к классам и проекту.


Thats fine. Feel free to revert & PR.


@anselmdk I don' think this should be here either.


Такие комментарии не несут стоящей информации, лучше удаляй их.


Please follow the consistent coding style.


90210372-ChillerElectricEIR-still-delivering-chilled-water-#4453 (lgu1234) - x86_64-MacOS-10.9-clang: OK (1649 of 1649 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)


An obvious piece of environment data for WIN32 which we've used in the past would be a screenshot. UID/PID is probably still actually useful on WIN32 as well, as maybe the fstat data for "." and "/".


Would it be faster (without introducing security issues) to check `isCanonicalEd25519Signature` before `ed25519_sign_open`?


That line (`add_foreign_key`) is unexpected. The command `rails g model Comment …` should not have added that. It's not a threat to your application or anything, but we don't actually need it. I'll search how/why it ended up in the migration file :)


@rouge8 ok... fixed the spaces


Ты либо ставь перевод строки после if, for, либо не ставь.


x


y


additionalSlinkyGHXWork (mmatts) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: Tests Failed

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


as a client, what should be included in the template and what should be left out is not really clear w/o looking at the impl.


should fix this: newSignHandler --> newHandler


А в .h прописать прототип функции?


You could also add a newline at end of file for a perfect style :)


We might want a comment here about the goals:

We want data that has diversity across the user-base, even if it's not high entropy, because with the strengthening it at least functions as salt, and of course as much actual unpredictableness as possible. We want to avoid functions which are gratuitously non-portable, or which would be frustrating to reasonable sandboxing efforts or cause concern to someone stracing the process (e.g. we've avoided trying to record background audio; the files we read are files pretty much every daemon reads, etc).


"else { if() {..}}" == "else if() {...}"


isSameScopeDescendentOf


all tests pass:
success: http://build.servo.org/builders/linux1/builds/994
success: http://build.servo.org/builders/linux2/builds/993
success: http://build.servo.org/builders/android/builds/1028
success: http://build.servo.org/builders/mac1/builds/1209
success: http://build.servo.org/builders/mac2/builds/1210
success: http://build.servo.org/builders/gonk/builds/180


@ragesoss Regenerating my fixtures did change the number to 1919. I'm not quite sure why... the end date for the course is in June of 2014. Updating this test for now and keeping an eye out.


ForInStatement | ForOfStatement


Из названия переменной r понятно лишь то, что у тебя что-то не в порядке.


This is not a POSIX function, so we might want to have it behind a feature check.


ow, damn it. then disable the whole npm bower stuff on opensuse.. :)
why is it needed again?


Методы не обязательно прописывать, а вот свойства нужно.


И где ты используешь это?


The extra space is mine, I think. Done. PR?


Yeah will add some doc. to a large extent it matters if you set Defaults when you register your workflow type, if you did the template only needs the WorkflowType. if not you'll have to fill out more.


great


Вс идея с таким родителем и последующим наследованием – это выстрел в ногу.


My bad.


Good! Can you use lowercase in assembly?


Hey, sorry I didn't respond for so long. Been asleep and trying to switch to Linux(distro recommendations needed btw) Here is where the code is from: http://simplestcodings.blogspot.com/2010/10/simple-logger-in-c.html. This is temporary, I just felt bad for not finishing before today, and wanted SOMETHING to get in there. Should hopefully be done with a good one soon.


fast-forwarding master to auto = 8e811229


else
{
}
}
}
}
}
}
else
(c)


would be nice to have a typed error for this


Both suggestions are implemented in the following two commits.


I think this needs to be `>= "4.03.0"` now.


You don't have to keep `fib_v1.s` since you are using GIT. Instead, you can create a new GIT branch to keep the implementation.


@ntdb I've been poking, and at that point, there are three courses instead of the one expected. I can't figure out why, though. Maybe some other test adds courses to the database and then they don't get cleaned up? After the whole test suite is done, there's nothing in the testing database, though.


So, please create a new branch and delete the file `fib_v1.s`.


Имя метода не отображает сути метода. Если возвращаемое значение BOOL, тогда метода название начинай с приставки is. (isDead, for example)


Thank you, forgot to hide portal filters when  the custom filter is not set... It needs to be done in an `if` so I'll add it soon :). And about `exec`, sorry, i don't understand what's wrong with that? Exec should change the sh process to phantomjs without staying in RAM


Please follow the consistent coding style.


@emilioeduardob Go ahead and remove it. You may need to apply a set width to the video-slide-wrapper class. If you get the slider working and have it saved to a branch, I can pull and clean up any styling issues. Maybe take a look at what was done with the Owl Carousel in the Channel video select.


Лучше бы просто у Лучника в классе дополнительный метод прописал.


minor, but would be better using appendf


There was a new bower module/state written and it came with tests. Ideally, we want to run the tests on all distros, but clearly there are some issues here. So we can just add openSUSE to list of "don't run these tests" in the salt-jenkins state like some of the other distros are.


saw approval from jdm
at https://github.com/yodalee/servo/commit/ff76b2f29030778d135395d35c9cdeac77df3279


merging yodalee/servo/issue4768-textencoder = ff76b2f2 into auto


yodalee/servo/issue4768-textencoder = ff76b2f2 merged ok, testing candidate = 9640c838


Diff-friendly would depend on the parser, as the JSON spec does not allow trailing commas (IIRC). It looks like the parser in use fits this description: `ValueError: Expecting property name`

I get irate pretty quickly with a configuration file that can't support comments, because there's inevitably an unintuitive reason a value might be a specific value, or temporarily toggled to another value, etc. In my experience separating that out from the config file is a mistake; the parser in question has the same `ValueError` for this case also.

Flipping the conversation for a minute, is there another upside to switching beyond the pip dependency? If we were trying for zero non-stdlib dependencies it would make more sense to me, but in this case [the toml library in question](https://github.com/uiri/toml) is dead-quiet and I don't exactly anticipate sweeping change anytime soon.


Even after above patch, i am still seeing following error in my logs, Everything works but why it is adding following logs in logs file.  I have homer running on kamailio. do you think that would be the issue?

ERROR:siptrace:pipport2su: bad protocol
ERROR:siptrace:pipport2su: bad protocol
ERROR:siptrace:pipport2su: bad protocol


90210372-ChillerElectricEIR-still-delivering-chilled-water-#4453 (lgu1234) - i386-Windows-7-VisualStudio-12: OK (1655 of 1655 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)


Looks good. My testing confirms it's working.


Sadly, I don't think it can be -- partly because `cbegin()` isn't available (that's [DM-2009](https://jira.lsstcorp.org/browse/DM-2009)) and partly because auto will baulk at deducing different types. Which is s shame, because I agree it would be nicer.

The assertion is a good idea -- thanks!


"Магические" константы – плохой стиль. Что за 50, 5, 10, 15? Определяй через макросы, или сделай категорию.


Perhaps make WorkflowType an explicit param? however, why can't it get this from the FSM?


@jserv  fixed.


Do we expect the type of that 3rd field to be the type of amount (Int?) or to be a list of them (List[Int]) ?
if you intend to flatten you could define flatten() to make dealing with lists clearer:
```
def flatten(List[T] l) : T = ???
```
the projection would become
```
flatten(_.getAccounts).getAmount
```
if not flattening you could define forAll
```
def forAll(List[T] l) : T = ???
forAll(_.getAccounts).getAmount
```


Delete this line


This is better. I think adding an autoImport would be the icing on the cake:

```scala
object autoImport = {
  object Versioning {
    val Semantic = SemanticVersion
    val Date = DateVerions
  }
}
```

I'll let you decide.


For the record, I'm assuming this `order` value is magic, and I'm assuming it's correct. If there's a reference somewhere, I'd be glad to check that you transcribed it correctly, or if there's an algorithm, I'll check your math. :)


@anselmdk its on line 1.

If you want to compile css into your bootstrap_mysite folder, then I've made this value updatable so that references to images, fonts etc will still work.


Hi thanks for your comment. Since i just added line 48 but didn't touch 47, i think your issue does not rely on my changes. But please feel free to open a new issue and maybe even provide a bugfix :)


basestring does not exist in python3. you should use string_types from kivy.compat.


Yes, all the defaults are filled in the $field array, so it should be fine. As long as a form has been updated since you added those settings, you can check it for errors.


Moi ça me gêne pas non plus dans la logique mais je crois que j'ai confondus avec la règle avec celle des schémas de BDD où la, il faut pas.


Sounds good; we can clean this up later then.


Complete, only 1 comment


some tests failed:
failure: http://build.servo.org/builders/linux1/builds/995
failure: http://build.servo.org/builders/linux2/builds/994
failure: http://build.servo.org/builders/mac2/builds/1211


Ah, I think of "intializer lists" as just being {{std::initializer_list}}, which is of course related, but not quite the same thing.  Good to know that it's at least possible with gcc 4.4, as per your experiment.

I was surprised to hear Stroustrup's opinion at first (and if you've got a reference, I'd be interested in reading it), but thinking about it again, I shouldn't have been.  Braced initializers were clearly supposed to be the solution to all of the constructor syntax problems in C++11, but then (IMO) they were basically torpedoed by the horrible interaction with {{std::initializer_list}}.  I personally consider the {{std::initializer_list}} problem to be a more serious one than the narrowing and most-vexing-parse problems with parentheses, but I won't really have much of a basis for that opinion until I've really tried using both for a while.

And on policies, I think you're actually not at all outvoted in wanting them to be less strict in general.  You've just stumbled into a few cases on this branch (this one and the namespace aliases) that I think really do need standardization and currently lack it.  Overall, I think we're stuck with lots of horribly strict policies in this department that are slowly getting better by getting more lenient as we find the energy to challenge them, but I think C++11 brings in some new things (this may be the biggest one) where we'd benefit from having a standard.  It may also be that using GitHub for code reviews (which are more globally visible than what we've had before) may make having standards more important, because they're the only way to resolve existing wars between different reviewers.

More than anything, though, I'm raising the "we need a policy for this" flag here frequently because I'm very sensitive to the problem of personal preferences being misrepresented (or even just misinterpreted) as agreed-up policies in code reviews (which isn't to say I haven't been guilty of that myself in the past).


Is this change intentional?


Just added bootstrap and jquery support.


@tenderlove maybe to avoid the collision with the hash of the string, add `1` or something?


The retry must be bounded to a limited number tries.


Aaah, so that explains the new top-level name then :) Alright.

It would be really great to have a better change:commit ratio here to be more readable + reviewable, but for now it's more trouble than its worth to tease this one apart.


Added comments to improve readability.


This function is for fast access to the buffer pool. Of course more functionality will be added but here is the prototype for now.


There are still some small pieces here and there but this should be the last "big" one ;)


Apparently it might depend on system configuration, maybe on kernel. My main desktop computer won't sync and will timeout :(
So it was that or doing dd with conf=fsync option or something like that... I wanted to check lower level docs, but I though i wasn't worth it for this. :stuck_out_tongue_closed_eyes:


Evil bug ...


It makes sense that there are three courses: Course.update_all_courses always checks two IDs ahead to look for new courses. Courses 352 and 353 must both be valid.

It turns out that 433 is the number of revisions for course 351 only. It must be that I had corrupted that cassette at some point while testing. Using 1919 for this test should be solid.


What I would do is adding the template name as meta information within the QuetionType enum


Ok ok, makasih


to much Go :-)


Doesn't match the format of log_print():

    void log_print(char* filename, char *fmt,...)


Ok, so now we have one single web applications handling all requests from multiple GUI, and no cluster problem, but still thread safety problem under the single node.
Let’s consider that multiple GUIs send same requests to backend web server, say controller bean OfferCatalogController which is a singleton, then this singleton bean’s method  will be call  in multiple thread, and when they all want to modify the ConcurrentOfferCatalog which is a property of the bean at the same time, then here comes the thread safety problem, which means that ConcurrentOfferCatalog is making the bean OfferCatalogController stateless, which is a typical spring bean scope caused thread safety problem.

From: SathyaNarayanan Srinivasan [mailto:notifications@github.com]
Sent: Thursday, March 12, 2015 9:57 AM
To: exuugug/SMARTUI
Cc: guang Xu
Subject: Re: [SMARTUI] update ongoingXXXupdates to local hashmap so that no cluster reference (32ebea8)

The GUI is the only application that will ever need a lock and this webapp is a single function application on tomcat. Hence this comment will not be a valid concern for thread safety under the same node.

From: exuugug <notifications@github.com<mailto:notifications@github.com<mailto:notifications@github.com%3cmailto:notifications@github.com>>>
Reply-To: exuugug/SMARTUI <reply@reply.github.com<mailto:reply@reply.github.com<mailto:reply@reply.github.com%3cmailto:reply@reply.github.com>>>
Date: Thursday, March 12, 2015 at 9:54 AM
To: exuugug/SMARTUI <SMARTUI@noreply.github.com<mailto:SMARTUI@noreply.github.com<mailto:SMARTUI@noreply.github.com%3cmailto:SMARTUI@noreply.github.com>>>
Cc: Sathyanarayanan Srinivasan <sathya.narayanan.s@ericsson.com<mailto:sathya.narayanan.s@ericsson.com<mailto:sathya.narayanan.s@ericsson.com%3cmailto:sathya.narayanan.s@ericsson.com>>>
Subject: Re: [SMARTUI] update ongoingXXXupdates to local hashmap so that no cluster reference (32ebea8)

Sorry, even only one spring web server, there are still thread safety issue operating the hashmap, because the OfferCatalogController is not stateless, might need to add synchronization/lock when acquire/release the lock on the hashmap operation part , if multiple web server, then need cluster wide synchronization.

—
Reply to this email directly or view it on GitHub<https://github.com/exuugug/SMARTUI/commit/32ebea890349d55452695ee82a391106f2236ff6#commitcomment-10156020>.

—
Reply to this email directly or view it on GitHub<https://github.com/exuugug/SMARTUI/commit/32ebea890349d55452695ee82a391106f2236ff6#commitcomment-10156042>.


This is a good start - adding an instance variable to hold a clean value from the textEdit


Could you please use vfprintf instead?

    int vfprintf(FILE * restrict stream, const char * restrict format, va_list ap);


Nem precisa continuar, ficou perfeito :+1:


Why not? :)


问题不在这里吧。问题是Erlang显示浮点数的时候是四舍五入的吧


Belum sempat baca referensi mybatis tp udh donlot ebooknya. Misal dibuat abstract class generic tuk DAO class trus semua DAO class inherit abstract class itu apa bisa bro ?


Good point, will refactor this


I lied, basic server is more functional than connection host. still probably just want osvr server though for routing.


Still can’t see the issue…. If Controller is a singleton… then it will have only 1 instance of catalog… which also means only 1 instance of the hash map. Regardless of being stateful or stateless, synchronization and concurency are not related and thus I do not see a connect.

Ankit, can you please commit these changes to smart-nemesis, for me to review the entire scenario?

From: exuugug <notifications@github.com<mailto:notifications@github.com>>
Reply-To: exuugug/SMARTUI <reply@reply.github.com<mailto:reply@reply.github.com>>
Date: Thursday, March 12, 2015 at 10:05 AM
To: exuugug/SMARTUI <SMARTUI@noreply.github.com<mailto:SMARTUI@noreply.github.com>>
Cc: Sathyanarayanan Srinivasan <sathya.narayanan.s@ericsson.com<mailto:sathya.narayanan.s@ericsson.com>>
Subject: Re: [SMARTUI] update ongoingXXXupdates to local hashmap so that no cluster reference (32ebea8)

Ok, so now we have one single web applications handling all requests from multiple GUI, and no cluster problem, but still thread safety problem under the single node.
Let’s consider that multiple GUIs send same requests to backend web server, say controller bean OfferCatalogController which is a singleton, then this singleton bean’s method will be call in multiple thread, and when they all want to modify the ConcurrentOfferCatalog which is a property of the bean at the same time, then here comes the thread safety problem, which means that ConcurrentOfferCatalog is making the bean OfferCatalogController stateless, which is a typical spring bean scope caused thread safety problem.

From: SathyaNarayanan Srinivasan [mailto:notifications@github.com]
Sent: Thursday, March 12, 2015 9:57 AM
To: exuugug/SMARTUI
Cc: guang Xu
Subject: Re: [SMARTUI] update ongoingXXXupdates to local hashmap so that no cluster reference (32ebea8)

The GUI is the only application that will ever need a lock and this webapp is a single function application on tomcat. Hence this comment will not be a valid concern for thread safety under the same node.

From: exuugug <notifications@github.com<mailto:notifications@github.com><mailto:notifications@github.com<mailto:notifications@github.com%3cmailto:notifications@github.com>>>
Reply-To: exuugug/SMARTUI <reply@reply.github.com<mailto:reply@reply.github.com><mailto:reply@reply.github.com<mailto:reply@reply.github.com%3cmailto:reply@reply.github.com>>>
Date: Thursday, March 12, 2015 at 9:54 AM
To: exuugug/SMARTUI <SMARTUI@noreply.github.com<mailto:SMARTUI@noreply.github.com><mailto:SMARTUI@noreply.github.com<mailto:SMARTUI@noreply.github.com%3cmailto:SMARTUI@noreply.github.com>>>
Cc: Sathyanarayanan Srinivasan <sathya.narayanan.s@ericsson.com<mailto:sathya.narayanan.s@ericsson.com><mailto:sathya.narayanan.s@ericsson.com<mailto:sathya.narayanan.s@ericsson.com%3cmailto:sathya.narayanan.s@ericsson.com>>>
Subject: Re: [SMARTUI] update ongoingXXXupdates to local hashmap so that no cluster reference (32ebea8)

Sorry, even only one spring web server, there are still thread safety issue operating the hashmap, because the OfferCatalogController is not stateless, might need to add synchronization/lock when acquire/release the lock on the hashmap operation part , if multiple web server, then need cluster wide synchronization.

—
Reply to this email directly or view it on GitHub<https://github.com/exuugug/SMARTUI/commit/32ebea890349d55452695ee82a391106f2236ff6#commitcomment-10156020>.

—
Reply to this email directly or view it on GitHub<https://github.com/exuugug/SMARTUI/commit/32ebea890349d55452695ee82a391106f2236ff6#commitcomment-10156042>.

—
Reply to this email directly or view it on GitHub<https://github.com/exuugug/SMARTUI/commit/32ebea890349d55452695ee82a391106f2236ff6#commitcomment-10156092>.


:heart:


@shime Ha. What made you switch? I'm really liking the structure of Ruhoh


Ahah! That explains the bump from 5.3 to 5.4, though to be clear, my question was about the very high floor on the minor revision number, not the "parent" revision number. Glad to see you tracked down what was causing your headache, though.


"to compile your files, with optimizations to the compiled files" is redundant. How about "to compile your crates with optimizations"?


ah, yes. it has the triple or more ~ code fence syntax instead. ugh.


yes - ive done this locally


abstract class AbstractDAO<T extends Serializable> {
        public abstract void create(T entity);
        public abstract void edit(T entity);
        public abstract void delete(T entity);
        public List<T> findAll();

        // Dst..
}


Oh! Well, the high minor revision number is mainly because Travis (and Homebrew) gets 5.x.y with y being the highest minor version it goes. As I look more into TravisCI, I might decrease the minor revision number, but I'm not sure how much flexibility I'll have with that.


:+1: LGTM


Thanks for the quick response. :)


@aldenml di mi brazo a torcer...


pero en realidad esto es una maravilla.


A note: This loads MediaWiki:Chat.js and the user's local /chat.js page _before_ a lot of the core chat code. Anything here that errors will break the client spectacularly - see http://i.imgur.com/dbH4OFp.png for an example of this (obtained from rs.wikia.com 2015-03-11)


shouldn't be ortografia and stilo?


ok, I saw this patch has been reversed. I will continue the work in your way :)


I don't think getting data in onCreate is necessarily what we want to do. For example, in ClaimInfoActivity, we might create the activity, then go to the expense list, and then come back. onCreate won't be called again, but the info has changed.


Así como saludo a todos los informáticos


How come? This was super useful for remembering state across reboots on Mac, feeling like all the native apps.


Kevin, how are we going to connect index.html with heatMap.html ? I mean when it is live how are getting from the user from the index page to the heatMap page ?


It looks like version 2.4.2 got published to npm (and possibly also bower) without updating the latest tag.  The new published version is somehow missing its index.js in the root, which is causing node to break when trying to require it.

Even though the latest tag wasn't updated, some packages include version '*' (including dependencies of rx itself), which is causing npm to pull in 2.4.2 instead of the 'latest' tag 2.4.1; and npm'd dedup'ing behavior is causing other packages that reference approximate versions (such as '^2.4.0') to instead use the broken 2.4.2 .

It appears that, at the moment, any package depending on rx is broken if npm install is run without a previous version of rx being cached.  You'll need to publish a new version (2.4.3) because npm won't let you overwrite an existing version.  You may also want to consider using 'npm unpublish' to remove the broken version.


I agree, but some of the activities I didn't want to change around too much to avoid any conflicts with what other people are working on. The way things are currently setup in ClaimInfoActivity I feel is a good way to do it.


:gem: good effort


`filter(Option::isSome)` 無くても良さそうですね。


Je pense qu'il faudrait enlever les fichiers de Visual Studio, car ils se génèrent automatiquement lors de l'ouverture d'Unity.


Might be good to wrap up the `Parent.Parent.Parent.Parent.Parent` into a helper method / property, since it's used in a couple other places and seems like it could easily break if another container widget is added or something like that


Better?


Stuff from the intent can be gotten in onCreate, cause that stuff won't change. Listeners could also be set in onCreate. Pretty much everything else could go in onResume.


What's tests case need onResume function, i think it will call onSystemUiVisibilityChange to enter full screen when it's not full screen after resuming activity.


Well, you can specify the requirement in the composer.json file like so:

"php": ">=5.4.0"

That will allow it to work with any version 5.4 or greater, but without forcing a minor revision number higher than clients might have available in their environment.

Or is your trouble with being able to test with an older minor revision number because of trouble getting your build system to install it?


確かにそうですね


fixes piwik/piwik#7412


Reviewed the code, currently the hashmap could be thread-safety, because before doing put operations on it, we first acquire a Isemaphore from cluster whose initial permits is 1 (if it does is 1), this will assure thread safety by chance.

Will keep an eye on this part to make sure of it.

Thanks

From: SathyaNarayanan Srinivasan [mailto:notifications@github.com]
Sent: Thursday, March 12, 2015 10:10 AM
To: exuugug/SMARTUI
Cc: guang Xu
Subject: Re: [SMARTUI] update ongoingXXXupdates to local hashmap so that no cluster reference (32ebea8)

Still can’t see the issue…. If Controller is a singleton… then it will have only 1 instance of catalog… which also means only 1 instance of the hash map. Regardless of being stateful or stateless, synchronization and concurency are not related and thus I do not see a connect.

Ankit, can you please commit these changes to smart-nemesis, for me to review the entire scenario?

From: exuugug <notifications@github.com<mailto:notifications@github.com<mailto:notifications@github.com%3cmailto:notifications@github.com>>>
Reply-To: exuugug/SMARTUI <reply@reply.github.com<mailto:reply@reply.github.com<mailto:reply@reply.github.com%3cmailto:reply@reply.github.com>>>
Date: Thursday, March 12, 2015 at 10:05 AM
To: exuugug/SMARTUI <SMARTUI@noreply.github.com<mailto:SMARTUI@noreply.github.com<mailto:SMARTUI@noreply.github.com%3cmailto:SMARTUI@noreply.github.com>>>
Cc: Sathyanarayanan Srinivasan <sathya.narayanan.s@ericsson.com<mailto:sathya.narayanan.s@ericsson.com<mailto:sathya.narayanan.s@ericsson.com%3cmailto:sathya.narayanan.s@ericsson.com>>>
Subject: Re: [SMARTUI] update ongoingXXXupdates to local hashmap so that no cluster reference (32ebea8)

Ok, so now we have one single web applications handling all requests from multiple GUI, and no cluster problem, but still thread safety problem under the single node.
Let’s consider that multiple GUIs send same requests to backend web server, say controller bean OfferCatalogController which is a singleton, then this singleton bean’s method will be call in multiple thread, and when they all want to modify the ConcurrentOfferCatalog which is a property of the bean at the same time, then here comes the thread safety problem, which means that ConcurrentOfferCatalog is making the bean OfferCatalogController stateless, which is a typical spring bean scope caused thread safety problem.

From: SathyaNarayanan Srinivasan [mailto:notifications@github.com]
Sent: Thursday, March 12, 2015 9:57 AM
To: exuugug/SMARTUI
Cc: guang Xu
Subject: Re: [SMARTUI] update ongoingXXXupdates to local hashmap so that no cluster reference (32ebea8)

The GUI is the only application that will ever need a lock and this webapp is a single function application on tomcat. Hence this comment will not be a valid concern for thread safety under the same node.

From: exuugug <notifications@github.com<mailto:notifications@github.com<mailto:notifications@github.com%3cmailto:notifications@github.com>><mailto:notifications@github.com<mailto:notifications@github.com%3cmailto:notifications@github.com<mailto:notifications@github.com%3cmailto:notifications@github.com%3cmailto:notifications@github.com>>>>
Reply-To: exuugug/SMARTUI <reply@reply.github.com<mailto:reply@reply.github.com<mailto:reply@reply.github.com%3cmailto:reply@reply.github.com>><mailto:reply@reply.github.com<mailto:reply@reply.github.com%3cmailto:reply@reply.github.com<mailto:reply@reply.github.com%3cmailto:reply@reply.github.com%3cmailto:reply@reply.github.com>>>>
Date: Thursday, March 12, 2015 at 9:54 AM
To: exuugug/SMARTUI <SMARTUI@noreply.github.com<mailto:SMARTUI@noreply.github.com<mailto:SMARTUI@noreply.github.com%3cmailto:SMARTUI@noreply.github.com>><mailto:SMARTUI@noreply.github.com<mailto:SMARTUI@noreply.github.com%3cmailto:SMARTUI@noreply.github.com<mailto:SMARTUI@noreply.github.com%3cmailto:SMARTUI@noreply.github.com%3cmailto:SMARTUI@noreply.github.com>>>>
Cc: Sathyanarayanan Srinivasan <sathya.narayanan.s@ericsson.com<mailto:sathya.narayanan.s@ericsson.com<mailto:sathya.narayanan.s@ericsson.com%3cmailto:sathya.narayanan.s@ericsson.com>><mailto:sathya.narayanan.s@ericsson.com<mailto:sathya.narayanan.s@ericsson.com%3cmailto:sathya.narayanan.s@ericsson.com<mailto:sathya.narayanan.s@ericsson.com%3cmailto:sathya.narayanan.s@ericsson.com%3cmailto:sathya.narayanan.s@ericsson.com>>>>
Subject: Re: [SMARTUI] update ongoingXXXupdates to local hashmap so that no cluster reference (32ebea8)

Sorry, even only one spring web server, there are still thread safety issue operating the hashmap, because the OfferCatalogController is not stateless, might need to add synchronization/lock when acquire/release the lock on the hashmap operation part , if multiple web server, then need cluster wide synchronization.

—
Reply to this email directly or view it on GitHub<https://github.com/exuugug/SMARTUI/commit/32ebea890349d55452695ee82a391106f2236ff6#commitcomment-10156020>.

—
Reply to this email directly or view it on GitHub<https://github.com/exuugug/SMARTUI/commit/32ebea890349d55452695ee82a391106f2236ff6#commitcomment-10156042>.

—
Reply to this email directly or view it on GitHub<https://github.com/exuugug/SMARTUI/commit/32ebea890349d55452695ee82a391106f2236ff6#commitcomment-10156092>.

—
Reply to this email directly or view it on GitHub<https://github.com/exuugug/SMARTUI/commit/32ebea890349d55452695ee82a391106f2236ff6#commitcomment-10156117>.


If you have a list or tuple of possible states (e.g., states=('a','c','g','t')), you could make the code more compact by using an index lookup like this - states.index('g') - to find the index, rather than writing a separate test for each state.


Test pull request


:trollface:


@SthephanShinkufag Ай-яй. Обнови бабел.


I agree that renaming `bytes` to `repeat` or `repCount` would make sense.


Please follow the consistent coding style.


It might be a bit cleaner to make 'void src*' a constant. Also, maybe consider doing things a little lower-level; using bitwise operators and such, I mean. If you can't find anything else, I found this source code: http://www.opensource.apple.com/source/xnu/xnu-2050.18.24/libsyscall/wrappers/memcpy.c  Yes, I know the website is centered around Apple products (the domain name's a bit of a give-away). But, the code looks quite professional. Otherwise, if your solution here is indeed faster, I'd suggest renaming it, as memcpy() is a standard function and certain people may expect it to work in a certain way (that way not being this large-data-optimized method). But don't get me wrong, your code looks fine to me as-is.


Ditto. Please follow the consistent coding style.


Well, you moved a bunch of the data-getting stuff in ClaimInfoActivity from onResume to onCreate. So are you okay with me moving it back?


Thanks! When you have time, could you force [OPENGL_CHECK_ERRORS](https://github.com/Narann/mupen64plus-video-rice/blob/master/src/OGLDebug.h#L25) definition here and pastebin the resulting log? I know it's related to shader compilation but I have troubles to retrieve the info log (see [url=https://www.opengl.org/discussion_boards/showthread.php/185796-Info-log-of-a-failed-shader-compile-0��t-What-is-wrong?p=1264830]here[/url]). As soon as I know why compilation fail I should be able to fix them quickly!


Would you like to give explicit name to each new task?


Did I? My mistake then. Go ahead and move it back.


> Or is your trouble with being able to test with an older minor revision number because of trouble getting your build system to install it?

Yeah, this is the problem that I'm having. phpenv doesn't seem to like OSX Yosemite too much. Plus, not having Travis run the tests on something like 5.x.0 doesn't help. I'll look into it though - thanks! :smiley:


Hmmm.... Image sizes *should* be optimisable. I'll see what I can do later today. Retina iPad has a lot of available resolution though. :confused:


This line here breaks the link to users.md, I am too lazy to make a pull though


Shouldn't we have a test for this?


LinqPadでうごかないよ
:-1:


Great work! Can you introduce buid-time configurable macros to analyze the timing information? So that, you only have to maintain one source tree.


That is good. You are using function pointer to beautify the source.


@Elsealabs

Callbacks are like event handlers, but a proper definition is that a callback is a function which is passed as an argument to another function which is expected to be called back at some point of time. In C, and GLFW, these callbacks are implemented as function pointers, and when set, the GLFW calls them whenever an event happens.

```c
void myWindowFocusCallback(GLFWWindow* window, int focus)
{
    // This is the native C callback function
}

// Register the callback function with GLFW
glfwSetWindowFocusCallback(myWindow, myWindowFocusCallback);
```

Now with these functional interfaces and Java 8 Method references, it is possible to port existing C code easily to Java. You can see how that makes the code looks identical except to some added OOP by me.

```java
private void myWindowFocusCallback(Window window, boolean focus)
{
    // This is the Java callback method
}

// Register the callback method with GLFW using method references
myWindow.setFocusCallback(this::myWindowFocusCallback);
```

This looks the same as with the native version, and is easier to code. Given an auto-complete IDE, it is even more faster to write that. If you are still unsure, I have written a complete article on this, see http://goharsha.com/lwjgl-tutorial-series/window-callbacks/


I think it would be more readable if above you had:
```
if (isRequestForRandomTitle(request.query)) {
  Article.getArticle() //...
  return;
}

Article.getData() /...


Calculating the pmatrix by matrix exponentiation is costly, so it's best to do it just once and then simply lookup the relevant entry for each site. I'd put this line just before the for loop across sites.


m


Same as above - I'd move the calculation of the p-matrix outside the for loop across sites.


このパターン名って「バランスボードサイクル」だった気がするんですけど、清書する段階で変更になったのでしょうか？


Yes, udah saya bikin, belum commit, weekend ya :), cuman bedanya mapping mybatis ndak perlu definisiin mapper pake kelas melainkan interface. Yang ada di local gw pake generic macam berikut (inspirasi dari spring-data)

public interface Repository<T, I> {
void save(T entity);
void save(Collection<T> entities);
findAll();
findOne(I id);

...dst...
}


It's for device with virtual home key. The listener was not fired after resuming activity.


Switched from Google map.


Hi, this looks like a good approach. My only hesitation is that 'number-source' is inconvenient to type. I understand the logic behind it -- you might replace the numeric source with a synth that has arbitrary control names, so the numeric source should have a distinct name -- but it would be helpful if this could be friendlier toward live-coding. Maybe `'#'` instead of `'number-source'`? Or perhaps `addSpec` could resolve nil to 'number-source' if the source is a number (but that would be a fix in quarks, not here)?


I might have missed something, but why do you have separate conditions for v==0 and v!=0?


Hang on. Where did you get this code from?


note to self: you can compute the list of packages that import `path` with `go list -f '{{.ImportPath}} {{.Imports}}' ./... | ag '[[ ]path[ \]]'`. We should remove as many of these as we can, because `path/filepath` has more functionality and is also OS-separator-aware.


名前が分かりにくいって誰かからフィードバックもらったと思うんですが、変えなきゃいけないと思いつつこの名前も仮のものです。最終的にもう一度相談したいと思ってました。何かいい名前ありませんかね…


@qrush I think you meant :metal:


rax retry tests


Thanks for the useful commit!  We will review this asap.


Forginve my wordy, if we use hashmap, there still will exist thread safety issue.
The Isemaphore acquire can only make thread safety for two/multiple thread editing same offer/resource.
There still exists conditions that two request editing two different offer/(other catalog) and trying to do put operations to the same hashmap.
Reference:
Modifying a HashMap without proper synchronization from two threads may easily lead to a race condition.
When a put() leads to a resize of the internal table, this takes some time and the other thread continues to write to the old table.
Two put() for different keys lead to an update of the same bucket if the keys' hashcodes are equal modulo the table size. (Actually, the relation between hashcode and bucket index is more complicated, but collisions may still occur.)

I think we should consider using ConcurrentHashMap.

From: SathyaNarayanan Srinivasan [mailto:notifications@github.com]
Sent: Thursday, March 12, 2015 10:10 AM
To: exuugug/SMARTUI
Cc: guang Xu
Subject: Re: [SMARTUI] update ongoingXXXupdates to local hashmap so that no cluster reference (32ebea8)

Still can’t see the issue…. If Controller is a singleton… then it will have only 1 instance of catalog… which also means only 1 instance of the hash map. Regardless of being stateful or stateless, synchronization and concurency are not related and thus I do not see a connect.

Ankit, can you please commit these changes to smart-nemesis, for me to review the entire scenario?

From: exuugug <notifications@github.com<mailto:notifications@github.com<mailto:notifications@github.com%3cmailto:notifications@github.com>>>
Reply-To: exuugug/SMARTUI <reply@reply.github.com<mailto:reply@reply.github.com<mailto:reply@reply.github.com%3cmailto:reply@reply.github.com>>>
Date: Thursday, March 12, 2015 at 10:05 AM
To: exuugug/SMARTUI <SMARTUI@noreply.github.com<mailto:SMARTUI@noreply.github.com<mailto:SMARTUI@noreply.github.com%3cmailto:SMARTUI@noreply.github.com>>>
Cc: Sathyanarayanan Srinivasan <sathya.narayanan.s@ericsson.com<mailto:sathya.narayanan.s@ericsson.com<mailto:sathya.narayanan.s@ericsson.com%3cmailto:sathya.narayanan.s@ericsson.com>>>
Subject: Re: [SMARTUI] update ongoingXXXupdates to local hashmap so that no cluster reference (32ebea8)

Ok, so now we have one single web applications handling all requests from multiple GUI, and no cluster problem, but still thread safety problem under the single node.
Let’s consider that multiple GUIs send same requests to backend web server, say controller bean OfferCatalogController which is a singleton, then this singleton bean’s method will be call in multiple thread, and when they all want to modify the ConcurrentOfferCatalog which is a property of the bean at the same time, then here comes the thread safety problem, which means that ConcurrentOfferCatalog is making the bean OfferCatalogController stateless, which is a typical spring bean scope caused thread safety problem.

From: SathyaNarayanan Srinivasan [mailto:notifications@github.com]
Sent: Thursday, March 12, 2015 9:57 AM
To: exuugug/SMARTUI
Cc: guang Xu
Subject: Re: [SMARTUI] update ongoingXXXupdates to local hashmap so that no cluster reference (32ebea8)

The GUI is the only application that will ever need a lock and this webapp is a single function application on tomcat. Hence this comment will not be a valid concern for thread safety under the same node.

From: exuugug <notifications@github.com<mailto:notifications@github.com<mailto:notifications@github.com%3cmailto:notifications@github.com>><mailto:notifications@github.com<mailto:notifications@github.com%3cmailto:notifications@github.com<mailto:notifications@github.com%3cmailto:notifications@github.com%3cmailto:notifications@github.com>>>>
Reply-To: exuugug/SMARTUI <reply@reply.github.com<mailto:reply@reply.github.com<mailto:reply@reply.github.com%3cmailto:reply@reply.github.com>><mailto:reply@reply.github.com<mailto:reply@reply.github.com%3cmailto:reply@reply.github.com<mailto:reply@reply.github.com%3cmailto:reply@reply.github.com%3cmailto:reply@reply.github.com>>>>
Date: Thursday, March 12, 2015 at 9:54 AM
To: exuugug/SMARTUI <SMARTUI@noreply.github.com<mailto:SMARTUI@noreply.github.com<mailto:SMARTUI@noreply.github.com%3cmailto:SMARTUI@noreply.github.com>><mailto:SMARTUI@noreply.github.com<mailto:SMARTUI@noreply.github.com%3cmailto:SMARTUI@noreply.github.com<mailto:SMARTUI@noreply.github.com%3cmailto:SMARTUI@noreply.github.com%3cmailto:SMARTUI@noreply.github.com>>>>
Cc: Sathyanarayanan Srinivasan <sathya.narayanan.s@ericsson.com<mailto:sathya.narayanan.s@ericsson.com<mailto:sathya.narayanan.s@ericsson.com%3cmailto:sathya.narayanan.s@ericsson.com>><mailto:sathya.narayanan.s@ericsson.com<mailto:sathya.narayanan.s@ericsson.com%3cmailto:sathya.narayanan.s@ericsson.com<mailto:sathya.narayanan.s@ericsson.com%3cmailto:sathya.narayanan.s@ericsson.com%3cmailto:sathya.narayanan.s@ericsson.com>>>>
Subject: Re: [SMARTUI] update ongoingXXXupdates to local hashmap so that no cluster reference (32ebea8)

Sorry, even only one spring web server, there are still thread safety issue operating the hashmap, because the OfferCatalogController is not stateless, might need to add synchronization/lock when acquire/release the lock on the hashmap operation part , if multiple web server, then need cluster wide synchronization.

—
Reply to this email directly or view it on GitHub<https://github.com/exuugug/SMARTUI/commit/32ebea890349d55452695ee82a391106f2236ff6#commitcomment-10156020>.

—
Reply to this email directly or view it on GitHub<https://github.com/exuugug/SMARTUI/commit/32ebea890349d55452695ee82a391106f2236ff6#commitcomment-10156042>.

—
Reply to this email directly or view it on GitHub<https://github.com/exuugug/SMARTUI/commit/32ebea890349d55452695ee82a391106f2236ff6#commitcomment-10156092>.

—
Reply to this email directly or view it on GitHub<https://github.com/exuugug/SMARTUI/commit/32ebea890349d55452695ee82a391106f2236ff6#commitcomment-10156117>.


finally! +1


LGTM. Merging outside of core because this one is still bleeding. (Sorry I didn't get to it earlier.)


A comment by @kronn: https://github.com/jenkinsci/github-api-plugin/commit/f3e9b90a6a70ece340ca5f587c49442df2e25311#commitcomment-10137387

Should we update `project/scm/tag`?


:boom: :+1; :heart:


Awesome - Thanks for all your commits!


Since the largest migration we support is from 1.4 -> 2.1, we can remove this migration now.


Thank you for pointing it out.
I added a comment at https://github.com/jenkinsci/github-api-plugin/commit/80ef3d5d3ce0f483a26eeedcee5d500d2e550c80#commitcomment-10156457


Uh, I looked through, and the only data being gotten in an onCreate when it shouldn't be is in ExpenseItemInfoActivity with the call to populateExpenseInfo(). All the getting of UserData and the claim UUID can be done in onCreate, cause those won't change. A claim's info will probably change, but its UUID won't. So I'm not sure what you're referring to.


Hm... You're right, actually. A reference to the claim is returned, but if it gets changed, the activity will be pointing to the right claim anyway. My mistake.


I think you want to write this as an if...elif...else statement. As currently written, the else clause will be activated and diff will be cut in half anytime that likeUp is not greater than likeCurr.


porfesor aqui estan todos los codigos espero su respuesta


Sounds good; opened #9


some tests failed:
failure: http://build.servo.org/builders/linux1/builds/995
failure: http://build.servo.org/builders/linux2/builds/994
failure: http://build.servo.org/builders/mac1/builds/1210
failure: http://build.servo.org/builders/mac2/builds/1211
failure: http://build.servo.org/builders/gonk/builds/181


What's your compiler ?


service-worker.js is empty


brak przecinka po nawiazie zamykającym:
},


If this is adding much entropy, we need a better AES implementation. :)


and also how these .properties files should be encoded: utf-8 and utf-8 without BOM ?


@jfindlay You're misunderstanding me. I know full well the file name comes in through the source parameter. My solutions precludes the difficulty you're talking about with *prepending `x`*. In my proposed solution you let the user specify all options, including `x`, because they are given a placeholder they can use with `f`, such as `{}` that will be replaced with the filename passed through the `source` parameter. i.e. `tar_options: xf {} --foo bar`


You might want to check with mbiarnes as to why you had to update these version numbers manually. AFAIK they should be updated as part of the release process - if kie-uberfire-extensions is configured properly.


Go ahead

Sent from my iPhone

> On 12 Mar 2015, at 6:12 pm, sezero <notifications@github.com> wrote:
>
> @chrisisonwildcode : This won't compile for djgpp. Would you allow me to push to this branch to fix it?
>
> —
> Reply to this email directly or view it on GitHub.
>


Only one form element in a document can have autofocus set.


Is there any chance that the target sprite would be null?


asdas


89911514-Allow-OtherSideCoefficients-in-AFN (jasondegraw) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: Coverage Too Low

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


My bad.


Решил скоротить код.


I meant add this before:
This does not support server with abnormal health levels. Damageable has a method *getMaxHealth()* you could use to add this support.


jd


Pls fix this. not working for me


Это была тренировка использования опциональных свойств.


I'm trying to find the latest version of the mediawiki-parser, if there is such a thing.  Do you have an overview how current your branch is compared to the other branches?  Is your branch a working version or more experimental?


VSHPWH (mbadams5) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: OK (1148 of 1150 tests passed)

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


michael that is my feature branch:  social_cluster


Почему? Так гараздо приятнее глазу.


Updating release wrapping scripts, I've discovered that this file has been removed accidentally - it is needed to unpack the `tools.tar.gz` which is part of the GAP distribution and contains the following "tools":
```
Unpacking tools.tar.gz archive ...
x etc/convert.pl
x etc/README.tools
x etc/patternstextbinary.txt
x etc/patternscolor.txt
x etc/packpack
x etc/buildman.pe
x etc/unzoo.c
x etc/classifyfiles.py
x doc/gapmacrodoc.tex
x doc/gapmacro.tex
x doc/manualindex
x doc/gapmacrodoc.pdf
Done!
```
I am going to restore it in the master branch therefore.


thanks that hard work


When will e be null?


@danstocker :smile:


wooow (y)


After your comment, I change file with vim.
But I don't know why it is not arranged on github. Werid.
![screen shot 2015-03-13 at 4 09 19 am](https://cloud.githubusercontent.com/assets/4955035/6627113/cb345e34-c936-11e4-8b4d-240c6e208946.png)


Impressive work - nice job!


@Bilge, that's true.  We at one point had `x` removed from `tar_options`, but since the state function is supposed to extract by default, it makes sense that the user shouldn't have to specify `x` in order to actually extract the archive.


Without, which is what it is now. (For UTF-8, without is much more common.)

Pull requests would be great, thanks.


Perhaps it would be better to take the remaining 224 bits of C as 9 starting positions, and repeat a smaller graph walk 9 times, once from each of those positions?   Might reduce the probability that you hit a 2-cycle and get much less entropy out of it (alternatively or in addition, the walking loop could have a Floyd's algorithm test to decide if its in a cycle and then break out by taking another starting position).


Looks good


Это не константы. Это дефолтное значение, его можно изменить в файле AZAppDelegate.m


This commit introduces a "last-inclusion" macro header to prevent silent changes of library headers


Good stuff!


I only pose it as a suggestion because this has been sitting idle for more than two weeks as soon as you mentioned *tests*, the dreaded *T* word. This issue has me version locked to 2014.7.0. If I knew the first thing about Salt internals or even Python for that matter I'd have submitted a PR for said tests already.


Good point and simple to implement!


Needs arguments passed and properly spelled filename


To hopefully make this a little clearer
{% gist a12d12f505cf014466ec %}


instantiation


To address #12


That's the end goal for sure - have each app as its own repo, build and release them independently. In our transitional phase, as there're a lot of co-dependent modules, I chose to go with this method.

For now, each build.js is a standalone app that is lazy-loaded into the larger application. Works fine for now.

My problem in #5 is that the build.css file doesn't get generated at all for the signup module - even though it is dependent on `base-styles` do you think that's related?


I think that is the best until a better nodejs support lands in default opensuse


Collisions isn't a problem given this is just the hash used to bucket the objects anyway.


In

> So in this case I would show a simple way.

In that case, I should restore the original code on your `bundler.js` readme - that only adds require for the first module.


> Those are mine.

Ah great - how is it looking in terms of confirmation speed these days?  I guess really a good test would be to include over 40 bytes in that (currently blank) OP_RETURN output to see how those get confirmed (since it's a non-standard transaction for all but 0.10 based clients)

> Imho all this should be specified first, at least loosely, in particular I'm interested in knowing what's going to happen with obfuscation and the marker, and when which encoding scheme is used.

Well, I thought that had kind of been settled through fringe discussions?  Perhaps I've got the wrong end of the stick - my logic is based on a few things that came up such as:

* It has been stated that getting rid of Exodus at this time is undecided and further suggested that OP_RETURN and removal of the Exodus marker be two separate topics
* While we have the Exodus marker, there is zero value to encoding (obfuscating) the data in the OP_RETURN output for censorship avoidance reasons
* I have yet to be convinced of the value of compression and it needs to be *very* helpful to us to consider including extra dependencies for compression libs (up until now we've always agreed to stick solely with the same deps as Bitcoin Core)

My work queue for 0.0.10 is huge so this leads me to look at OP_RETURN with this KISS principle I keep being told about hehe - what do we *need* for Class C?  In light of not removing Exodus, I see very little reason not to simply plain encode the packet into the OP_RETURN script as per ```033542c944d9a495f6dd3f7f980295cbb3349e3101fe7c4235746a16abf48c05```.  Then (*if*) at a later date it is decided to remove Exodus to improve censorship resistance, that would then be another new class of transaction; say Class D which is an OP_RETURN based Class with the Exodus output removed, a new identification method chosen, and a new method for obfuscating the OP_RETURN data included.

We could look at encoding data now, look at compression now (still not sold :P), and look at a new identification method now and so on, but personally I see zero value to any of that until the decision is made to remove the Exodus marker (which is debatable at present due to loss of ability to pull all transactions just via Exodus eg in mobile clients).

TL:DR; Suggest to simply store the data plainly at this point - unless there are any other factors I haven't considered here :)

Thanks
Z


Thanks.  You mean, two out of three are implemented.  The last one, about using the name, requires a small change in Param, which is annoying since we just released Param.


It didn't seem necessary to keep committing when I just played with the position top and left/rights for two hours, usually I do a lot more commits.


Maybe describe which proc/sys entries since there are a lot.


@jonathanong another question:

> You may create a standalone build, including all dependencies

But the dependencies are not provided for the AMD style, the dependencies for AMD are hardcoded `[]` if you check this line: https://github.com/componentjs/builder2.js/blob/master/lib/builders/scripts.js#L69

They should be provided this way: https://github.com/umdjs/umd/blob/master/returnExportsGlobal.js#L20 or I don't get the UMD stuff?


sorry for those


We should probably use the getrandom()/getentropy() call here, if available. Among other advantages (e.g. no open() failure risk) getrandom can be told to block if and only if the system hasn't adequately seeded itself (E.g. after first boot), which is the behavior we want.  It also can't get mknoded to the wrong device.


So judging by other examples I've found and what I've read this is OK, and we could also choose to drop the GPL header.

The other requirement is to put a copy of the apache2 license in the project, which I still need to do.


89911514-Allow-OtherSideCoefficients-in-AFN (jasondegraw) - i386-Windows-7-VisualStudio-12: OK (1656 of 1656 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)


Why not keep DeepSeaTooth even with Shell Smash? Clamperl desperately needs it, I think, at least more than White Herb.


Do we really want to do this when !slow?  It allocates 16MB memory and thrashes the caches pretty good. :)


Just to follow up on this: I removed the entire second argument in consecutive patches, so this doesn't matter anymore.


This was a mistake and should not be used


Exemplos de *trans-abstract* sem o atributo *xml:lang*


use fixed width
800px


:-)

Sent on the move
On 12 Mar 2015 20:09, "Eder Ignatowicz" <notifications@github.com> wrote:

> michael that is my feature branch: social_cluster
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/droolsjbpm/kie-uberfire-extensions/commit/25ee3f372a71d9bc00691b96a7796be8ee1f5ba5#commitcomment-10172796>
> .
>


When i get home from work, i will most likely be adding a number of issues and comments. I used to do dev work for a server that used this plugin extensively.


Dugtrio still has Reversal, so the check for that move was not redundant.


additionalSlinkyGHXWork (mmatts) - x86_64-MacOS-10.9-clang: OK (1638 of 1649 tests passed)

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


Still might need a DatabaseCleaner in the future, we'll cross that bridge when we get there.


there is nothing difference for me


can we just say this:
```javascript
if (self.uri.indexOf('//') !== -1) {
```


Looks good.


we package it UMD just for exporting. no dependencies are used outside of the build, AMD or otherwise


Why did you need to upgrade the Ruby version?


We should extract all these informations to a config file.


We should extract this to an env variable, or check if Heroku already have an env variable set up for this.


Ha. Didn't see that.


Why?


Really?


Fucking Apache rewrite rules, right? :smirk_cat:


Probably should remove this TODOs if you're going to submit to Pat


This is identical to 1.5. You sure its the right package?


@dwaynebailey These are the instructions to upgrade to 2.7.0, thus this change. You are right that there is a missing step for users trying to upgrade from a version older than 2.7.0.


This doesn't quite have the behavior I'd expect.  Lets assume, for a moment, that the OS entropy is good but fortuna (or our implementation of it) is busted.  I'd expect this function to still return strongly random data, but it wouldn't in that case.

Is there a reason to not return buf[i] = osent[i] ^ GetRandBytes()[i] ?

I believe simplifying in this way would at a minimum make the code easier to audit (because if you were satisfied with GetOSentropy you could just ignore the fortuna completely).


Well, I end up being redirected to /application/application/application/application/application/...

:(


Just wanted you to know, I'll be back in around two hours and going to push some helpers for encoding ('EncodeBareMultisig()', 'EncodeBareMultisigObfuscated()', 'EncodeNullData()', ...), which basically considers all payloads as stream oft data, decoupled of actual obfuscation etc.


Good


Why is OpenShift using Apache anyway?


I've added a license on my repo too: https://github.com/allegrem/pour-qui-faut-il-voter/blob/master/LICENSE.md


This is actually unneeded now that you set it so that it doesn't swap to abilities with ratings 1 and below (its other ability is Cute Charm, which has a rating of 1 and thus won't swap)


alternatively, you could have:
```ruby
@edit[:wf], pre_pov_values = workflow_instance_from_vars(req)
```


I guess you meant https://github.com/allegrem/pour-qui-faut-il-voter/blob/master/README.md


No idea. Maybe so people can struggle with .htaccess like I do :D


1348-IDFMeasureWorkspace (kbenne) - x86_64-MacOS-10.9-clang: Tests Failed

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


89911514-Allow-OtherSideCoefficients-in-AFN (jasondegraw) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: Tests Failed

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


Si vous pouvez avoir une structure sans cycles, comme en BDD, tant mieux. Trop de cycles et des liaisons redondantes deviennent vite gênants. Il faut juste le faire en connaissance de cause.


Nice! A step up from my manual method. :)


needed?


ditto - needed?


@dwaynebailey These are the instructions to upgrade to 2.7.0, and only to that version. If we don't specify the version here, if for some reason a user reads this instructions when Pootle 3.2.0 is out and the version is not specified here, they might not notice they are in the wrong place and also the Pootle package will be upgraded to the wrong version (3.2.0). So IMHO living docs doesn't apply here.


Hi!
What about inserting this code in `vm_mngr.h` instead of `vm_mngr.c`?
There already are some similar definitions:
```C
#ifdef __APPLE__
#define __BYTE_ORDER __BYTE_ORDER__
#elif defined(__NetBSD__)
#define __BYTE_ORDER _BYTE_ORDER
#define __BIG_ENDIAN _BIG_ENDIAN
#define __LITTLE_ENDIAN _LITTLE_ENDIAN
#endif
```

Maybe these kind of define is better placed in a header file.
Can you re-PR?

Anyway, I will test this code as soon as I have access to a mac!


Ok, I put both of these into some test programs just to try the algorithms. Given a fake analog read of "0011111111", the first algorithm chops off the last two bits, whereas the new one rolls it over as I believe you were intending:

$ ./testkey
Seed char 0:11111111
Seed char 1:11111111
Seed char 2:11111111
Seed char 3:11111111
Seed char 4:11111111
Seed char 5:11111111
Seed char 6:11111111
Seed char 7:11111111
Seed char 8:11111111
Seed char 9:11111111

$ ./testkey2
Seed char 0:11111111
Seed char 1:11111100
Seed char 2:11110011
Seed char 3:11001111
Seed char 4:00111111
Seed char 5:11111111
Seed char 6:11111100
Seed char 7:11110011
Seed char 8:11001111
Seed char 9:00111111


what does this do?


90210372-ChillerElectricEIR-still-delivering-chilled-water-#4453 (lgu1234) - Win64-Windows-7-VisualStudio-12: OK (1655 of 1655 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)


for the rendering_engine.js  I think we need add legendlabel_container  and legend_container for use to render legends


It does, we just checked :)


yay. param.to_sym is also a security issue. double win.


alternative instead of augmented?


Reference to the issue : https://github.com/couchbase/couchbase-lite-ios/issues/624


what is the difference between alternatives and augmented?


Isn't there a better way to run this than say set a 0 timeout?


90019124_RadianceData (rpg777) - Win64-Windows-7-VisualStudio-12: OK (2172 of 2184 tests passed)

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


ok, thanks.


there is no way to include the sass file from the messenger directory (I'm guessing this file is a copy from a file there)?


Hello @ZaneDubya , thanks for info. WorldClient and WorldInput done in * 07533b2ea7878f44fb67e6676c2cbef2be3ea037

Now looking at IsometricWorldView.cs


Using awk like this causes awk to buffer its input. This is a problem because the date only gets prepended when the buffer is full. So, if  a job only outputs occasionally, each log entry will get buffered up and then prepended with the same time stamp.

Also, if you ssh into the vm and tail the logs, you have to wait until awk flushes to see the changes.

Is there any chance we could get gawk on the stemcell and do something like "| awk '{ print strftime("%Y-%m-%d %H:%M:%S"), $0; fflush(); }'" which flushes the buffer with each line?


@Deathamns thanks!


I was about to do the same thing! You beat me by a few hours ;)


Sorry, but that doesn't work either.  It just puts the literal string " around the logging file that should be produced.  I think in this case, to properly add quotes around the log file, you should escape the quotes similar to how you wrote line 278.  Resulting code would be:

string.Format(CultureInfo.CurrentCulture, "/l:FileLogger,Microsoft.Build.Engine;{0}verbosity={1};logfile=\"{2}\"", append, this.MultiLogVerbosity, logfileName)

Thanks


Okay, I assumed that.

I'm not sure who is doing it wrong for this case:
there is this lib __switchery__ which has a dependency to __fastclick__.
Both provide a component.json. Fastclick has a UMD in the client script:
https://github.com/ftlabs/fastclick/blob/master/lib/fastclick.js#L829-L840

Switchery just do a `require('fastclick')` in its client script and don't provide a UMD in the (raw) client script.
https://github.com/abpetkov/switchery/blob/master/switchery.js#L19

But the github repo of Switchery provide also a `dist` directory with a compeonnt --standalone build: https://github.com/abpetkov/switchery/blob/master/dist/switchery.js#L1546-L1552

Now @Bartzy is using this `dist` file + RequireJS and it fails.
IMO it's wrong to use the `dist` version of Switchery, if someone want to use it with RequireJS or what do you think?


I will check the issue with #5

> Each app/module has its required scope and doesn't touch other apps/modules or their dependencies.

Okay, but this is not what the readme of the bundler is describing, maybe this is a special case and we can do this version in a extra branch of this repo.

> In that case, I should restore the original code on your bundler.js readme - that only adds require for the first module.

yep


darn
almost feels like
```ruby
src_vm_id = @edit.fetch_path(:new, :src_vm_id, 0) || @src_vm_id || params[:src_vm_id]
```


ok, the only edit I do want you to change:
```ruby
@edit[:new][:src_host_ids] = Array.wrap(params[:prov_id])
```


Uaaah. @jirutka any ideas?

```
$ LANG=C wget http://www.730ne.cz/
--2015-03-12 21:40:13--  http://www.730ne.cz/
Resolving www.730ne.cz (www.730ne.cz)... 54.226.71.16
Connecting to www.730ne.cz (www.730ne.cz)|54.226.71.16|:80... connected.
HTTP request sent, awaiting response... 301 Moved Permanently
Location: http://www.730ne.cz/ [following]
--2015-03-12 21:40:13--  http://www.730ne.cz/
Reusing existing connection to www.730ne.cz:80.
HTTP request sent, awaiting response... 301 Moved Permanently
Location: http://www.730ne.cz/ [following]
--2015-03-12 21:40:13--  http://www.730ne.cz/
Reusing existing connection to www.730ne.cz:80.
HTTP request sent, awaiting response... 301 Moved Permanently
Location: http://www.730ne.cz/ [following]

```


the node.js file used to read all json information from the device server and create the device.js file under /blocks to define new devices. (the new devices won't automatically show on the .html page until the toolbox in demo.html is manually updated)


So `://` means there's a scheme present, which is bad.  As you said before, starting with `//` is bad, but `//` in the middle of a path is OK.  It's not invalid to request `http://example.com/api/test//path`.  Then again it's kind of a weird edge case and it wouldn't work with `request('//path', { baseUrl: 'http://example.com/api/test' })`.  I could go either way on this one.


I think I'll actually put this change in the 1.1.x branch and do a 1.1.6.RELEASE.